Quanlong Huang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23870 )
Change subject: IMPALA-14670: Fix NullPointerException in getEventProcessorSummary ...................................................................... IMPALA-14670: Fix NullPointerException in getEventProcessorSummary TEventBatchProgressInfo fields are populated from MetastoreEventsProcessor state that may be updated concurrently during event processing. To avoid inconsistent reads, the method is expected to snapshot all required fields into local variables before using them. However, currentFilteredEvent_ was accessed directly after a null check, allowing it to become null concurrently and causing a NullPointerException while populating the progress. This change snapshots currentFilteredEvent_ into a local variable and uses the local reference consistently, preventing the race condition. Testing: - Ran existing tests. Change-Id: I3c9101811cd706e0eba487595852db3435232cf5 Reviewed-on: http://gerrit.cloudera.org:8080/23870 Reviewed-by: Quanlong Huang <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java 1 file changed, 10 insertions(+), 7 deletions(-) Approvals: Quanlong Huang: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/23870 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3c9101811cd706e0eba487595852db3435232cf5 Gerrit-Change-Number: 23870 Gerrit-PatchSet: 2 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]>
