[GitHub] carbondata pull request #1439: [CARBONDATA-1628] Re-factory LoadTableCommand...

2017-10-28 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1439#discussion_r147551479 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/DataLoadingUtil.scala --- @@ -0,0 +1,300 @@ +/* + *

[GitHub] carbondata pull request #1439: [CARBONDATA-1628] Re-factory LoadTableCommand...

2017-10-27 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1439#discussion_r147454328 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/CommonUtil.scala --- @@ -831,4 +835,207 @@ object CommonUtil

[GitHub] carbondata pull request #1439: [CARBONDATA-1628] Re-factory LoadTableCommand...

2017-10-27 Thread QiangCai
GitHub user QiangCai opened a pull request: https://github.com/apache/carbondata/pull/1439 [CARBONDATA-1628] Re-factory LoadTableCommand to reuse code for streaming ingest in the future Re-factory LoadTableCommand to reuse code for streaming ingest in the future You can merge this