maytasm commented on a change in pull request #11492:
URL: https://github.com/apache/druid/pull/11492#discussion_r676860053
##########
File path: services/src/main/java/org/apache/druid/cli/CliIndexer.java
##########
@@ -130,6 +130,7 @@ public void configure(Binder binder)
JsonConfigProvider.bind(binder, "druid", DruidNode.class,
Parent.class);
JsonConfigProvider.bind(binder, "druid.worker",
WorkerConfig.class);
+ CliPeon.configureIntermediaryData(binder);
Review comment:
There are many ITs that run indexers with the new config unset (which
basically fallback to using "local" storage for storing intermediary segments
via `LocalIntermediaryDataManager`). Specifically, `input source integration
test with Indexer` runs ingestion with Hashed partitioning and
maxNumConcurrentSubTasks=10, which would run the ingestion in two phases (first
phase which persist to local using `LocalIntermediaryDataManager` and second
phase which reads segments from first phase). Similarly, there are also some
other ITs in compaction/auto compaction that uses Hashed partitioning.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
