Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2816#discussion_r228206123
--- Diff: store/CSDK/CarbonReader.cpp ---
@@ -74,27 +75,41 @@ jobject CarbonReader::withHadoopConf(char *key, char
*value) {
return carbonReaderBuilderObject;
}
+jobject CarbonReader::withBatch(int batch) {
+ jclass carbonReaderBuilderClass =
jniEnv->GetObjectClass(carbonReaderBuilderObject);
+ jmethodID buildID = jniEnv->GetMethodID(carbonReaderBuilderClass,
"withBatch",
--- End diff --
Need to add validation. GetObjectClass, GetMethodID can return null.
Also need to take java exception to cpp for CallObjectMethodA.
---
