[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: fs: allow format with user-specified uuid .. fs: allow format with user-specified uuid The CLI tool is going to use this for the "handling permanent failur

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 5: Verified+1 The test failure was unrelated, and I've got a patch ready to go for it. -- To view, visit ht

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3968 To unsubscribe, visit http://gerrit

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/4/src/kudu/util/oid_generator-test.cc File src/kudu/

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3968 to look at the new patch set (#5). Change subject: fs: allow format with user-specified uuid .. fs: al

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2930/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: (1 comment) LGTM except for nit http://gerrit.cloudera.org:8080/#/c/3968/4/src/kudu/util/oid_generator-

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3968 To unsubscribe, visit http://gerrit

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3968 to look at the new patch set (#4). Change subject: fs: allow format with user-specified uuid .. fs: allow format wi

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2927/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3968/2/src/kudu/fs/fs_manager.cc File src/kudu/fs/fs_mana

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: fs: allow format with user-specified uuid .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2926/ -- To view, visit http://gerrit.cloudera.org

[kudu-CR] fs: allow format with user-specified uuid

2016-08-15 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3968 to look at the new patch set (#3). Change subject: fs: allow format with user-specified uuid .. fs: allow format wi