[GitHub] carbondata pull request #2623: [CARBONDATA-2844] Pass SK/AK to executor by s...

2018-08-26 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2623#discussion_r212869553 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSourceProvider.java --- @@ -79,6 +80,7 @@ @Override

[GitHub] carbondata pull request #2623: [CARBONDATA-2844] Pass SK/AK to executor by s...

2018-08-26 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2623#discussion_r212868174 --- Diff: integration/presto/src/test/scala/org/apache/carbondata/presto/server/PrestoServer.scala --- @@ -98,6 +99,7 @@ object PrestoServer {

[GitHub] carbondata pull request #2623: [CARBONDATA-2844] Pass SK/AK to executor by s...

2018-08-26 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2623#discussion_r212868149 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSourceProvider.java --- @@ -79,6 +80,7 @@ @Override