[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 7: Verified+1 -- To view, visit

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. [hms] Sync external HMS tables with purge property This

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [hms] Sync external HMS tables with purge property .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14179 To

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 7: Code-Review+2 -- To view, visit

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 6: (1 comment)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#7). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 6: (1 comment) LGTM, just one more test case to have

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#6). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 4: (1 comment)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-10 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#5). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 4: (1 comment)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [hms] Sync external HMS tables with purge property .. Removed -Verified by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14179 To

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 4: Verified+1 -- To view, visit

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Grant Henke has removed a vote on this change. Change subject: [hms] Sync external HMS tables with purge property .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/14179 To

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#4). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Greg Solovyev, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#3). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-09 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 2: (27 comments)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 2: (16 comments)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 2: (5 comments)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-05 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 2: (2 comments)

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-05 Thread Grant Henke (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/14179 to look at the new patch set (#2). Change subject: [hms] Sync external HMS tables with purge property

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-05 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/14179 ) Change subject: [hms] Sync external HMS tables with purge property .. Patch Set 1: (6 comments) Just passing through; Andrew/Hao should

[kudu-CR] [hms] Sync external HMS tables with purge property

2019-09-05 Thread Grant Henke (Code Review)
Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14179 Change subject: [hms] Sync external HMS tables with purge property .. [hms] Sync external HMS tables with purge property This patch is