Vihang Karajgaonkar has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17210


Change subject: IMPALA-10598: Deflake test_cache_reload_validation
......................................................................

IMPALA-10598: Deflake test_cache_reload_validation

This patch deflakes the test test_cache_reload_validation in
test_hdfs_caching.py e2e test. The util method which the test relies on to
get the count of list of cache directives by parsing the output of command
"hdfs cacheadmin -listDirectives -stats" does not consider that the output
may contain trailing new lines or headers. Hence the test fails because the
expected number of cache directives does not match the number of lines
of the output.

The fix parses the line "Found <int> entries" in the output when available
and returns the count from that line. If the line is not found, it fallbacks
to the earlier implementation of using the number of lines.

Testing:
1. The test was failing for me when run individually. After the patch, I looped
the test 10 times without any errors.

Change-Id: I2d491e90af461d5db3575a5840958d17ca90901c
---
M tests/query_test/test_hdfs_caching.py
1 file changed, 18 insertions(+), 2 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2d491e90af461d5db3575a5840958d17ca90901c
Gerrit-Change-Number: 17210
Gerrit-PatchSet: 1
Gerrit-Owner: Vihang Karajgaonkar <vih...@cloudera.com>

Reply via email to