Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23814 )
Change subject: IMPALA-14646: StorageDescriptor normalization should deal with parameters ...................................................................... IMPALA-14646: StorageDescriptor normalization should deal with parameters When checking whether an ALTER_TABLE event has trivial changes in StorageDescriptor, we normalize fields that are unrelated to file metadata, e.g. cols, bucketCols, sortCols, etc. However, the parameters map of StorageDescriptor is not normalized, which causes null and empty map be considered as different. This patch adds the normalization on the parameters map of StorageDescriptor. Also improves the logs when non-trival SD changes is detected. Currently we just dump the full SD objects, which is pretty verbose and hard to analyze. This patches add logs to show the actual changes. Tests - Added FE test Change-Id: I6a9fcf2d60a41e9669d49412d49a2416c13d17bc Reviewed-on: http://gerrit.cloudera.org:8080/23814 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/catalog/events/MetastoreEvents.java M fe/src/test/java/org/apache/impala/catalog/events/MetastoreEventsProcessorTest.java 2 files changed, 68 insertions(+), 8 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/23814 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6a9fcf2d60a41e9669d49412d49a2416c13d17bc Gerrit-Change-Number: 23814 Gerrit-PatchSet: 5 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]>
