Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23819
Change subject: [codegen] encode projection key once ...................................................................... [codegen] encode projection key once There is no need to encode row projection key multiple times: one time when scheduling a compilation task, another time when running it, and one more time when inserting a new entry into the codegen cache. The latter two steps were done by the same thread in the codegen compilation pool, and there is just a single thread in the pool. Apparently, it's better to encode the key only once by the thread which schedules the compilation task upon requesting the corresponding row projection. With this update, there isn't a need for JITWrapper::EncodeOwnKey() method anymore. Change-Id: Ief2a1cf184c86c33031f90d92a10755da3e86e0b --- M src/kudu/codegen/code_cache.cc M src/kudu/codegen/code_cache.h M src/kudu/codegen/compilation_manager.cc M src/kudu/codegen/jit_wrapper.h M src/kudu/codegen/row_projector.h 5 files changed, 42 insertions(+), 54 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/23819/1 -- To view, visit http://gerrit.cloudera.org:8080/23819 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ief2a1cf184c86c33031f90d92a10755da3e86e0b Gerrit-Change-Number: 23819 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
