[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r236539095 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r236538593 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r236538137 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/2945 ---

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r236531750 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-26 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r236182445 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-23 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r235892400 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-23 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r235891858 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-23 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r235892016 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-23 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2945#discussion_r235890834 --- Diff: store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java --- @@ -66,9 +63,6 @@ this.readers = readers;

[GitHub] carbondata pull request #2945: [CARBONDATA-3123] Fixed JVM crash issue with ...

2018-11-22 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/carbondata/pull/2945 [CARBONDATA-3123] Fixed JVM crash issue with CarbonRecordReader(SDK Reader). **Problem:** As CarbonReaderBuilder is executed on the main thread therefore while Reader creation we are