[GitHub] carbondata issue #1267: [CARBONDATA-1326] Findbug fixes

2017-08-23 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1267 LGTM --- 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

[GitHub] carbondata pull request #1289: [CARBONDATA-1406] Fixed the issue with the Qu...

2017-08-23 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/carbondata/pull/1289 [CARBONDATA-1406] Fixed the issue with the Quotation marks You can merge this pull request into a Git repository by running: $ git pull

[jira] [Created] (CARBONDATA-1406) Fix inconsistent usage of QUOTATION MARK " and LEFT DOUBLE QUOTATION MARK “ in installation.md file

2017-08-23 Thread Pallavi Singh (JIRA)
Pallavi Singh created CARBONDATA-1406: - Summary: Fix inconsistent usage of QUOTATION MARK " and LEFT DOUBLE QUOTATION MARK “ in installation.md file Key: CARBONDATA-1406 URL:

[GitHub] carbondata pull request #1288: [CARBONDATA-1405]Added support for gathering ...

2017-08-23 Thread sraghunandan
GitHub user sraghunandan opened a pull request: https://github.com/apache/carbondata/pull/1288 [CARBONDATA-1405]Added support for gathering coverage information when tests are run Using jacoco for coverage Using coveralls for CI integration You can merge this pull request into

[jira] [Created] (CARBONDATA-1405) Configure Coverage report to be shown for every build

2017-08-23 Thread Raghunandan (JIRA)
Raghunandan created CARBONDATA-1405: --- Summary: Configure Coverage report to be shown for every build Key: CARBONDATA-1405 URL: https://issues.apache.org/jira/browse/CARBONDATA-1405 Project:

[GitHub] carbondata pull request #1287: Fix Findbugs

2017-08-23 Thread sraghunandan
GitHub user sraghunandan opened a pull request: https://github.com/apache/carbondata/pull/1287 Fix Findbugs You can merge this pull request into a Git repository by running: $ git pull https://github.com/sraghunandan/carbondata-1 fix_findbugs Alternatively you can review and

[GitHub] carbondata pull request #1281: [CARBONDATA-1326] Fixed findbug issue and uni...

2017-08-23 Thread sraghunandan
Github user sraghunandan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1281#discussion_r134849821 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/filter/FilterExpressionProcessor.java --- @@ -163,17 +163,17 @@ public

[GitHub] carbondata issue #1285: [CARBONDATA-1403] Compaction log is not correct

2017-08-23 Thread chenerlu
Github user chenerlu commented on the issue: https://github.com/apache/carbondata/pull/1285 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 pull request #1257: [CARBONDATA-1347] Implemented Columnar Readin...

2017-08-23 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1257#discussion_r134780421 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/constants/PrestoCommonConstants.java --- @@ -0,0 +1,29 @@ +/* + *

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

2017-08-23 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1257#discussion_r134779856 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataRecordSetProvider.java --- @@ -73,27 +60,23 @@ public

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

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

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

2017-08-23 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1259 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/3424/ --- If your project is set up for it, you can reply to this email and have

[GitHub] carbondata issue #1265: [CARBONDATA-1128] Add direct string encoding for sho...

2017-08-23 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/carbondata/pull/1265 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 pull request #1286: [CARBONDATA-1404] Added Unit test cases for H...

2017-08-23 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/carbondata/pull/1286 [CARBONDATA-1404] Added Unit test cases for Hive Integration You can merge this pull request into a Git repository by running: $ git pull

[GitHub] carbondata issue #1285: [CARBONDATA-1403] Compaction log is not correct

2017-08-23 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1285 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

[jira] [Created] (CARBONDATA-1404) Add Unit test cases for Hive Integration

2017-08-23 Thread Pallavi Singh (JIRA)
Pallavi Singh created CARBONDATA-1404: - Summary: Add Unit test cases for Hive Integration Key: CARBONDATA-1404 URL: https://issues.apache.org/jira/browse/CARBONDATA-1404 Project: CarbonData

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

2017-08-23 Thread sraghunandan
Github user sraghunandan commented on the issue: https://github.com/apache/carbondata/pull/1259 ok to test --- 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,

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

2017-08-23 Thread sraghunandan
Github user sraghunandan commented on the issue: https://github.com/apache/carbondata/pull/1259 ok to test --- 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,

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

2017-08-23 Thread sraghunandan
Github user sraghunandan commented on the issue: https://github.com/apache/carbondata/pull/1259 ok to test --- 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,

[GitHub] carbondata issue #1284: [CARBONDATA-1330] Not able to drop temporary table b...

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1284 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

[GitHub] carbondata issue #1278: [CARBONDATA-1326] Fixed high priority findbug issues

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1278 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

[GitHub] carbondata pull request #1285: [CARBONDATA-1403] Compaction log is not corre...

2017-08-23 Thread chenerlu
GitHub user chenerlu opened a pull request: https://github.com/apache/carbondata/pull/1285 [CARBONDATA-1403] Compaction log is not correct Modify reason: The log of compaction is not correct, So change it. You can merge this pull request into a Git repository by running: $

[jira] [Created] (CARBONDATA-1403) Compaction log is not correct

2017-08-23 Thread chenerlu (JIRA)
chenerlu created CARBONDATA-1403: Summary: Compaction log is not correct Key: CARBONDATA-1403 URL: https://issues.apache.org/jira/browse/CARBONDATA-1403 Project: CarbonData Issue Type: Bug

[GitHub] carbondata issue #1282: [CARBONDATA-1221] Documentation - removed the unused...

2017-08-23 Thread Ayushi93
Github user Ayushi93 commented on the issue: https://github.com/apache/carbondata/pull/1282 @chenliang It is already configured. ☺️ --- 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

[GitHub] carbondata pull request #1278: [CARBONDATA-1326] Fixed high priority findbug...

2017-08-23 Thread PannerselvamVelmyl
GitHub user PannerselvamVelmyl reopened a pull request: https://github.com/apache/carbondata/pull/1278 [CARBONDATA-1326] Fixed high priority findbug issues No new test cases are required. Few of the findbugs issues fixed. You can merge this pull request into a Git repository by

[GitHub] carbondata pull request #1278: [CARBONDATA-1326] Fixed high priority findbug...

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl closed the pull request at: https://github.com/apache/carbondata/pull/1278 --- 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

[GitHub] carbondata issue #1278: [CARBONDATA-1326] Fixed high priority findbug issues

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1278 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

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

2017-08-23 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1257#discussion_r134694501 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataRecordSet.java --- @@ -79,30 +81,27 @@ private Expression

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

2017-08-23 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1257#discussion_r134692412 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataRecordSetProvider.java --- @@ -73,27 +60,23 @@ public

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

2017-08-23 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1257#discussion_r134691842 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/constants/PrestoCommonConstants.java --- @@ -0,0 +1,29 @@ +/* + *

[GitHub] carbondata issue #1266: [CARBONDATA-1393] Avoid to throw NPE when execute 'f...

2017-08-23 Thread zzcclp
Github user zzcclp commented on the issue: https://github.com/apache/carbondata/pull/1266 @ravipesala @jackylk please review, thanks. --- 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

[GitHub] carbondata issue #1282: [CARBONDATA-1221] Documentation - removed the unused...

2017-08-23 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1282 @Ayushi93 please config your email info in your local git environment. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] carbondata issue #1282: [CARBONDATA-1221] Documentation - removed the unused...

2017-08-23 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/1282 LGTM --- 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

[GitHub] carbondata issue #1278: [CARBONDATA-1326] Fixed high priority findbug issues

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1278 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

[GitHub] carbondata pull request #1248: [CARBONDATA-1371] Support creating decoder ba...

2017-08-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/1248 --- 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

[GitHub] carbondata issue #1248: [CARBONDATA-1371] Support creating decoder based on ...

2017-08-23 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1248 LGTM --- 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

[GitHub] carbondata issue #1278: [CARBONDATA-1326] Fixed high priority findbug issues

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1278 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

[GitHub] carbondata issue #1284: [CARBONDATA-1330] Not able to drop temporary table b...

2017-08-23 Thread PannerselvamVelmyl
Github user PannerselvamVelmyl commented on the issue: https://github.com/apache/carbondata/pull/1284 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

[GitHub] carbondata issue #1259: [Review][CARBONDATA-1381] Add test cases for missing...

2017-08-23 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/1259 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/3423/ --- If your project is set up for it, you can reply to this email and have