Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1590
@QiangCai yes we will do it now
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1580
[CARBONDATA-1808] Inconsistency in create table and alter table usage for
char and varchar
â¦har column
Be sure to do all of the following checklist to help us incorporate
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1545
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1545
retest this please
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1545
[CARBONDATA-1710]Resolved The Bug For Alter Tabel on decimal typethrows
exception when first alter is failed
Be sure to do all of the following checklist to help us incorporate
your
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1491
retest this please
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/931
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/944
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/967
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/570
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/531
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1491
@jackylk why build is failing i have clean build on machine and also
resolved all review comments
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1491
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1491
Retest this
---
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1491#discussion_r150858848
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/dataload/TestLoadDataFrame.scala
---
@@ -70,7 +82,17
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1491
[CARBONDATA-1651] [Supported Boolean Type When Saving DataFrame] Provided
Support For Boolean Data Type In CarbonDataFrameWriter
1.Provided Support For Boolean Data Type
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1475
@chenliang613 my pr 1486 has resolved both the issues
CARBONDATA-1694,CARBONDATA-1677 I cut my branch CARBONDATA-1694 from this
branch to make sure that if this pr get merged first
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/1475
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1413
@ravipesala can you review this PR?
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1486
@chenliang613 can you please review this pr?
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1486
[CARBONDATA-1694] [BugFix] Resolved bug for Incorrect exception on presto
CLI when a column is dropped from carbon
**Steps to reproduce** : same as
https://issues.apache.org/jira/browse
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1475
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1475
@chenliang can you please review this pr?
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1475
Retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1475
@chenliang
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1475
[CARBONDATA-1677] [BugFix] resolved bug for presto not showing table not
found exception when table is dropped from carbon
https://issues.apache.org/jira/browse/CARBONDATA-1677
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/1050
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1050
@sraghunandan i am unable to reproduce this bug closing the ticket
---
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1413#discussion_r145876968
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSource.scala ---
@@ -233,10 +233,17 @@ object CarbonSource
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1291
@cenyuhai I tried running your pr and got this error
Caused by: java.lang.NullPointerException
at
org.apache.carbondata.hive.MapredCarbonInputFormat.populateCarbonTable
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1363
@cenyuhai what is the grammar for creating carbon table in hive?
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
retest this please
---
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1291#discussion_r139315604
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -133,7 +133,7 @@ public static void setTableInfo
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1356
@chenliang613 I attached the csv with same JIRA issue
https://issues.apache.org/jira/browse/CARBONDATA-1477
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1356
Retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1353
Retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1360
@chenliang613
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1360
[CARBONDATA-1482] fixed the failing test cases of presto integration
1.builds are passing
2.added dependency for spark guava
3.throw exception in case carbondatastore creator
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1353
restest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1356
Retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
this pr required #1356 to be merged first because of wrong values of date
type issue in hive
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1356
@chenliang613 can you review this pr?
---
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1356#discussion_r138659199
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupport.java
---
@@ -228,9 +229,13 @@ private
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1356#discussion_r138656706
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupport.java
---
@@ -228,9 +229,13 @@ private
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1356
@chenliang613 @cenyuhai
---
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1347#discussion_r138604260
--- Diff: integration/hive/hive-guide.md ---
@@ -91,7 +91,6 @@ $HIVE_HOME/bin/hive
### Query data from hive table
```
-alter
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1347#discussion_r138604013
--- Diff: integration/hive/hive-guide.md ---
@@ -91,7 +91,6 @@ $HIVE_HOME/bin/hive
### Query data from hive table
```
-alter
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1356
[CARBONDATA-1477] corrected the bug for wrong values of date data type
https://issues.apache.org/jira/browse/CARBONDATA-1477
You can merge this pull request into a Git repository by running
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
@chenliang613 merge conflict is fixed
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
Retest this please
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1347
@chenliang613 @cenyuhai
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1347
[CARBONDATA-1465] resolved bug for hive cant query carbon table when column
name is in â¦
1.resolved bug for hive can't query carbon when column name is in small
letters
2.corrected
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/985#discussion_r137943202
--- Diff: integration/hive/pom.xml ---
@@ -126,20 +126,20 @@
+src/test/scala
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/985#discussion_r137943182
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupport.java
---
@@ -231,11 +231,11 @@ private
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/1257
---
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1338
[CARBONDATA-1461]resolved bug for date and timestamp in hive
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anubhav100/incubator
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/1326
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1326
retest this please
---
Github user anubhav100 closed the pull request at:
https://github.com/apache/carbondata/pull/897
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/897
I am closing this pr I have raised another pr for this issue
https://github.com/apache/carbondata/pull/1326
---
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
@cenyuhai @chenliang613 can you review 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 not have
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
retest this please
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1312
retest this please
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1312
@cenyuhai @chenliang613
---
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 anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1312
[CARBONDATA-1441]resolved the bug for schema not reflect back in hive
https://issues.apache.org/jira/browse/CARBONDATA-1441
You can merge this pull request into a Git repository by running
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
ok to test
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1303
@chenliang613 can you please review 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 not have
GitHub user anubhav100 opened a pull request:
https://github.com/apache/carbondata/pull/1303
[CARBONDATA-1423] added integration test cases for presto
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/anubhav100/incubator
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1239
@cenyhui can you tried running hive example and see whether it's working
or not last time I check it was working fine on latest master
---
If your project is set up for it, you can reply
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r134780421
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/constants/PrestoCommonConstants.java
---
@@ -0,0 +1,29
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r134779856
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataRecordSetProvider.java
---
@@ -73,27 +60,23 @@ public
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1257
@chenliang613 i have update the pr you can have a look
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer as you suggested i tried it on 3 node presto cluster but bug is not
getting replicated
in presto
select * from nation where n_name='INDIA';
n_nationkey | n_name
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r134219468
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/memory/LocalMemoryContext.java
---
@@ -0,0 +1,43
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r134219490
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/memory/AbstractAggregatedMemoryContext.java
---
@@ -0,0 +1,40
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r134219416
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/processor/AbstractDataBlockIterator.java
---
@@ -0,0 +1,239
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1255#discussion_r134084229
--- Diff: integration/hive/pom.xml ---
@@ -36,122 +36,96 @@
-org.apache.hive
-hive-exec
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1239
@cenyuhai this issue is resolved in latest master
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer
in carbon
create table nation5 ( n_nationkey int, n_name string, n_regionkey int,
n_comment string ) stored by 'carbondata';
LOAD DATA INPATH '/nation.csv
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer ya i will 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 feature
enabled
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer please give me nation_null.CSV as well I will try to replicate it
again today
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer please correct your PR title as per :
https://github.com/apache/carbondata/blob/master/docs/How-to-contribute-to-Apache-CarbonData.md
---
If your project is set up for it, you can
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@steven-qin
1.please correct your PR title as per :
https://github.com/apache/carbondata/blob/master/docs/How-to-contribute-to-Apache-CarbonData.md
2.there is still problem
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@chenliang sure he has done a great job
---
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
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133407524
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/constants/CarbonCommonConstants.java
---
@@ -0,0 +1,1319
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer how to test this pr can you provide reproducable steps?
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@steven-qin this pr has resolved this issue with more optimizations #1257
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1235
@steven-qin can you please provide reproducable steps?
---
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 anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133371582
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/expression/ExpressionResult.java
---
@@ -183,7 +183,7 @@ public String getString
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/985
retest this please
---
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 anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1218#discussion_r131846650
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -510,9 +511,16 @@
* LOAD_STATUS SUCCESS
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1218
@chenliang613 should i add unit test cases for all newly added classes
---
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 anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1218#discussion_r131846232
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/ColumnBasedResultCollector.java
---
@@ -0,0 +1,233
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1231
retest this please
---
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 anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1231
@cenyuhai that problem is resolved by 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 not have
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1231
@cenyuhai there is one more problem we are not able to run hive example
this PR also fixed that issue, can you please try running hive example from
your PR I don't think that will work from
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1218
@chenliang613 my last commit was slower because it is giving wrong row
count it is giving row count as zero that's why it was faster,so I corrected
that in the latest commit so it is slower
201 - 300 of 366 matches
Mail list logo