Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2792#discussion_r228053433
--- Diff: store/CSDK/CarbonReader.cpp ---
@@ -19,15 +19,14 @@
#include <jni.h>
jobject CarbonReader::builder(JNIEnv *env, char *path, char *tableName) {
-
jniEnv = env;
jclass carbonReaderClass =
env->FindClass("org/apache/carbondata/sdk/file/CarbonReader");
jmethodID carbonReaderBuilderID =
env->GetStaticMethodID(carbonReaderClass, "builder",
--- End diff --
If class path is wrong or class is not found in given path. This will be
NPE and jvm will crash.
Need to validate this and throw exception or return null with log to the
user.
---
