[GitHub] incubator-carbondata pull request #767: fix sort_columns issue(12-dev)

2017-04-10 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/767#discussion_r110694824 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/filter/resolver/RowLevelRangeFilterResolverImpl.java --- @@ -147,12 +148,16 @@

[GitHub] incubator-carbondata pull request #767: fix sort_columns issue(12-dev)

2017-04-10 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/767#discussion_r110694868 --- Diff: core/src/main/java/org/apache/carbondata/core/util/ByteUtil.java --- @@ -540,7 +543,7 @@ public static int toInt(byte[] bytes, int

[GitHub] incubator-carbondata pull request #767: fix sort_columns issue(12-dev)

2017-04-08 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/767#discussion_r110531208 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/filter/resolver/RowLevelRangeFilterResolverImpl.java --- @@ -147,12 +148,16 @@

[GitHub] incubator-carbondata pull request #767: fix sort_columns issue(12-dev)

2017-04-08 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/767#discussion_r110531179 --- Diff: core/src/main/java/org/apache/carbondata/core/util/ByteUtil.java --- @@ -540,7 +543,7 @@ public static int toInt(byte[] bytes, int