[kudu-CR] [location awareness] Add location info in ksck report

2018-10-30 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 15: Code-Review+2 -- To view, visit

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 15: Code-Review+1 -- To view, visit

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 12: (2 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#15). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 14: Code-Review+1 (2 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#14). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 13: (8 comments) > (3 comments) > > The

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#13). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 12: (3 comments) The location isn't getting piped

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 12: (8 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 11: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-26 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 11: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-23 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 11: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-17 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 11: Re-based on top of

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-17 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#11). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-17 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 10: > Fixed IWYU. Great, thanks for fixing IWYU

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 10: Fixed IWYU. -- To view, visit

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#10). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 9: > Build Failed > >

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 9: (5 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#9). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: > > This patch should be depended on

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: > This patch should be depended on

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-13 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: This patch should be depended on

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: (5 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-05 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-05 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 8: (11 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-05 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#8). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (5 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 7: (6 comments) Looks good, but needs support for

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#7). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 6: The master can't register the new tablet server

[kudu-CR] [location awareness] Add location info in ksck report

2018-10-01 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 6: Hi Will & Alexey, do you guys have ideas about

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-28 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#6). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 5: > Uploaded patch set 5. It seems

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#5). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#4). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 3: (4 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#3). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 2: (3 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-24 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 2: > Uploaded patch set 2. Please address the

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-20 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 1: (11 comments) Thanks!

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-20 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11422 to look at the new patch set (#2). Change subject: [location_awareness] Add location info in ksck report

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 1: (1 comment)

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-12 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 1: (6 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 1: (7 comments)

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-11 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11422 ) Change subject: [location_awareness] Add location info in ksck report .. Patch Set 1: The expected output is

[kudu-CR] [location awareness] Add location info in ksck report

2018-09-11 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11422 Change subject: [location_awareness] Add location info in ksck report .. [location_awareness] Add location info in ksck report This