Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90031795
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala
---
@@ -101,47 +101,107 @@ object DistributionUtil
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/367
I think @ravipesala is still working on #333
---
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
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/366#discussion_r89998317
--- Diff: pom.xml ---
@@ -93,6 +93,7 @@
common
+ format
--- End diff --
should not add format
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/278
LGTM
CI passed
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/700/
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/367
Why include PR333 in this PR? Does it depends on PR333?
---
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
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/278#discussion_r89949404
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/csvreaderstep/CsvInput.java
---
@@ -384,21 +383,75 @@ public boolean
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/278#discussion_r89948611
--- Diff:
integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala
---
@@ -932,7 +942,8 @@ object
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/365
CI passed
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/696/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/365
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/689/
---
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
GitHub user jackylk opened a pull request:
https://github.com/apache/incubator-carbondata/pull/365
[CARBONDATA-463] Extract code to spark-common
Move MM
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jackylk/incubator-carbondata
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/363
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/681/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/357
LGTM
---
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 feature
enabled and wishes so
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/360
CI
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/677/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/359
LGTM
---
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 feature
enabled and wishes so
GitHub user jackylk opened a pull request:
https://github.com/apache/incubator-carbondata/pull/363
[CARBONDATA-461] clean partitioner in carbon RDD package
To make carbon RDDs reusable in spark2 integration, need to remove
partitioner in RDDs
You can merge this pull request
Github user jackylk closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/356
---
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 feature
enabled and wishes so
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/212
This will be done in another PR for spark2 integration
closing it
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jackylk closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/212
---
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 feature
enabled and wishes so
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/242
This will be done in another PR
closing it
---
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
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/262
CI running
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/661/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/262
CI running
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/659/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/351
CI passed
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/652/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
GitHub user jackylk opened a pull request:
https://github.com/apache/incubator-carbondata/pull/351
[CARBONDATA-447] Use Carbon log service instead of spark Logging
Change all place that uses spark Logging interface, use log service in
carbon-common module. This is a preparation
GitHub user jackylk opened a pull request:
https://github.com/apache/incubator-carbondata/pull/339
[CARBONDATA-429][WIP] Remove unnecessary file name check in dictionary cache
In dictionary cache, there are currently necessary file name check for each
column, which increase
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/324
LGTM
---
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 feature
enabled and wishes so
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/254
please rebase.
And why it can not support create table like carbon table? It just need the
schema of the like table, right?
---
If your project is set up for it, you can reply
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/278
please rebase
---
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 feature
enabled
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/324#discussion_r88768124
--- Diff: conf/carbon.properties.template ---
@@ -52,7 +52,7 @@ carbon.compaction.level.threshold=4,3
#default size (in MB) for major
501 - 528 of 528 matches
Mail list logo