[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-19 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11923 to look at the new patch set (#6). Change subject: [location_awareness] Register client with location when connect to cluster

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-14 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11923 ) Change subject: [location_awareness] Register client with location when connect to cluster .. Patch Set 5: (1 comment)

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11923 ) Change subject: [location_awareness] Register client with location when connect to cluster .. Patch Set 5: (1 comment)

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11923 ) Change subject: [location_awareness] Register client with location when connect to cluster .. Patch Set 5: (1 comment)

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11923 to look at the new patch set (#5). Change subject: [location_awareness] Register client with location when connect to cluster

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Fengling Wang has restored this change. ( http://gerrit.cloudera.org:8080/11923 ) Change subject: [location_awareness] Register client with location when connect to cluster .. Restored -- To view, visit

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Fengling Wang has abandoned this change. ( http://gerrit.cloudera.org:8080/11923 ) Change subject: [location_awareness] Register client with location when connect to cluster .. Abandoned I'll start a new one. -- To view,

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11923 to look at the new patch set (#4). Change subject: [location_awareness] Register client with location when connect to cluster

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-13 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11923 to look at the new patch set (#3). Change subject: [location_awareness] Register client with location when connect to cluster

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-12 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11923 to look at the new patch set (#2). Change subject: [location_awareness] Register client with location when connect to cluster

[kudu-CR] [location awareness] Register client with location when connect to cluster

2018-11-12 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11923 Change subject: [location_awareness] Register client with location when connect to cluster .. [location_awareness] Register client

[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 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-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] [client] expose location via internal API

2018-10-25 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 12: Alexey, thanks for rebasing! -- To view, visit

[kudu-CR] [client] expose location via internal API

2018-10-25 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#10). Change subject: [client] expose location via internal API

[kudu-CR] [client] expose location via internal API

2018-10-25 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#9). Change subject: [client] expose location via internal API

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 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/11727 to look at the new patch set (#7). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 7: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#8). Change subject: [client] expose location via internal API

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [client] expose location via internal API .. Patch Set 2: (1 comment)

[kudu-CR] [client] expose location via internal API

2018-10-24 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#7). Change subject: [client] expose location via internal API

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 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/11727 to look at the new patch set (#6). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-24 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 6: (6 comments)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-23 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/11727 to look at the new patch set (#5). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-23 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/11727 to look at the new patch set (#4). Change subject: [location_awareness] Add ts location to TSInfoPB

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-23 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (1 comment)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

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

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-23 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (7 comments)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-22 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (1 comment) I'm thinking maybe the test is better

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-22 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: (2 comments)

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-22 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 2: Added a test to verify that the TS location is

[kudu-CR] [location awareness] Add ts location to TSInfoPB

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

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-22 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#6). Change subject: [location_awareness] Add ts location to both client and internal client

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-22 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#5). Change subject: [location_awareness] Add ts location to both client and internal client

[kudu-CR] [location awareness] Add ts location to TSInfoPB

2018-10-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11727 ) Change subject: [location_awareness] Add ts location to TSInfoPB .. Patch Set 1: Hi Adar, thanks for the suggestion. But it seems

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (1 comment)

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 4: I made a separate

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: [location_awareness] Add ts location to both client and internal client .. Patch Set 2: (4 comments)

[kudu-CR] [location awareness] Add ts location to both client and internal client

2018-10-18 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Alexey Serbin, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11679 to look at the new patch set (#4). Change subject: [location_awareness] Add ts location to both client and internal client

[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] Add ts location to both client and internal client

2018-10-15 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: Add ts location to both client and internal client .. Patch Set 2: (1 comment)

[kudu-CR] Add ts location to both client and internal client

2018-10-15 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/11679 to look at the new patch set (#2). Change subject: Add ts location to both client and internal client

[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 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-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] Add ts location to both client and internal client

2018-10-13 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/11679 ) Change subject: Add ts location to both client and internal client .. Patch Set 1: I'm kind of stuck on how to test this. I was

[kudu-CR] Add ts location to both client and internal client

2018-10-13 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11679 Change subject: Add ts location to both client and internal client .. Add ts location to both client and internal client I added the

[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-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 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 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 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-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] Add a tool to dump master data in a readable format

2018-09-14 Thread Fengling Wang (Code Review)
Fengling Wang has abandoned this change. ( http://gerrit.cloudera.org:8080/10697 ) Change subject: Add a tool to dump master data in a readable format .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/10697 To

[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

[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 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 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 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-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 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-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

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-20 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10734 ) Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. Patch Set 5: yes, makes sense. -- To view, visit

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10734 ) Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Hello Tidy Bot, Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10734 to look at the new patch set (#4). Change subject: KUDU-2314 Fix printing empty wal & metadata dirs

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10734 ) Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. Patch Set 2: Done. Thanks! -- To view, visit

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10734 to look at the new patch set (#3). Change subject: KUDU-2314 Fix printing empty wal & metadata dirs ..

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10734 ) Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. Patch Set 1: (2 comments)

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-18 Thread Fengling Wang (Code Review)
Hello Kudu Jenkins, Andrew Wong, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10734 to look at the new patch set (#2). Change subject: KUDU-2314 Fix printing empty wal & metadata dirs ..

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-15 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10734 ) Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. Patch Set 1: (1 comment)

[kudu-CR] KUDU-2314 Fix printing empty wal & metadata dirs

2018-06-15 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10734 Change subject: KUDU-2314 Fix printing empty wal & metadata dirs .. KUDU-2314 Fix printing empty wal & metadata dirs Change-Id:

[kudu-CR] Fix ksck checksum scan printing

2018-06-12 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10535 to look at the new patch set (#7). Change subject: Fix ksck checksum_scan printing .. Fix ksck

[kudu-CR] Fix ksck checksum scan printing

2018-06-12 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10535 ) Change subject: Fix ksck checksum_scan printing .. Patch Set 7: (5 comments) http://gerrit.cloudera.org:8080/#/c/10535/6//COMMIT_MSG

[kudu-CR] Add a tool to dump master data in a readable format

2018-06-12 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10697 ) Change subject: Add a tool to dump master data in a readable format .. Patch Set 1: (1 comment)

[kudu-CR] Add a tool to dump master data in a readable format

2018-06-12 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10697 ) Change subject: Add a tool to dump master data in a readable format .. Patch Set 1: (3 comments) To dump the master data (which is

[kudu-CR] Add a tool to dump master data in a readable format

2018-06-12 Thread Fengling Wang (Code Review)
Fengling Wang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10697 Change subject: Add a tool to dump master data in a readable format .. Add a tool to dump master data in a readable format e.g. $ kudu

[kudu-CR] Fix ksck checksum scan printing

2018-06-06 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10535 ) Change subject: Fix ksck checksum_scan printing .. Patch Set 5: (1 comment)

[kudu-CR] Fix ksck checksum scan printing

2018-05-30 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10535 ) Change subject: Fix ksck checksum_scan printing .. Patch Set 4: (2 comments)

[kudu-CR] Fix ksck checksum scan printing

2018-05-30 Thread Fengling Wang (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10535 to look at the new patch set (#5). Change subject: Fix ksck checksum_scan printing .. Fix ksck

[kudu-CR] Fix ksck checksum scan printing

2018-05-29 Thread Fengling Wang (Code Review)
Fengling Wang has posted comments on this change. ( http://gerrit.cloudera.org:8080/10535 ) Change subject: Fix ksck checksum_scan printing .. Patch Set 4: (2 comments)

  1   2   3   >