[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-06 Thread Dan Hecht (Code Review)
Dan Hecht has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 14: Code-Review+1 This looks okay to me. I focus

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 14: Code-Review+1 -- To view, visit http://

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 11: (1 comment) Carry +1 http://gerrit.clo

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, Dimitris Tsirogiannis, Alex Behm, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9123 to look at the new patch set (#14). Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates ..

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 13: Code-Review+1 (2 comments) http://gerrit.

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, Dimitris Tsirogiannis, Alex Behm, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9123 to look at the new patch set (#13). Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates ..

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 11: (13 comments) Addressed most of the com

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-05 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, Dimitris Tsirogiannis, Alex Behm, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9123 to look at the new patch set (#12). Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates ..

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-03 Thread Tianyi Wang (Code Review)
Tianyi Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/9123 ) Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates .. Patch Set 11: (12 comments) http://gerrit.cloudera.org:

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-02-01 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, Dimitris Tsirogiannis, Alex Behm, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9123 to look at the new patch set (#11). Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates ..

[Impala-ASF-CR] IMPALA-6437: separate AC/scheduler from catalog topic updates

2018-01-30 Thread Tim Armstrong (Code Review)
Hello Tianyi Wang, Dimitris Tsirogiannis, Alex Behm, Bikramjeet Vig, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9123 to look at the new patch set (#10). Change subject: IMPALA-6437: separate AC/scheduler from catalog topic updates ..