impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3764: fuzz test HDFS scanners
Date Thu, 21 Jul 2016 23:01:16 GMT
Alex Behm has posted comments on this change.

Change subject: IMPALA-3764: fuzz test HDFS scanners
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3448/5/tests/query_test/test_scanners.py
File tests/query_test/test_scanners.py:

Line 649:     random_seed = os.environ.get("SCANNER_FUZZ_SEED") or time.time()
> Agree better to discuss f2f. There's a tradeoff between coverage and build 
MikeB, Tim and I discussed this offline and decided to enable these tests in core with a random
seed. The thinking does that having the coverage is very valuable, and it remains to be see
how many "unrelated" builds break due to finding new bugs. If the latter becomes a problem
we can revisit this choice.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I50cf43195a7c582caa02c85ae400ea2256fa3a3b
Gerrit-PatchSet: 5
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: David Knupp <dknupp@cloudera.com>
Gerrit-Reviewer: Taras Bobrovytsky <tbobrovytsky@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message