[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap pre an...

2018-06-21 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2389#discussion_r197027345 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonCreateDataMapCommand.scala --- @@ -128,7

[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap pre an...

2018-06-21 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2389#discussion_r197026107 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala --- @@ -272,6 +284,13 @@ class

[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap pre an...

2018-06-21 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2389#discussion_r197025960 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala --- @@ -272,6 +284,13 @@ class

[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap pre an...

2018-06-21 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2389#discussion_r197019345 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/events/DataMapEvents.scala --- @@ -0,0 +1,76 @@ +/* + * Licensed

[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap pre an...

2018-06-21 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2389#discussion_r197019227 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala --- @@ -272,6 +284,13 @@ class

[GitHub] carbondata pull request #2389: [CARBONDATA-2623][DataMap] Add DataMap Pre an...

2018-06-20 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/carbondata/pull/2389 [CARBONDATA-2623][DataMap] Add DataMap Pre and Pevent listener Added Pre and Post Execution Events for index datamap - [X] Any interfaces changed? None - [X] Any