[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-861 Support changing default, storage attributes This patch adds support for adding, changin

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 17: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4310 To unsubscribe, visit

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/4310/17/src/kudu/cfile/rle_block.h File src

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-07 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 17: (5 comments) http://gerrit.cloudera.org:8080/#/c/4310/12/src/kudu/common/schema.cc File s

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-06 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#17). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-06 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#16). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-12-06 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#15). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 14: (4 comments) http://gerrit.cloudera.org:8080/#/c/4310/12/src/kudu/common/schema.cc File src

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4310/12/src/kudu/common/schema.h File src/k

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#14). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 12: (1 comment) http://gerrit.cloudera.org:8080/#/c/4310/12/java/kudu-client/src/test/java/or

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 13: (12 comments) http://gerrit.cloudera.org:8080/#/c/4310/12/java/kudu-client/src/main/java/

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-30 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#13). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-17 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 12: (13 comments) http://gerrit.cloudera.org:8080/#/c/4310/12/java/kudu-client/src/main/java/or

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 11: Verified+1 TSAN build hit a Jenkins bug post-C++ and Java builds. Others OK so overriding.

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-08 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#11). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-08 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#10). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-8

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-08 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#9). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-86

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-08 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 8: (27 comments) http://gerrit.cloudera.org:8080/#/c/4310/8//COMMIT_MSG Commit Message: Line

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-11-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 8: (19 comments) didn't make it 100% through the tests, but mostly looking good. http://gerrit

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-10-29 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#8). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-86

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-10-29 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#7). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-86

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-28 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/4310/6/java/kudu-client/src/main/java/org/a

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-24 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 6: (10 comments) I'm going to add coverage in alter_table-randomized-test but wanted to post

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 6: (10 comments) http://gerrit.cloudera.org:8080/#/c/4310/6/java/kudu-client/src/main/java/org/

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-861 Support changing default, storage attributes .. Patch Set 6: Still going through the code changes, but wanted to note that you should add the new alterat

[kudu-CR] KUDU-861 Support changing default, storage attributes

2016-09-18 Thread Will Berkeley (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4310 to look at the new patch set (#6). Change subject: KUDU-861 Support changing default, storage attributes .. KUDU-86