You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ja...@apache.org on 2023/07/31 16:26:13 UTC

[pinot] branch master updated: Fix null pointer exception in segment debug endpoint (#11228)

This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 030d9c08c8 Fix null pointer exception in segment debug endpoint (#11228)
030d9c08c8 is described below

commit 030d9c08c8360c49cc6094efc7150182552e1be3
Author: Caner Balci <ca...@gmail.com>
AuthorDate: Mon Jul 31 09:26:05 2023 -0700

    Fix null pointer exception in segment debug endpoint (#11228)
---
 .../java/org/apache/pinot/controller/api/resources/DebugResource.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/DebugResource.java b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/DebugResource.java
index 2058cfadab..c6eb2b9b21 100644
--- a/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/DebugResource.java
+++ b/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/DebugResource.java
@@ -349,7 +349,7 @@ public class DebugResource {
           Map<String, SegmentServerDebugInfo> segmentServerDebugInfoMap = segmentEntry.getValue();
           SegmentServerDebugInfo segmentServerDebugInfo = segmentServerDebugInfoMap.get(segmentName);
 
-          if (verbosity > 0 || (segmentServerDebugInfo != null) && segmentHasErrors(segmentServerDebugInfo, evState)) {
+          if (segmentServerDebugInfo != null && (verbosity > 0 || segmentHasErrors(segmentServerDebugInfo, evState))) {
             segmentServerState.put(instanceName,
                 new TableDebugInfo.SegmentState(isState, evState, segmentServerDebugInfo.getSegmentSize(),
                     segmentServerDebugInfo.getConsumerInfo(), segmentServerDebugInfo.getErrorInfo()));


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org