[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2018-04-04 Thread Andrew Wong (Code Review)
Andrew Wong has abandoned this change. ( http://gerrit.cloudera.org:8080/8048 ) Change subject: KUDU-2135 (part 1): add persistent disk states .. Abandoned no longer in scope for disk failure handling -- To view, visit http://g

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2018-04-04 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/8048 ) Change subject: KUDU-2135 (part 1): add persistent disk states .. Patch Set 7: (2 comments) Should this be abandoned? I think it was re-

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-2135 (part 1): add persistent disk states .. Patch Set 7: (14 comments) I think Todd should review this. http://gerrit.cloudera.org:8080/#/c/8048/7//COMMIT_M

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-2135 (part 1): add persistent disk states .. Patch Set 7: (1 comment) There is a tidywarning but this is reviewable. http://gerrit.cloudera.org:8080/#/c/804

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-15 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#7). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (par

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-13 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#5). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (par

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-13 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#4). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (par

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-13 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: KUDU-2135 (part 1): add persistent disk states .. Patch Set 2: Ah, Adar mentioned it'd be a good idea to decompose DataDirManager::Open() since it's so long. I'll

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-12 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#3). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (par

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-12 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8048 to look at the new patch set (#2). Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (par

[kudu-CR] KUDU-2135 (part 1): add persistent disk states

2017-09-12 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new change for review. http://gerrit.cloudera.org:8080/8048 Change subject: KUDU-2135 (part 1): add persistent disk states .. KUDU-2135 (part 1): add persistent disk states This patch introduces a n