projjal commented on a change in pull request #11193:
URL: https://github.com/apache/arrow/pull/11193#discussion_r713970110
##########
File path: cpp/src/gandiva/engine.h
##########
@@ -54,6 +54,13 @@ class GANDIVA_EXPORT Engine {
functions_to_compile_.push_back(fname);
}
+ /// Set BaseObjectCache.
+ template <class KeyType>
+ Status SetLLVMObjectCache(GandivaObjectCache<KeyType>& object_cache) {
+ execution_engine_->setObjectCache(&object_cache);
+ return Status::OK();
Review comment:
Ok. looks like earlier versions didnot contain this method. Also looking
at setObjectCache impl it just sets the member variable so no need to check for
error
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
