[kudu-CR] fs: generate report during Open

2017-04-27 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: fs: generate report during Open .. fs: generate report during Open This patch modifies the FsManager and block managers to produce a filesystem report

[kudu-CR] fs: generate report during Open

2017-04-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6581 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-27 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6581 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-24 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 9: There were two real changes from v8: - Rename of the "full space preallocation check" (to generalize for hole

[kudu-CR] fs: generate report during Open

2017-04-24 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#9). Change subject: fs: generate report during Open

[kudu-CR] fs: generate report during Open

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6581 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 8: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6581 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 7: Code-Review+2 (2 comments)

[kudu-CR] fs: generate report during Open

2017-04-13 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.cc File

[kudu-CR] fs: generate report during Open

2017-04-13 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#7). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.cc File

[kudu-CR] fs: generate report during Open

2017-04-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/6581/4//COMMIT_MSG Commit Message: PS4, Line 9: This patch

[kudu-CR] fs: generate report during Open

2017-04-12 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6581/5/src/kudu/fs/fs_report.h File src/kudu/fs/fs_report.h:

[kudu-CR] fs: generate report during Open

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/6581/4//COMMIT_MSG Commit Message: PS4, Line 9: This

[kudu-CR] fs: generate report during Open

2017-04-11 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#5). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] fs: generate report during Open

2017-04-11 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/fs_report.cc File src/kudu/fs/fs_report.cc:

[kudu-CR] fs: generate report during Open

2017-04-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/fs_report.cc File src/kudu/fs/fs_report.cc:

[kudu-CR] fs: generate report during Open

2017-04-10 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#4). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] fs: generate report during Open

2017-04-10 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 3: (5 comments) http://gerrit.cloudera.org:8080/#/c/6581/3/src/kudu/fs/fs_report.cc File src/kudu/fs/fs_report.cc:

[kudu-CR] fs: generate report during Open

2017-04-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#3). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] fs: generate report during Open

2017-04-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6581 to look at the new patch set (#2). Change subject: fs: generate report during Open .. fs: generate report during

[kudu-CR] fs: generate report during Open

2017-04-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 1: (9 comments) http://gerrit.cloudera.org:8080/#/c/6581/1//COMMIT_MSG Commit Message: Line 7: fs: generate report

[kudu-CR] fs: generate report during Open

2017-04-07 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 1: (8 comments) only skimmed through so far since I see David took a look

[kudu-CR] fs: generate report during Open

2017-04-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 1: (22 comments) http://gerrit.cloudera.org:8080/#/c/6581/1/src/kudu/fs/block_manager.h File

[kudu-CR] fs: generate report during Open

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 1: (11 comments) first round of reviews. this patch is pretty heftly so I'll review it in 2-3 rounds or

[kudu-CR] fs: generate report during Open

2017-04-06 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6581 to review the following change. Change subject: fs: generate report during Open .. fs: generate