TODOs in asab.library

[ateska]

There are remainders from #239

• export library feature on the service level. Only a first library should be exported (or there should be a flag indicating whole/first library content to be exported).
  Add export feature to asab.library #260

• Periodical reload of the .disabled.yaml

  asab/asab/library/service.py

  Line 83 in 849fe99

  # TODO: Call this on tick (periodically)

  
  Enhancement: asab.library read disabled on tick #271

• Improved logging in -v mode that provides info about where each library provider is connected
  Improved logging #264

• Detect version changes in the ASAB library #284
  

  asab/asab/library/providers/zookeeper.py

  Line 160 in d705cd9

  L.info("Version changed", struct_data={'version': version, 'name': self.Library.Name})

• .read() method(s) has to be reworked to provide file-based output to avoid loading the whole item content in the memory. It is similar to the export feature.
  asab.library read now returns io instead ot bytes #256

• Support for zk://./path form of Zookeeper provider path - in this case path from zookeeper container and path should be concat.ed (NICE TO HAVE feature)
  Add handling zk://./path #261

[eliska-n]

Maybe my user experience will be useful:

asab/asab/library/providers/zookeeper.py

Line 155 in 8f79d06

L.warning("File `/.version.yaml` likely doesn't exists.")

Yes, theres's no version file in my library. Could asab.library make one for me? It would make me very happy. 🐷 And it even periodically tries to read the non-existing version file.

This is ok?

[ateska]

Maybe my user experience will be useful:

asab/asab/library/providers/zookeeper.py

Line 155 in 8f79d06

L.warning("File `/.version.yaml` likely doesn't exists.")

Yes, theres's no version file in my library. Could asab.library make one for me? It would make me very happy. 🐷 And it even periodically tries to read the non-existing version file.
This is ok?

This is intentionally a silent error - there is no degradation of the READ functionality, if the file is missing
ASAB Library will create .version.yaml file.
Reading non-existing version file periodically is acceptable.

[eliska-n]

I can add .version.yaml into my library or get a bunch of NoNode errors every minute. It is not very silent. Quite hard to accept.

[ateska]

I can add .version.yaml into my library or get a bunch of NoNode errors every minute. It is not very silent. Quite hard to accept.

Then the error must be silent.