[kudu-CR] separate DataDirManager from BlockManagers

2017-08-22 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: separate DataDirManager from BlockManagers .. separate DataDirManager from BlockManagers Currently, the DataDirManager is owned by the BlockManagers.

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 13: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7602 To unsubscribe, visit

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-22 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 13: This patch hasn't changed much; mainly been rebasing it. It's been +2'd though without significant

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-22 Thread Andrew Wong (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#13). Change subject: separate DataDirManager from BlockManagers ..

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7602 To unsubscribe, visit

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Andrew Wong (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#10). Change subject: separate DataDirManager from BlockManagers ..

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 9: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/7602/9/src/kudu/fs/fs_manager.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 8: (5 comments) http://gerrit.cloudera.org:8080/#/c/7602/8/src/kudu/fs/block_manager-stress-test.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#9). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/7602/8/src/kudu/fs/block_manager-stress-test.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 7: (11 comments) http://gerrit.cloudera.org:8080/#/c/7602/7/src/kudu/fs/block_manager-test.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-16 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#8). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 7: (8 comments) http://gerrit.cloudera.org:8080/#/c/7602/7/src/kudu/fs/block_manager-test.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-15 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#7). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-15 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 6: (34 comments) http://gerrit.cloudera.org:8080/#/c/7602/6//COMMIT_MSG Commit Message: PS6, Line 23:

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 6: (34 comments) http://gerrit.cloudera.org:8080/#/c/7602/6//COMMIT_MSG Commit Message: PS6, Line 23:

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-11 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#6). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-08 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#4). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#3). Change subject: separate DataDirManager from BlockManagers .. separate

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-07 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: separate DataDirManager from BlockManagers .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/7602/1/src/kudu/fs/data_dirs.cc File

[kudu-CR] separate DataDirManager from BlockManagers

2017-08-07 Thread Andrew Wong (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7602 to look at the new patch set (#2). Change subject: separate DataDirManager from BlockManagers .. separate