[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-09-05 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. [location_awareness] Add 'location' column in

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-09-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 6: NB we can't merge this until the

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-09-04 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 6: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-09-04 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 6: Code-Review+1 -- To view, visit

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-31 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 6: (4 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-31 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11313 to look at the new patch set (#6). Change subject: [location_awareness] Add 'location' column in tserver list

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 5: (4 comments) overall looks good, just a

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-30 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11313 to look at the new patch set (#5). Change subject: [location_awareness] Add 'location' column in tserver list

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 4: > Uploaded patch set 4. It seems lint is

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 4: (1 comment)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 4: (1 comment)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11313 to look at the new patch set (#4). Change subject: [location_awareness] Add 'location' column in tserver list

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 3: (2 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 2: (1 comment)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11313 to look at the new patch set (#3). Change subject: [location_awareness] Add 'location' column in tserver list

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-28 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 2: (1 comment)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-27 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 2: (13 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-27 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11313 to look at the new patch set (#2). Change subject: [location_awareness] Add 'location' column in tserver list

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 1: (2 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 1: (2 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 1: (7 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11313 ) Change subject: [location_awareness] Add 'location' column in tserver list .. Patch Set 1: (9 comments)

[kudu-CR] [location awareness] Add 'location' column in tserver list

2018-08-23 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11313 Change subject: [location_awareness] Add 'location' column in tserver list .. [location_awareness] Add 'location' column in tserver