Hello Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/22994

to look at the new patch set (#2).

Change subject: IMPALA-14130: Wait catalog.curr-version in 
start-impala-cluster.py
......................................................................

IMPALA-14130: Wait catalog.curr-version in start-impala-cluster.py

IMPALA-13850 changed the behavior of bin/start-impala-cluster.py to wait
for the number of tables to be at least one. This is needed to detect
that the catalog has seen at least one update. There is special logic in
dataload to start Impala without tables in that circumstance.

This broke the perf-AB-test job, which starts Impala before loading
data. There are other times when we want to start Impala without tables,
and it is inconvenient to need to specify --wait_num_tables each time.

After IMPALA-13850 (part 2), we can tweak the script until it sees
catalog.curr-version >= 100. This patch replace the waiting mechanism to
wait until Coordinator's catalog.curr-version >= 100 in legacy catalog
mode. Minicluster in local catalog mode does not need to wait for
catalog.curr-version.

Testing:
- Pass dataload with updated script.
- Manually run start-impala-cluster.py in both legacy and local catalog
  mode and confirm it works.
- Pass custom cluster test_concurrent_ddls.py and test_catalogd_ha.py

Change-Id: I4a3956417ec83de4fb3fc2ef1e72eb3641099f02
---
M bin/start-impala-cluster.py
M testdata/bin/create-load-data.sh
M tests/common/impala_cluster.py
3 files changed, 16 insertions(+), 18 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/22994/2
--
To view, visit http://gerrit.cloudera.org:8080/22994
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4a3956417ec83de4fb3fc2ef1e72eb3641099f02
Gerrit-Change-Number: 22994
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>

Reply via email to