[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/4/java/kudu-client/

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/4/java/kudu-

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/4/java/kudu-client/

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/4/java/kudu-

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/4/java/kudu-client/

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java client] Refactor all server info into a single class, add locality .. [java client] Refactor all server info into a single class, add localit

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/4836/3/java/kudu

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-27 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4836 to look at the new patch set (#4). Change subject: [java client] Refactor all server info into a single class, add locality ...

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-26 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/4836/3/java/kud

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new patch set (#3). Change subject: [java client] Refactor all server info into a single class, add locality .. [java client] Refactor all server info into a single class, add locality Having t

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/4836/2/java/kudu

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4836/2/java/kudu-client/s

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/4836/1/java/kudu-client

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Refactor all server info into a single class, add locality .. Patch Set 2: > Removed the following votes: > > * Verified-1 by Kudu Jenkins

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Jean-Daniel Cryans (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4836 to look at the new patch set (#2). Change subject: [java client] Refactor all server info into a single class, add locality ...

[kudu-CR] [java client] Refactor all server info into a single class, add locality

2016-10-25 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has uploaded a new change for review. http://gerrit.cloudera.org:8080/4836 Change subject: [java client] Refactor all server info into a single class, add locality .. [java client] Refactor all server info