zhztheplayer commented on code in PR #9446:
URL: https://github.com/apache/incubator-gluten/pull/9446#discussion_r2068196031


##########
gluten-flink/runtime/src/main/java/org/apache/gluten/table/runtime/operators/GlutenSingleInputOperator.java:
##########
@@ -75,7 +83,8 @@ public GlutenCalOperator(String plan, String id, String 
inputType, String output
     public void open() throws Exception {
         super.open();
         outElement = new StreamRecord(null);
-        session = 
Velox4j.newSession(MemoryManager.create(AllocationListener.NOOP));
+        memoryManager = MemoryManager.create(AllocationListener.NOOP);
+        session = Velox4j.newSession(memoryManager);
 
         inputQueue = new LinkedBlockingQueue<>();

Review Comment:
   Could move onto this API 
https://github.com/velox4j/velox4j/blob/40474f2da7e59dfb1b2bf2682310eec2d2b0ab85/src/main/java/io/github/zhztheplayer/velox4j/connector/ExternalStreams.java#L35-L37
 while the Velox4J version is updated in the future.



-- 
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: commits-unsubscr...@gluten.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@gluten.apache.org
For additional commands, e-mail: commits-h...@gluten.apache.org

Reply via email to