[GitHub] carbondata issue #1244: [Don't merge] Only for test apache jenkin CIs, to av...

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1244 SDV Build Success with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/141/ --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread sraghunandan
Github user sraghunandan commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wis

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 There's already an old build in progress. I will clean up before trigger a new one. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] carbondata pull request #1218: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-08 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1218#discussion_r131841330 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/ColumnBasedResultCollector.java --- @@ -0,0 +1,233 @@ +/* + *

[GitHub] carbondata pull request #1218: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-08 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1218#discussion_r131842839 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -510,9 +511,16 @@ * LOAD_STATUS SUCCES

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/142/ --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] carbondata pull request #1218: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-08 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1218#discussion_r131845008 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/ColumnBasedResultCollector.java --- @@ -0,0 +1,233 @@ +/* ---

[GitHub] carbondata issue #1218: [CARBONDATA-1347] Implemented Columnar Reading Of Da...

2017-08-08 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1218 @anubhav100 please put test cases for this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] carbondata pull request #1218: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-08 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1218#discussion_r131846232 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/ColumnBasedResultCollector.java --- @@ -0,0 +1,233 @@ +/* --- En

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wis

[GitHub] carbondata issue #1218: [CARBONDATA-1347] Implemented Columnar Reading Of Da...

2017-08-08 Thread anubhav100
Github user anubhav100 commented on the issue: https://github.com/apache/carbondata/pull/1218 @chenliang613 should i add unit test cases for all newly added classes --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] carbondata pull request #1218: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-08 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1218#discussion_r131846650 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -510,9 +511,16 @@ * LOAD_STATUS SUCCESS

[GitHub] carbondata issue #1218: [CARBONDATA-1347] Implemented Columnar Reading Of Da...

2017-08-08 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1218 @anubhav100 Yes. Suggest creating a new PR for test cases to cover all presto functionality. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/143/ --- If your project is set up for it, you can reply to this email and have your reply app

[jira] [Created] (CARBONDATA-1366) When sort_scope=global_sort, use 'StorageLevel.MEMORY_AND_DISK_SER' instead of 'StorageLevel.MEMORY_AND_DISK' for 'convertRDD' persisting to improve loading perform

2017-08-08 Thread Zhichao Zhang (JIRA)
Zhichao Zhang created CARBONDATA-1366: -- Summary: When sort_scope=global_sort, use 'StorageLevel.MEMORY_AND_DISK_SER' instead of 'StorageLevel.MEMORY_AND_DISK' for 'convertRDD' persisting to improve loading performance Key:

[GitHub] carbondata pull request #1245: [CARBONDATA-1366]Change rdd storage level to ...

2017-08-08 Thread zzcclp
GitHub user zzcclp opened a pull request: https://github.com/apache/carbondata/pull/1245 [CARBONDATA-1366]Change rdd storage level to 'MEMORY_AND_DISK_SER' to improve loading performance when sort_scope=global_sort My testing env and configs are as followings: **Env:**

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1245 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fea

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread xuchuanyin
Github user xuchuanyin commented on the issue: https://github.com/apache/carbondata/pull/1245 Would it be better to make a configuration (or read configuration from spark) for this?cc @jackylk --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/144/ --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/1245 @jackylk Agreed with @xuchuanyin, Spark’s storage levels are meant to provide different trade-offs between memory usage and CPU efficiency. So different environment correspond to different st

[GitHub] carbondata issue #1204: [CARBONDATA-1326] Fixed normal/low priority findbug ...

2017-08-08 Thread dhatchayani
Github user dhatchayani commented on the issue: https://github.com/apache/carbondata/pull/1204 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wish

[jira] [Created] (CARBONDATA-1367) Fix wrong dependency of carbondata-examples-flink

2017-08-08 Thread xuchuanyin (JIRA)
xuchuanyin created CARBONDATA-1367: -- Summary: Fix wrong dependency of carbondata-examples-flink Key: CARBONDATA-1367 URL: https://issues.apache.org/jira/browse/CARBONDATA-1367 Project: CarbonData

[GitHub] carbondata pull request #1246: [CARBONDATA-1367] Change module dependency fr...

2017-08-08 Thread xuchuanyin
GitHub user xuchuanyin opened a pull request: https://github.com/apache/carbondata/pull/1246 [CARBONDATA-1367] Change module dependency from spark1 to spark2 # Problems When compiling Carbondata with profile spark-2.1, `carbondata-examples-flink` will be compiled too. But this m

[GitHub] carbondata issue #1246: [CARBONDATA-1367] Change module dependency from spar...

2017-08-08 Thread xuchuanyin
Github user xuchuanyin commented on the issue: https://github.com/apache/carbondata/pull/1246 This problem has already been solved by @chenliang613 in https://github.com/apache/carbondata/pull/1232/commits/fd8299497f68c1e7307450e321e761913a817788 by moving `example-flink` from spark-2

[GitHub] carbondata pull request #1246: [CARBONDATA-1367] Change module dependency fr...

2017-08-08 Thread xuchuanyin
Github user xuchuanyin closed the pull request at: https://github.com/apache/carbondata/pull/1246 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the featur

[GitHub] carbondata issue #1246: [CARBONDATA-1367] Change module dependency from spar...

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1246 SDV Build Success with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/145/ --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/146/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/147/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread sraghunandan
Github user sraghunandan commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wis

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/148/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 Jenkins 2.1 issue is due to: ``` 16:23:28 java.lang.RuntimeException: com.sun.tools.attach.AttachNotSupportedException: Unable to open socket file: target process not responding or HotS

[GitHub] carbondata pull request #1238: [CARBONDATA-1363] Add DataMapWriter interface

2017-08-08 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1238#discussion_r131967510 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/dev/DataMap.java --- @@ -26,13 +28,6 @@ public interface DataMap { /**

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/149/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1238: [CARBONDATA-1363] Add DataMapWriter interface

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1238 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/150/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata pull request #1247: Fix HDFS lock issue in SDV cluster

2017-08-08 Thread ravipesala
GitHub user ravipesala opened a pull request: https://github.com/apache/carbondata/pull/1247 Fix HDFS lock issue in SDV cluster You can merge this pull request into a Git repository by running: $ git pull https://github.com/ravipesala/incubator-carbondata test-fix Alternative

[GitHub] carbondata issue #1247: Fix HDFS lock issue in SDV cluster

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1247 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/151/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1247: Fix HDFS lock issue in SDV cluster

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1247 SDV Build Success with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/152/ --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] carbondata issue #1247: Fix HDFS lock issue in SDV cluster

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1247 SDV Build Success with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/153/ --- If your project is set up for it, you can reply to this email and have your reply

[jira] [Created] (CARBONDATA-1368) HDFS lock issue in SDV cluster

2017-08-08 Thread Ravindra Pesala (JIRA)
Ravindra Pesala created CARBONDATA-1368: --- Summary: HDFS lock issue in SDV cluster Key: CARBONDATA-1368 URL: https://issues.apache.org/jira/browse/CARBONDATA-1368 Project: CarbonData Iss

[GitHub] carbondata issue #1247: [CARBONDATA-1368] Fix HDFS lock issue in SDV cluster

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1247 SDV Build Success with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/154/ --- If your project is set up for it, you can reply to this email and have your reply

[jira] [Closed] (CARBONDATA-1367) Fix wrong dependency of carbondata-examples-flink

2017-08-08 Thread xuchuanyin (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1367?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xuchuanyin closed CARBONDATA-1367. -- Resolution: Duplicate > Fix wrong dependency of carbondata-examples-flink > ---

[GitHub] carbondata issue #1238: [CARBONDATA-1363] Add DataMapWriter interface

2017-08-08 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/carbondata/pull/1238 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] carbondata issue #1238: [CARBONDATA-1363] Add DataMapWriter interface

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1238 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/155/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1247: [CARBONDATA-1368] Fix HDFS lock issue in SDV cluster

2017-08-08 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/carbondata/pull/1247 LGTM spark2.1 CI failed due to apache Jenkin issue --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not ha

[GitHub] carbondata pull request #1247: [CARBONDATA-1368] Fix HDFS lock issue in SDV ...

2017-08-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/1247 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[jira] [Resolved] (CARBONDATA-1368) HDFS lock issue in SDV cluster

2017-08-08 Thread Jacky Li (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1368?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jacky Li resolved CARBONDATA-1368. -- Resolution: Fixed Fix Version/s: 1.2.0 > HDFS lock issue in SDV cluster > --

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread jbonofre
Github user jbonofre commented on the issue: https://github.com/apache/carbondata/pull/1230 @chenliang613 The build is ok 2.1: it failed due to tests: https://builds.apache.org/job/carbondata-pr-spark-2.1/org.apache.carbondata$carbondata-spark-common-test/118/testReport/junit/

[GitHub] carbondata issue #1230: [WIP]corrected sdv test cases

2017-08-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1230 SDV Build Failed with Spark 2.1, Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/156/ --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread zzcclp
Github user zzcclp commented on the issue: https://github.com/apache/carbondata/pull/1245 @jackylk @watermen @xuchuanyin thanks for your suggestion, I have added option 'carbon.global.sort.rdd.storage.level', please review. --- If your project is set up for it, you can reply to this