[Impala-ASF-CR] IMPALA-4053: Address follow up comments for IMPALA-3610

2017-09-14 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 .. Patch Set 3: There was quite some disagreement with the approach. The point of this patch was to

[Impala-ASF-CR] IMPALA-4053: Address follow up comments for IMPALA-3610

2017-09-14 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has abandoned this change. Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4306 To unsubscribe, visit

[Impala-ASF-CR] IMPALA-4053: Address follow up comments for IMPALA-3610

2017-09-14 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 .. Patch Set 3: Is this still relevant? Sailesh, what should we do with this change? -- To view, visit

[Impala-ASF-CR] IMPALA-4053: Address follow up comments for IMPALA-3610

2016-09-19 Thread Marcel Kornacker (Code Review)
Marcel Kornacker has posted comments on this change. Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 .. Patch Set 2: (1 comment) let's put this on hold until mostafa verifies that the general approach

[Impala-ASF-CR] IMPALA-4053: Address follow up comments for IMPALA-3610

2016-09-15 Thread Sailesh Mukil (Code Review)
Sailesh Mukil has posted comments on this change. Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 .. Patch Set 3: (15 comments) http://gerrit.cloudera.org:8080/#/c/4306/2/be/src/runtime/coordinator.cc