NishthaShah commented on PR #20449: URL: https://github.com/apache/druid/pull/20449#issuecomment-5885104971
Coverage-jacoco is failing with `Insufficient line coverage of 0% (0/1)`. The single flagged line is `AbstractBatchIndexTask.java:765`, which is inside `catch (InterruptedException)`; the existing `waitForSegmentAvailability` tests don't exercise the interrupt path, so the pre-existing coverage on that line is also 0. This PR only edits the log-message string (`availablity` -> `availability`); there is no logic change, and the branch that produces the line was already uncovered before the PR. Would a committer be willing to bypass the diff-coverage check, or would you prefer I add a separate test that interrupts the wait? Happy to do whichever is easier to merge. -- 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]
