[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 22: Code-Review+2 (1 comment) Looks good to me!

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. KUDU-2823 Place tablet replicas based on dimension When

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 22: Code-Review+1 -- To view, visit

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 22: (18 comments)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#22). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-15 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#21). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-14 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#20). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-14 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#19). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-12 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 18: (18 comments)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-12 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 18: Add some unit tests for a few scenarios. -- To view,

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-12 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#18). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-11 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 17: (1 comment)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 17: Code-Review+1 OK sure. I think Alexey should take

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-11 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 17: (1 comment)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-11 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#17). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 16: (1 comment)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 16: (2 comments)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#16). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 15: (2 comments)

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Yao Xu (Code Review)
Yao Xu has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 15: (12 comments) > (12 comments) > > Just passing

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-10 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#15). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-08 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13632 ) Change subject: KUDU-2823 Place tablet replicas based on dimension .. Patch Set 14: (12 comments) Just passing through; didn't look at

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-08 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#14). Change subject: KUDU-2823 Place tablet replicas based on dimension

[kudu-CR] KUDU-2823 Place tablet replicas based on dimension

2019-07-08 Thread Yao Xu (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/13632 to look at the new patch set (#13). Change subject: KUDU-2823 Place tablet replicas based on dimension