[
https://issues.apache.org/jira/browse/CARBONDATA-463?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ravindra Pesala resolved CARBONDATA-463.
Resolution: Fixed
> Extract spark-common module
> ---
>
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/365
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
[
https://issues.apache.org/jira/browse/CARBONDATA-361?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15707814#comment-15707814
]
SWATI RAO commented on CARBONDATA-361:
--
[~ravi.pesala] : As mentioned on your wiki page which is
GitHub user kunal642 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/371
[CARBONDATA-472] Added unit test for cache package
Added unit test for CarbonLRUCacheTest
Added unit test for ColumnDictionaryChunkIteratorTest
Added unit test for
SWATI RAO created CARBONDATA-472:
Summary: Improve code coverage for core.cache package.
Key: CARBONDATA-472
URL: https://issues.apache.org/jira/browse/CARBONDATA-472
Project: CarbonData
GitHub user PallaviSingh1992 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/370
[CARBONDATA-457] Added Unit Test cases for core.writer package
You can merge this pull request into a Git repository by running:
$ git pull
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/369
@allwefantasy Thanks for your suggestion. Yes, we can use the Spark's
memory manager instead of our own. I will open the interface and provide
implementation to use
Github user allwefantasy commented on the issue:
https://github.com/apache/incubator-carbondata/pull/369
Does this PR have considered allocating memory from TaskMemoryManagerï¼
Many Spark application runs on Yarn,if you use off-heap,it's easy to trigger
behavior of yarn's killing
GitHub user ravipesala opened a pull request:
https://github.com/apache/incubator-carbondata/pull/369
[CARBONDATA-470][WIP]Add unsafe offheap and on-heap sort in carbodata
loading
In the current carbondata system loading performance is not so encouraging
since we need to sort the
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/333
Updated PR, please review 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 not have this
Ravindra Pesala created CARBONDATA-471:
--
Summary: Optimize no kettle flow and fix issues in cluster
Key: CARBONDATA-471
URL: https://issues.apache.org/jira/browse/CARBONDATA-471
Project:
Github user kumarvishal09 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/265
http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/711/
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Ravindra Pesala created CARBONDATA-470:
--
Summary: Add unsafe offheap and on-heap sort in carbodata loading
Key: CARBONDATA-470
URL: https://issues.apache.org/jira/browse/CARBONDATA-470
Project:
Github user kumarvishal09 closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/354
---
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, or
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/365
LGTM except few comments.
It seems there is an issue in pom files while using maven default build and
using format. I will fix them in new PR.
---
If your project is set up for
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/358
CI cannot pass right now as there are thrift changes. But I have verified
in local and all tests are passing.
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/365#discussion_r90044286
--- Diff:
integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/NewCarbonDataLoadRDD.scala
---
@@ -51,7 +51,6 @@ class
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/365#discussion_r90044359
--- Diff:
integration/spark-common/src/main/java/org/apache/carbondata/integration/spark/merger/CarbonCompactionUtil.java
---
@@ -38,7 +38,6
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/365#discussion_r90040864
--- Diff:
integration/spark-common/src/main/java/org/apache/carbondata/integration/spark/merger/CarbonCompactionUtil.java
---
@@ -38,7
Github user allwefantasy commented on the issue:
https://github.com/apache/incubator-carbondata/pull/367
Ok, i will remove the 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 does not have this
Github user allwefantasy closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/367
---
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, or
Ravindra Pesala created CARBONDATA-469:
--
Summary: Optimize join in spark using bucketing information
Key: CARBONDATA-469
URL: https://issues.apache.org/jira/browse/CARBONDATA-469
Project:
[
https://issues.apache.org/jira/browse/CARBONDATA-467?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ravindra Pesala updated CARBONDATA-467:
---
Summary: Add bucketing information while creating table and update in
thrift
Ravindra Pesala created CARBONDATA-467:
--
Summary: Add bucketing information while creating table and update
in thrift format.
Key: CARBONDATA-467
URL: https://issues.apache.org/jira/browse/CARBONDATA-467
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90033298
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala
---
@@ -101,47 +101,107 @@ object DistributionUtil
Ravindra Pesala created CARBONDATA-466:
--
Summary: Implement bucketing table in carbondata
Key: CARBONDATA-466
URL: https://issues.apache.org/jira/browse/CARBONDATA-466
Project: CarbonData
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90032032
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala
---
@@ -101,47 +101,107 @@ object DistributionUtil
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90032072
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala
---
@@ -101,47 +101,107 @@ object DistributionUtil
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90031584
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/CarbonContext.scala ---
@@ -208,4 +208,36 @@ object CarbonContext {
}
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/362#discussion_r90031916
--- Diff:
integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala
---
@@ -101,47 +101,107 @@ object DistributionUtil
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 ashokblend commented on the issue:
https://github.com/apache/incubator-carbondata/pull/366
i have removed changes in pom.xml, but before merging we need to deploy
carbon-format as there is change in schema.thrift.
---
If your project is set up for it, you can reply to
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 not
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 allwefantasy commented on the issue:
https://github.com/apache/incubator-carbondata/pull/367
Can we merge PR333 first then merge this PR?
---
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 QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/366
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 and
Github user allwefantasy commented on the issue:
https://github.com/apache/incubator-carbondata/pull/367
I am not sure whether it depends on PR333ãI try to resolve issues when
carbondata working with Spark Streaming by Merging PR333 however it does not
work. Maybe I should
[
https://issues.apache.org/jira/browse/CARBONDATA-368?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jacky Li resolved CARBONDATA-368.
-
Resolution: Fixed
Fix Version/s: 0.3.0-incubating
> Should improve performance of
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 on
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
40 matches
Mail list logo