zhztheplayer opened a new issue, #9785:
URL: https://github.com/apache/incubator-gluten/issues/9785

   ### Description
   
   Enhance this API 
https://github.com/apache/incubator-gluten/blob/2feb5f936ccb81b11b053f27cbfbf653e1a3c742/gluten-arrow/src/main/java/org/apache/gluten/vectorized/NativePlanEvaluator.java#L46-L47
 by adding a parameter that allows additional configuration options being 
passed to the native Velox runtime.
   
   By doing this we could avoid adding JNI parameters every time we need some 
customization 
([example](https://github.com/apache/incubator-gluten/pull/9634/files#diff-4f3b38d38be472d3634702bf87c6ac1a990997eafe4207f67ce48f7b5f550bb1R401))
 for native execution.
   
   For example, JNI signature
   
   ```cpp
   
Java_org_apache_gluten_vectorized_PlanEvaluatorJniWrapper_nativeCreateKernelWithIterator(
 // NOLINT
       JNIEnv* env,
       jobject wrapper,
       jbyteArray planArr,
       jobjectArray splitInfosArr,
       jobjectArray iterArr,
       jint stageId,
       jint partitionId,
       jlong taskId,
       jboolean enableDumping,
       jstring spillDir,
       jboolean enableCudf)
   ```
   
   may be simplified to
   
   ```cpp
   
Java_org_apache_gluten_vectorized_PlanEvaluatorJniWrapper_nativeCreateKernelWithIterator(
 // NOLINT
       JNIEnv* env,
       jobject wrapper,
       jbyteArray planArr,
       jobjectArray splitInfosArr,
       jobjectArray iterArr)
   ```
   
   since then.
   
   ### Gluten version
   
   None


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to