Todd Lipcon has submitted this change and it was merged.

Change subject: Add a test for the code cache
......................................................................


Add a test for the code cache

This isn't a regression test for any particular existing bug. However, there
were some bugs in some revisions of the NVM cache work that were not exposed by
existing tests, and this seems to catch them.

Change-Id: I0b8f450c5f81f2b21f90e5f1059ca59550ca7fb6
Reviewed-on: http://gerrit.cloudera.org:8080/2618
Tested-by: Kudu Jenkins
Reviewed-by: Dan Burkert <[email protected]>
---
M src/kudu/codegen/codegen-test.cc
M src/kudu/codegen/compilation_manager.cc
M src/kudu/codegen/compilation_manager.h
3 files changed, 53 insertions(+), 2 deletions(-)

Approvals:
  Dan Burkert: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0b8f450c5f81f2b21f90e5f1059ca59550ca7fb6
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to