Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23814
to look at the new patch set (#2).
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
---
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, 70 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/14/23814/2
--
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: newpatchset
Gerrit-Change-Id: I6a9fcf2d60a41e9669d49412d49a2416c13d17bc
Gerrit-Change-Number: 23814
Gerrit-PatchSet: 2
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
