[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3048 ---

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-06 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245534079 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- @@ -179,7 +181,8 @@ public CarbonWriterBuilder

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-06 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245532720 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- @@ -179,7 +181,8 @@ public CarbonWriterBuilder

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-04 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245336706 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- @@ -179,7 +182,8 @@ public CarbonWriterBuilder

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-04 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245328731 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- @@ -179,7 +182,8 @@ public CarbonWriterBuilder

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-04 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245252822 --- Diff: store/CSDK/test/main.cpp --- @@ -526,6 +526,71 @@ void testCarbonProperties(JNIEnv *env) { } } +bool

[GitHub] carbondata pull request #3048: [CARBONDATA-3224] Support SDK/CSDK validate t...

2019-01-04 Thread ajantha-bhat
Github user ajantha-bhat commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3048#discussion_r245246228 --- Diff: store/CSDK/test/main.cpp --- @@ -526,6 +526,71 @@ void testCarbonProperties(JNIEnv *env) { } } +bool