[GitHub] carbondata pull request #927: [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for ...

2017-05-19 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/927#discussion_r117488392 --- Diff: integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala --- @@ -495,15 +495,36 @@ abstract class

[GitHub] carbondata pull request #927: [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for ...

2017-05-19 Thread rahulforallp
GitHub user rahulforallp reopened a pull request: https://github.com/apache/carbondata/pull/927 [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for other table properties and NPE in compaction 1. Measure shouldn't supported inside no_inverted_index , if it is not included in

[GitHub] carbondata pull request #927: [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for ...

2017-05-19 Thread rahulforallp
Github user rahulforallp commented on a diff in the pull request: https://github.com/apache/carbondata/pull/927#discussion_r117447500 --- Diff: core/src/main/java/org/apache/carbondata/core/util/ByteUtil.java --- @@ -66,6 +66,26 @@ public static int compare(byte[] buffer1, byte[]

[GitHub] carbondata pull request #927: [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for ...

2017-05-19 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/927#discussion_r117430177 --- Diff: processing/src/main/java/org/apache/carbondata/processing/store/CarbonFactDataHandlerColumnar.java --- @@ -1366,8 +1350,9 @@ public

[GitHub] carbondata pull request #927: [CARBONDATA-1066] Fixed NO_INVERTED_INDEX for ...

2017-05-19 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/927#discussion_r117429668 --- Diff: core/src/main/java/org/apache/carbondata/core/util/ByteUtil.java --- @@ -66,6 +66,26 @@ public static int compare(byte[] buffer1, byte[]