jihoonson commented on pull request #10144:
URL: https://github.com/apache/druid/pull/10144#issuecomment-655738971
```
33 F | L private static final Logger LOG = new
Logger(ReportTimelineMissingSegmentQueryRunner.class);
34 F
45 F | L LOG.debug("Reporting a missing segment[%s] for
query[%s]", descriptor, queryPlus.getQuery().getId());
------------------------------------------------------------------------------
Diff coverage statistics:
------------------------------------------------------------------------------
| lines | branches | functions | path
------------------------------------------------------------------------------
| 0% (0/2) | 100% (0/0) | 100% (3/3) |
org/apache/druid/query/ReportTimelineMissingSegmentQueryRunner.java
------------------------------------------------------------------------------
Total diff coverage:
- lines: 0% (0/2)
- branches: 100% (0/0)
- functions: 100% (3/3)
ERROR: Insufficient line coverage of 0% (0/2). Required 50%.
```
@gianm @clintropolis thank you for the review. I fixed the spell check
failure, but the test coverage check is just not happy about the logging I
added. I'm going to merge this PR.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
