Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/2931
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2931#discussion_r242444376
--- Diff: store/CSDK/test/main.cpp ---
@@ -822,16 +831,17 @@ int main(int argc, char *argv[]) {
// init jvm
JNIEnv *env;
env
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2931#discussion_r242428274
--- Diff: store/CSDK/test/main.cpp ---
@@ -822,16 +831,17 @@ int main(int argc, char *argv[]) {
// init jvm
JNIEnv *env;
env
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2931#discussion_r242387341
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java
---
@@ -147,34 +170,55 @@ public static Schema
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2931#discussion_r242034641
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java
---
@@ -147,34 +170,55 @@ public static Schema
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/2931
[CARBONDATA-2999] support read schema from S3
[CARBONDATA-2999] support read schema from S3
SDK and CSDK both support it.
Be sure to do all of the following checklist to help us
