[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 12: (20 comments) http://gerrit.cloudera.org:8080/#/c/7374/12//COMMIT_MSG C

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 12: (1 comment) 1 small comment, but otherwise LGTM. I'd like to give Ada

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 12: The failure spark test does not seem to be relate. -- To view, visit http

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#12). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#11). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/7374/9/src/kudu/fs/log_blo

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#10). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 9: (11 comments) http://gerrit.cloudera.org:8080/#/c/7374/8/src/kudu/fs/block_

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#9). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error ..