[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

[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

[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

[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

[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)

[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

[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)

[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)

[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)

[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)

[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)

[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)

[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

[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)

[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)