Hello Tianyi Wang, Todd Lipcon, Impala Public Jenkins, Vuk Ercegovac,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11056
to look at the new patch set (#2).
Change subject: IMPALA-7225: REFRESH..PARTITION shoud not reset partition's num
rows
......................................................................
IMPALA-7225: REFRESH..PARTITION shoud not reset partition's num rows
The bug was that HdfsTable#reloadPartition() was not setting the
partition's numRows_ after reloading the partition.
Fix: Move the code to set numRows_ to createPartition() call so that
the callers need not explicitly set it after calling createPartition().
Additionally consolidates other common code in createPartition().
Added a test.
Change-Id: I5886684e26b453c76f331f6e807f813146c6bf3a
---
M fe/src/main/java/org/apache/impala/catalog/HdfsTable.java
M tests/metadata/test_refresh_partition.py
2 files changed, 42 insertions(+), 25 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/56/11056/2
--
To view, visit http://gerrit.cloudera.org:8080/11056
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5886684e26b453c76f331f6e807f813146c6bf3a
Gerrit-Change-Number: 11056
Gerrit-PatchSet: 2
Gerrit-Owner: Bharath Vissapragada <bhara...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tianyi Wang <tw...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
