If each method needs to call statsd, what are the returns for?
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Mar 3 2020
In D2757#65748, @olasd wrote:That's not an either/or affair IMO. By definition as long as the replayer is running, the exclusion file is always out of date (and updating it is quite costly).
What about updating the exclusion file instead?
We probably want to keep svn_author_to_swh_person, as it deals with the None case. As for convert_commit_author, I just kept it as is, but probably not.
In D2743#65674, @olasd wrote:I'm surprised that this change doesn't incur a change of the id some of the tested revisions (e.g. in the npm case).
This is something that we should probably cover better.
Mar 2 2020
nice!
wait, so that contextmanager didn't work at all?
nice!
Feb 28 2020
add test
Replaced by D2690
caf51a044377cf62f73a02cd6c641d94b4e32c95
Feb 27 2020
remove prints
It's still missing a description of what it does (ie. check that files exist in swh), in the CLI help and/or README.
fix base
comments
Make build_revision return Revision objects instead of dicts.
add test + negative utc
apply comments
Also, compute_hash is a staticmethod that takes a dict as argument.
Oh, I forgot about compute_hash. And also that the id is "optional" (well not optional, but can be set to b'')
Feb 26 2020
Feb 25 2020
although you should change the commit message: "The full contents of the journal" -> "The existing contents of the journal"
Perfect, thanks!
Feb 24 2020
remove print
apply comment