[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. IMPALA-1048: show sinks in exec summary The exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 15: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 14: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 14: Code-Review+2 (1 comment) Carry

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#14). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 15: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 13: Code-Review+2 (1 comment) Had a nit, otherwise LGTM

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 13: Anyone want to +2? -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-04 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 13: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-04 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 13: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-04 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#13). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: (3 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 12: Build started:

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#12). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: (3 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: Build started:

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 11: It looks like the sink summary has a time of zero sometimes

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#11). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 10: Verified-1 Build failed:

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 10: Build started:

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 9: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#9). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 10: Code-Review+1 carry -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 8: (14 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-12-03 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 9: (3 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-29 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 8: (14 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-29 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 8: Code-Review+1 I suppose it's safe to assume that no external

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 8: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 7: (7 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#8). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Michael Ho (Code Review)
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 7: (7 comments)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 7: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-28 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, Impala Public Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#7). Change subject: IMPALA-1048: show sinks in exec summary

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 6: Build Successful

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-27 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 6: (1 comment)

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11967 ) Change subject: IMPALA-1048: show sinks in exec summary .. Patch Set 6: Ok I think this is ready for review. -- To view, visit

[Impala-ASF-CR] IMPALA-1048: show sinks in exec summary

2018-11-27 Thread Tim Armstrong (Code Review)
Hello Michael Ho, Lars Volker, Philip Zeyliger, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11967 to look at the new patch set (#6). Change subject: IMPALA-1048: show sinks in exec summary