[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-11-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. KUDU-2191 (2/n): Hive Metastore client This patch lays the groundwork

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-11-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 37: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-11-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (2 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-11-01 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#37). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 36: (2 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 36: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 36: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 36: OK I think this is ready to review again. Sorry for the churn.

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#36). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#35). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#34). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#33). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#32). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 31: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (14 comments) I've added TODOs to expand the error handling

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-30 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#31). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (15 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (2 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: Verified+1 -- To view, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Dan Burkert (Code Review)
Dan Burkert has removed a vote on this change. Change subject: KUDU-2191 (2/n): Hive Metastore client .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/7053 To unsubscribe, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 30: (4 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-23 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#30). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 24: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (4 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: Code-Review+1 IWYU is confused. -- To view, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Dan Burkert (Code Review)
Dan Burkert has removed a vote on this change. Change subject: KUDU-2191 (2/n): Hive Metastore client .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/7053 To unsubscribe, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (1 comment)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-19 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 29: (3 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#29). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 27: Dissapointingly, the 10s limit on starting up the mini hms was

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 27: (4 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#28). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 27: (6 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#27). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 24: (6 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 26: (5 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#26). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 22: (5 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Dan Burkert (Code Review)
Hello Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#25). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 24: (6 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Dan Burkert has removed a vote on this change. Change subject: KUDU-2191 (2/n): Hive Metastore client .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/7053 To unsubscribe, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 24: Code-Review+1 I can't make heads or tails of the remaining IWYU

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Dan Burkert has removed Tidy Bot from this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Removed reviewer Tidy Bot. -- To view, visit

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#24). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 23: (17 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#23). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 22: (17 comments) I focused mostly on the glue, since I think I (and

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#22). Change subject: KUDU-2191 (2/n): Hive Metastore client

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/7053 ) Change subject: KUDU-2191 (2/n): Hive Metastore client .. Patch Set 19: (2 comments)

[kudu-CR] KUDU-2191 (2/n): Hive Metastore client

2017-10-17 Thread Dan Burkert (Code Review)
Hello Tidy Bot, Alexey Serbin, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7053 to look at the new patch set (#21). Change subject: KUDU-2191 (2/n): Hive Metastore client