NDN-CCL

03/26/2015

03:58 PM Feature #2566 (In Progress): Implement Face.setInterestFilter

See the jNDN branch issue/2566-setInterestFilter. See the updates to the CHANGELOG:<br>
https://github.com/named-dat... Anonymous

12:03 PM Feature #2566: Implement Face.setInterestFilter

In my opinion, this is correct and desired behavior. Alex Afanasyev

11:30 AM Feature #2566: Implement Face.setInterestFilter

"Longest prefix match" question: The ndn-cxx function processInterestFilters calls every callback which matches.<br>
... Anonymous

03:55 PM Task #2579 (In Progress): Provide send/sendData in Face() and deprecate in Transport

See updates in Feature #2566.
Anonymous

10:48 AM Task #2624: Deprecate ndnx packet format support, remove in a later release

Added "remove in a later release" to the Subject since this issue will track through the final removal of NDNx support. Anonymous

10:39 AM Bug #1517: registerPrefix adds the onInterest callback even when registration fails

This applies to all libraries. Set to blocked by test #2624 because the current process to register a prefix is tied ... Anonymous

03/18/2015

12:38 PM Bug #1047: The removeRegisteredPrefix API should "unregister" from the forwarding daemon

Yes, callbacks are missing. Alex Afanasyev

11:10 AM Bug #1047: The removeRegisteredPrefix API should "unregister" from the forwarding daemon

Need to add optional callbacks for success and failure. Anonymous

09:43 AM Feature #2566: Implement Face.setInterestFilter

This form of unsetInterestFilter is a convenience alias for unregisterPrefix. It also matches convenience version of... Alex Afanasyev

07:51 AM Feature #2566: Implement Face.setInterestFilter

unsetInterestFilter(const RegisteredPrefixId* registeredPrefixId) seems to do the same thing as unregisterPrefix(cons... Anonymous

03/17/2015

04:09 PM Task #1921 (Closed): Provide equivalent to NDN_MAX_PACKET_SIZE

Added the check in putData. Anonymous

03/16/2015

03:18 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

Not exactly. Data packet can has other fields that are bundled with the Data (e.g., LocalControlHeader), which needs... Alex Afanasyev

12:53 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

Question: In ndn-cxx, Face.put says "It is highly recommended to use a Data packet that was created using make_shared... Anonymous

01:11 PM Feature #2531 (In Progress): LocalControlHeader

Re-opened and moved to NDN-CCL since this applies to all libraries. It is implemented in jNDN. Need to implement in t... Anonymous

01:03 PM Feature #2532 (In Progress): Remote Prefix Registration

Re-opened and moved to NDN-CCL since this applies to all libraries. It is implemented in jNDN. Need to implement in t... Anonymous

03/12/2015

05:22 PM Task #2620 (Closed): Interest setter methods should return this

Updated the code and docs. Anonymous

03/10/2015

10:36 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

I'm not sure that I'm following the reasoning (I cannot foresee a need for "putInterest") and documentation may not b... Alex Afanasyev

03:56 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

For example, KeyChain.sign is overloaded to sign a data buffer, an Interest and a Data. They all do very different th... Anonymous

04:53 PM Bug #2015 (Closed): name.toUri() does not return a valid URI: missing scheme

Following note 1, added an optional param includeScheme. Anonymous

03:25 PM Task #1921 (In Progress): Provide equivalent to NDN_MAX_PACKET_SIZE

In NDN-CPP branch issue/1921-NDN_MAX_PACKET_SIZE, added static inline method getMaxNdnPacketSize(). Noted that expres... Anonymous

03/09/2015

02:05 PM Feature #2075: The library should have an API to inform the application when the connection is lost to the forwarder

s/architecture/nfd/g. The point of the question is less the phrasing and more a request to specific if/how the libra... Jeff Burke

01:42 PM Feature #2075: The library should have an API to inform the application when the connection is lost to the forwarder

This is not architectural question. This is an artifact of current implementation. If NDN was a networking stack em... Alex Afanasyev

01:15 PM Feature #2075: The library should have an API to inform the application when the connection is lost to the forwarder

Hello Beichuan. We assigned this to you to see if you can bring this up as an architectural question. Should the appl... Anonymous

01:39 PM Bug #2575: Unspecified fields should not be present in packets

Why is it assigned to me?
The only logical way is to not include something into packets if it wasn't specified by ... Alex Afanasyev

12:46 PM Bug #2575: Unspecified fields should not be present in packets

So, if the field is unspecified, it should not show up in the Interest at all? (This is fine, but is this part of th... Jeff Burke

01:36 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

What exactly is the problem with python and javascript? Alex Afanasyev

01:22 PM Task #2579: Provide send/sendData in Face() and deprecate in Transport

We will add Face.putData(data) and deprecate the application's access to Transport.send. Also the OnInterest callbac... Anonymous

01:25 PM Feature #2568 (Closed): Face.put(Data)

Feature #2627 was created for the issue to group high-level operations. And we have task #2579 for adding the method ... Anonymous

01:08 PM Feature #2574: Explicit usage of time abstractions in public API

I disagree with closing of the issues that are open ended and may need to be discussed later. Closing them basically... Alex Afanasyev

12:51 PM Feature #2574 (Closed): Explicit usage of time abstractions in public API

This approach is taken in NDN-CCL for clarity in typeless languages. (Re-open with design motivation and solution th... Jeff Burke

12:58 PM Feature #2567: InterestFilter abstraction

JeffB/JeffT agree. JeffT will look at implementing in jNDN first. May need some discussion. Jeff Burke

12:56 PM Feature #2569: Reconsider necessity for "Node" class

JeffT will document why this exists across NDN-CCL and what the long-term plan is for using it. Then we can discuss. Jeff Burke

12:53 PM Feature #2627: Group high-level operations for Interest/Data under Face

This is internal detail. So far, I talked specifically about LocalControlHeader encoding/decoding. Alex Afanasyev

12:48 PM Feature #2627: Group high-level operations for Interest/Data under Face

Alex, can you make this more specific? Thanks... Jeff Burke

12:40 PM Feature #2627 (Closed): Group high-level operations for Interest/Data under Face

From AlexA: "some of the common high-level operations for Interest and Data packets needs to be grouped under the umb... Jeff Burke

12:53 PM Bug #2572: NDN packet does not define Interest.Selectors.AnswerOriginKind and it should not be used in the code

Sorry, my mistake - I forgot it was in jNDN.
This will be deprecated, then removed as #2624 is completed; a note w... Jeff Burke

12:47 PM Bug #2628 (New): Default value for Interest.Selectors.MustBeFresh should be false

Current implementation, does not follow the semantics of NDN packet format, where by default interests request any da... Jeff Burke

12:35 PM Feature #2577 (Closed): Redesign Signature* classes

Closing unless there is a proposed benefit to the ndn-cxx approach or a drawback to the jNDN approach? (Re-open if t... Jeff Burke

11:34 AM Feature #2566: Implement Face.setInterestFilter

My opinion is that it shouldn't. These are flags for RIB/FIB. The application dispatch can always use InterestFilte... Alex Afanasyev

06:19 AM Feature #2566: Implement Face.setInterestFilter

(a question from a related issue) Should setInterestFilter follow the behavior of child inherit and capture which wer... Anonymous

10:23 AM Task #2368 (Closed): In security library, support KeyParams

KeyParams is added to all libraries. Anonymous

06:39 AM Task #2368 (In Progress): In security library, support KeyParams

Anonymous

03/08/2015

10:57 AM Task #2624 (Closed): Deprecate ndnx packet format support, remove in a later release

Deprecate ndnx packet format support in next platform release, remove code in subsequent platform release. Jeff Burke

03/06/2015

02:44 PM Bug #2575: Unspecified fields should not be present in packets

This issue is NOT CCL and has limited relation to "deprecating" of other packet support: why if I didn't specify Must... Alex Afanasyev

02:41 PM Bug #2572: NDN packet does not define Interest.Selectors.AnswerOriginKind and it should not be used in the code

I created this in jndn. In my opinion, jndn should not have had support for ndnx as it was deprecated at the time of... Alex Afanasyev

02:35 PM Feature #2567: InterestFilter abstraction

I personally prefer this to be implemented in jNDN, as it is what has high priority for android applications. Alex Afanasyev

02:33 PM Feature #2571: Reconsider the need for ConnectionInfo abstractions

This should not be part of CCL at all, neither CCL should say anything about transports (may be just general about tr... Alex Afanasyev

02:28 PM Feature #2569: Reconsider necessity for "Node" class

I created this issue for jndn, as it applies specifically there. It is not related to NDN-CCL.
Without code refac... Alex Afanasyev

02:23 PM Feature #2566: Implement Face.setInterestFilter

One form (2) is a convenience method to do prefix registration and setting up dispatch.
Form (1) is just to set up... Alex Afanasyev

12:40 PM Feature #2531 (Closed): LocalControlHeader

I added integration test TestLocalControlHeader. Merged to master. Anonymous

11:06 AM Feature #2531: LocalControlHeader

I take that back. The documentation says that the enable local control header command only affects the face which rec... Anonymous

11:03 AM Feature #2531: LocalControlHeader

The integration test would have to modify NFD to send LocalControlHeader packets. So I'm going to delay writing the i... Anonymous

03/05/2015

05:43 PM Feature #2532 (Closed): Remote Prefix Registration

Branch 'remote-prefix-registration' merged. Anonymous

03:33 AM Feature #2532: Remote Prefix Registration

As suggested, I moved TestRemotePrefixRegistration from integration-tests to examples. Anonymous

05:40 PM Feature #2531: LocalControlHeader

I want to write an integration test and merge tomorrow.
Since you said you don't need to set the outgoing Interest... Anonymous

04:06 PM Feature #2531: LocalControlHeader

Makes sense. When are you thinking of merging and releasing? Andrew Brown

03:51 PM Feature #2531: LocalControlHeader

> why don't we just expose one getter/setter, e.g. getLocalControlHeader() ... it would be consistent with how getMet... Anonymous

03:23 PM Feature #2531: LocalControlHeader

Looks good to me. (One minor question re: Data/Interest API: why don't we just expose one getter/setter, e.g. getLoca... Andrew Brown

11:40 AM Feature #2531: LocalControlHeader

Hi Andrew. Please check out the branch issue/2531-LocalControlHeader: https://github.com/named-data/jndn/commits/issu... Anonymous

03:47 PM Task #2543: Port SegmentFetcher

I have no issues with porting. I cannot fully agree that this is related to NDN-CCL. This is just an utility helper... Alex Afanasyev

03:37 PM Task #2543 (In Progress): Port SegmentFetcher

Ported SegmentFetcher to jNDN and made TestListRib, etc. use it. Anonymous

11:20 AM Task #2620 (Closed): Interest setter methods should return this

In the Data class, the setter methods like setName() return this so that you can chain calls to update values. http:/... Anonymous

03/04/2015

11:05 PM Feature #2570: Refactoring of Exclude public interface

+1 to consider this. Moving to NDN-CCL unless there is a need to consider it in jNDN first. Jeff Burke

11:04 PM Feature #2566: Implement Face.setInterestFilter

Could you explain why?
Moving to NDN-CCL unless there is a reason to only look at this in jNDN. Jeff Burke

11:03 PM Feature #2567: InterestFilter abstraction

Moving to NDN-CCL unless there's a reason to consider in jNDN first? Jeff Burke

11:02 PM Feature #2569: Reconsider necessity for "Node" class

I can see the point on this and will have to revisit with JeffT what the original motivation was for the Node. (We h... Jeff Burke

10:59 PM Feature #2568: Face.put(Data)

+1, we will work on this. Jeff Burke

10:59 PM Bug #2572: NDN packet does not define Interest.Selectors.AnswerOriginKind and it should not be used in the code

NDN-CCL issue. Essentially duplicates #2575. Related to deprecating support for NDNx. See responses to other related... Jeff Burke

10:57 PM Feature #2573 (Rejected): Remove support for encoding other than NDN-TLV

Please respond to my email about a process for this, send 10/10/14 and resent 10/26/14, 1/12/15, 3/4/15. Jeff Burke

10:52 PM Feature #2574: Explicit usage of time abstractions in public API

This is not a jNDN specific issue. Moved to NDN-CCL. Jeff Burke

10:47 PM Bug #2575: Unspecified fields should not be present in packets

This requires deprecating AnswerOriginKind support in NDN-CCL (or at least jNDN) and phasing out the NDNx packet form... Jeff Burke

10:39 PM Feature #2576: ForwardingFlags should be removed

+1 but does it need to be considered across all CCL? Jeff Burke

10:38 PM Feature #2577: Redesign Signature* classes

The basis for design for jndn is NDN-CCL, not ndn-cxx. Should we move this to a NDN-CCL issue and discuss more broad... Jeff Burke

10:31 PM Feature #2532: Remote Prefix Registration

To resolve AlexA's points earlier -
Agreed that the logic and state for isLocal() goes in Transport().
Each ... Jeff Burke

02:26 PM Feature #2532: Remote Prefix Registration

Good point, it's more an example than a test. Because it has some setup involved (potentially modifying the NFD confi... Andrew Brown

11:14 AM Feature #2532: Remote Prefix Registration

Ah, now I see the documentation in the Java file. It is clear enough. I was able to run it. It registers a prefix and... Anonymous

10:08 AM Feature #2532: Remote Prefix Registration

Jeff, the syntax for running the integration test should be `mvn test -DclassName=TestRemotePrefixRegistration -Dip=[... Andrew Brown

08:56 AM Feature #2532: Remote Prefix Registration

Hi Andrew. Your existing pull request https://github.com/named-data/jndn/pull/6 adds a Face method isLocal() which si... Anonymous

10:31 PM Feature #2571: Reconsider the need for ConnectionInfo abstractions

Can you expand on this? Jeff Burke

03/03/2015

04:04 PM Feature #2531 (In Progress): LocalControlHeader

Anonymous

10:15 AM Feature #2531: LocalControlHeader

Sounds good. Since this is a low-use feature, there won't be a problem to change to a new approach at some point when... Anonymous