Page MenuHomeSoftware Heritage

cli: fix max-message handling in the journal-client command

Authored by douardda on Nov 5 2019, 12:23 PM.



the max_messages value must be passed to the client constructor, otherwise
it will hang forever for max-messages<20 (eg. in tests).

Diff Detail

rDCIDX Metadata indexer
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Nov 5 2019, 12:23 PM
vlorentz requested changes to this revision.Nov 5 2019, 12:25 PM
vlorentz added a subscriber: vlorentz.

No, the right fix is D2205

This revision now requires changes to proceed.Nov 5 2019, 12:25 PM
vlorentz accepted this revision.Nov 5 2019, 2:56 PM
This revision is now accepted and ready to land.Nov 5 2019, 2:56 PM
douardda updated this revision to Diff 7650.Nov 5 2019, 3:07 PM

reformat for style consistency in the function