[GitHub] incubator-carbondata pull request #841: [CARBONDATA-976] Fixed issue for wro...

2017-04-25 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/841#discussion_r113147895 --- Diff: integration/spark2/src/test/scala/org/apache/spark/carbondata/restructure/rowreader/AddColumnTestCases.scala --- @@ -207,6 +2

[GitHub] incubator-carbondata pull request #841: [CARBONDATA-976] Fixed issue for wro...

2017-04-25 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/841#discussion_r113147015 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/AlterTableCommands.scala --- @@ -51,15 +52,15 @@ pri

[GitHub] incubator-carbondata pull request #841: [CARBONDATA-976] Fixed issue for wro...

2017-04-24 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/841 [CARBONDATA-976] Fixed issue for wrong value getting deleted from evolutionEntry Analysis: If alter command throws exception then in the revert flow the condition is to check whether