[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-13 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 9: R9 includes another case that isn't hand

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-13 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#9). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (3 comments) I've pushed a new test whi

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-12 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#8). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: > Patch Set 7: > > > I can see how a dou

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: > I can see how a double apply could lea

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-12 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) > Patch Set 7: > > (1 commen

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:8

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (1 comment) http://gerrit.cloudera.org:

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (13 comments) http://gerrit.cloudera.org

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (3 comments) http://gerrit.cloudera.org

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 7: (23 comments) I still need to address A

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-11 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#7). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 6: (25 comments) http://gerrit.cloudera.org

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#6). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#5). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8313 to look at the new patch set (#4). Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener ..

[kudu-CR] KUDU-2191 (10/n): Hive Metastore notification log event listener

2018-04-10 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8313 ) Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener .. Patch Set 4: (24 comments) http://gerrit.cloudera.or