impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sailesh Mukil (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3587: Get rid of not default fs skip marker
Date Thu, 16 Jun 2016 06:50:36 GMT
Sailesh Mukil has submitted this change and it was merged.

Change subject: IMPALA-3587: Get rid of not_default_fs skip marker
......................................................................


IMPALA-3587: Get rid of not_default_fs skip marker

The SkipIf.not_default_fs used in test_compute_stats(), was meant to
skip non-HDFS filesystems. However, since we've changed our test
infrastructure to run with S3 and Isilion as default Filesystems,
this test has been working just fine and this skip marker has had no
effect.

I've also confirmed that this works on localFS.

Change-Id: I7776ad60ce0298d08b20c2573894463bdfb9f892
Reviewed-on: http://gerrit.cloudera.org:8080/3382
Reviewed-by: Sailesh Mukil <sailesh@cloudera.com>
Tested-by: Sailesh Mukil <sailesh@cloudera.com>
---
M tests/common/skip.py
M tests/metadata/test_compute_stats.py
M tests/metadata/test_ddl.py
3 files changed, 0 insertions(+), 5 deletions(-)

Approvals:
  Sailesh Mukil: Looks good to me, approved; Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3382
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7776ad60ce0298d08b20c2573894463bdfb9f892
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sailesh@cloudera.com>

Mime
View raw message