search

UCLA-IRL / ndnts-aux

The SyncAgent and other useful functionalities used in NDN Workspace.
ISC License
2 stars 2 forks source link

NDNts-aux snapshot v1.1 #10

Closed CAWorks-ChrisA closed 1 month ago

CAWorks-ChrisA commented 1 month ago

Hotfixes on the following issues:

  1. Correcting a minor error on snapshot generation so it adheres to the configurable "snapshot topic" specified via the constructor in ndn-svs-adaptor
  2. Extend the interval's configuration capabilities from the ndn-svs-adaptor constructor to the workspace.create()'s opts.
  3. Default snapshot interval changed from 10 to 100.

PS: Sorry for the mess of the long chain of commits.. I think after this I should be able to remove commits within my fork.

zjkmxy commented 1 month ago

PS: Sorry for the mess of the long chain of commits..

Not a big issue for me, but I need to use squash merge in this case, which means only your final version will be accepted in the mainstream as a single commit. If you need your history (likely not), you need to keep a separate branch on your repo.