Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18055 )

Change subject: [dense_node-itest] enable on macOS
......................................................................

[dense_node-itest] enable on macOS

This patch addresses a TODO to enable this test on macOS once the file
cache is used for WAL segments. This occurred in commit
fc4ab691a502067bc4d5bdff30507cac7feb7cfe.

Since macOS doesn't currently support the LogBlockManager, this also
updates the test to look for some BlockManager-agnostic metrics if not
using the LogBlockManager.

I also found we can add more block-related stress to the test scenario
by disabling compactions, which I've done optionally (though leaving the
default behavior the same).

Change-Id: I240331c4c9293d4f449d52b6a4ed2392a6ec9f3c
Reviewed-on: http://gerrit.cloudera.org:8080/18055
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/integration-tests/CMakeLists.txt
M src/kudu/integration-tests/dense_node-itest.cc
2 files changed, 30 insertions(+), 11 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/18055
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I240331c4c9293d4f449d52b6a4ed2392a6ec9f3c
Gerrit-Change-Number: 18055
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to