Jackie-Jiang commented on code in PR #8523:
URL: https://github.com/apache/pinot/pull/8523#discussion_r848961214
##########
pinot-core/src/main/java/org/apache/pinot/core/operator/BaseOperator.java:
##########
@@ -35,7 +35,7 @@ public final T nextBlock() {
if (Thread.interrupted()) {
throw new EarlyTerminationException();
}
- try (InvocationScope execution =
Tracing.getTracer().createScope(getClass())) {
+ try (InvocationScope ignored =
Tracing.getTracer().createScope(getClass())) {
Review Comment:
Changed it because the IDE is putting a warning on unused variable. Another
way to suppress the warning would be adding a comment as following if that is
preferred:
```
//noinspection unused
try (InvocationScope scope =
Tracing.getTracer().createScope(getClass())) {
return getNextBlock();
}
```
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
