tyrobro opened a new pull request, #20818:
URL: https://github.com/apache/druid/pull/20818

   Fixes #20816.
   
   ### Description
   
   `CompactionTestBase.createCluster()` used the default 10-second 
`LatchableEmitter` 
   timeout. On loaded CI runners, the `index_parallel` task spawned by 
   `CompactionSparseColumnTest` takes longer than 10s to emit `task/run/time`, 
causing 
   `waitForTaskToFinish` to time out with an `ISE`.
   
   #### Increased LatchableEmitter timeout in CompactionTestBase
   
   Added `.useDefaultTimeoutForLatchableEmitter(60)` to 
`CompactionTestBase.createCluster()`, 
   matching the pattern already used in `StreamIndexTestBase`.
   
   <hr>
   
   ##### Key changed/added classes in this PR
   * `CompactionTestBase`
   
   <hr>
   
   This PR has:
   
   - [x] been self-reviewed.
   - [ ] ~~using the concurrency checklist~~ (not relevant)
   - [ ] ~~added documentation~~ (no user-facing change)
   - [ ] ~~a release note entry~~ (fixing a flaky test, no user impact)
   - [ ] ~~added Javadocs~~ (not applicable)
   - [ ] ~~added or updated license information~~ (not applicable)
   - [ ] ~~added comments~~ (change is self-explanatory)
   - [x] added unit tests or modified existing tests to cover new code paths.
   - [ ] ~~added integration tests~~ (not applicable)
   - [ ] ~~been tested in a test Druid cluster~~ (test infrastructure fix only)


-- 
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]

Reply via email to