Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1657: read-only FsManager::Open on active tablet can crash ......................................................................
KUDU-1657: read-only FsManager::Open on active tablet can crash This fixes a race condition when opening a read-only FsManager on a data directory with lbm containers that are actively being appended to. See the code comment for more details. A best-effort repro test case is included; it typically takes about 35 seconds to repro without the fix. This issue was identified because it was causing alter_table-randomized-test to be significantly flaky. Change-Id: If68b04f1f1b8cd099120a220b1245ecf8f422770 Reviewed-on: http://gerrit.cloudera.org:8080/4551 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Dan Burkert <d...@cloudera.com> --- M src/kudu/fs/log_block_manager.cc M src/kudu/integration-tests/CMakeLists.txt A src/kudu/integration-tests/open-readonly-fs-itest.cc 3 files changed, 158 insertions(+), 1 deletion(-) Approvals: Dan Burkert: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4551 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If68b04f1f1b8cd099120a220b1245ecf8f422770 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>