Closed justinlittman closed 10 months ago
since parallel may be desired in the future, could this be either 1) deployed as a branch for testing or PR / commit message say "removing parallel for rolling indexer"
@ndushay I added it to the commit message.
Adding parallel is trivial; there is no reason to leave it in if we don't have immediate uses for it.
Why was this change made? 🤔
To slow down the rolling indexer is configurable ways.
How was this change tested? 🤨
âš¡ âš If this change has cross service impact, run integration tests that exercise indexing (e.g. searches in Argo for newly created/updated items, access_indexing_spec) and/or test in [stage|qa] environment, in addition to specs. âš¡
QA