Anurag Mantripragada has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14240


Change subject: IMPALA-8761: Improve events processor configuration validation.
......................................................................

IMPALA-8761: Improve events processor configuration validation.

This patch aims to improve the validation of configuration keys from the HMS.

IMPALA-8559 introduced configuration validation for events processor 
configurations.
In the existing implementation, the events processor errors out as soon as it 
sees a
validation failure. If there are more than one configuration errors, the users 
may
have to restart HMS each time they fix an configuration error. This is bad user
experience. This change collects all the configuration issues and logs expected
values before erroring out. Users can now fix all issues in one go.

Testing:
Added testValidateConfigs() to assert if multiple incorrect values are detected 
at once.

Change-Id: I73480872ef93215d05c1fd922e64eb68a8a63a42
---
M 
fe/src/main/java/org/apache/impala/catalog/events/MetastoreEventsProcessor.java
M 
fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java
2 files changed, 43 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/40/14240/1
--
To view, visit http://gerrit.cloudera.org:8080/14240
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I73480872ef93215d05c1fd922e64eb68a8a63a42
Gerrit-Change-Number: 14240
Gerrit-PatchSet: 1
Gerrit-Owner: Anurag Mantripragada <anu...@cloudera.com>

Reply via email to