[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-09 Thread zzcclp
Github user zzcclp commented on the issue: https://github.com/apache/carbondata/pull/1245 @jackylk @watermen @xuchuanyin thanks for your suggestion, I have added option 'carbon.global.sort.rdd.storage.level', please review. --- If your project is set up for it, you can reply to this

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/1245 @jackylk Agreed with @xuchuanyin, Spark’s storage levels are meant to provide different trade-offs between memory usage and CPU efficiency. So different environment correspond to different

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread xuchuanyin
Github user xuchuanyin commented on the issue: https://github.com/apache/carbondata/pull/1245 Would it be better to make a configuration (or read configuration from spark) for this?cc @jackylk --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] carbondata issue #1245: [CARBONDATA-1366]Change rdd storage level to 'MEMORY...

2017-08-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/1245 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this