[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Zach Amsden (Code Review)
Zach Amsden has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Code-Review-1 I think we have more important things to do and there are probably even more egregious tests.

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-06-27 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Do we plan to move forward with this? -- To view, visit http://gerrit.cloudera.org:8080/6477 To unsubscribe,

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-03-30 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. Change subject: IMPALA-5128: BE Test Infrastructure .. Patch Set 2: Code-Review+1 (3 comments) http://gerrit.cloudera.org:8080/#/c/6477/2/be/src/exprs/expr-test.cc File

[Impala-ASF-CR] IMPALA-5128: BE Test Infrastructure

2017-03-28 Thread Zach Amsden (Code Review)
Zach Amsden has uploaded a new patch set (#2). Change subject: IMPALA-5128: BE Test Infrastructure .. IMPALA-5128: BE Test Infrastructure Investigating a crash in backend exprs test and this test takes a long time to run. Lets