Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2991
retest this please
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r246647432
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r246645671
--- Diff: docs/csdk-guide.md ---
@@ -40,6 +66,7 @@ release the memory and destroy JVM.
C++ SDK support read batch row. User can set batch
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r246644993
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r246644367
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r246643403
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244668325
--- Diff: store/CSDK/test/main_ft.cpp ---
@@ -0,0 +1,1172 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244667973
--- Diff: pom.xml ---
@@ -587,6 +587,9 @@
examples/spark2
datamap/lucene
datamap/bloom
+
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244509375
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244509021
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244508891
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244508413
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244508238
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244507697
--- Diff: docs/csdk-guide.md ---
@@ -40,6 +66,7 @@ release the memory and destroy JVM.
C++ SDK support read batch row. User can set batch
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244507596
--- Diff: docs/csdk-guide.md ---
@@ -29,6 +29,32 @@ code and without CarbonSession.
In the carbon jars package, there exist a carbondata
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r244507492
--- Diff: store/CSDK/test/main.cpp ---
@@ -665,6 +699,208 @@ bool readFromS3(JNIEnv *env, char *path, char
*argv[]) {
printResult(env, reader
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r243915655
--- Diff: store/CSDK/test/main.cpp ---
@@ -665,6 +699,208 @@ bool readFromS3(JNIEnv *env, char *path, char
*argv[]) {
printResult(env, reader
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2897#discussion_r243812841
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/util/CarbonVectorizedRecordReader.java
---
@@ -68,6 +67,10 @@
private
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2897#discussion_r243812352
--- Diff:
core/src/main/java/org/apache/carbondata/core/memory/UnsafeMemoryManager.java
---
@@ -173,6 +174,7 @@ public synchronized void freeMemoryAll
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2987#discussion_r243167366
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/DataFrameSDKS3Example.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2897#discussion_r242629291
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/LocalDictDimensionDataChunkStore.java
---
@@ -94,10 +93,9 @@ public
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r242049571
--- Diff: pom.xml ---
@@ -587,6 +587,9 @@
examples/spark2
datamap/lucene
datamap/bloom
+
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2991#discussion_r24204
--- Diff: pom.xml ---
@@ -587,6 +587,9 @@
examples/spark2
datamap/lucene
datamap/bloom
+
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2991
[WIP] Add build script and add test case with GoogleTest Framework for CSDK
## What changes were proposed in this pull request?
Below Points are targeted in this PR
1. Handle build
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2944#discussion_r235866771
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/processor/DataBlockIterator.java
---
@@ -262,6 +262,7 @@ public void close
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2944
[CARBONDATA-3122]CarbonReader memory leak
**Issue Detail**
CarbonReader has List of initialized RecordReader for each Split and
each split holds page data till the reference
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2935
LGTM
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2897#discussion_r230736008
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -64,7 +64,8 @@
private Map options
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2897
[WIP] Supporting local dictionary enable by default for SDK
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [ ] Any
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2794#discussion_r222632757
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/merger/CarbonDataMergerUtil.java
---
@@ -744,6 +744,7 @@ private static long
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2796
[CARBONDATA-2991]NegativeArraySizeException during query execution
Issue :- During Query Execution sometime NegativeArraySizeException
Exception in Some Tasks . And sometime Executor is lost
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2794
[CARBONDATA-2985]Fix issues in Table level compaction and TableProperâ¦
Issue :-
1. If 2nd Level compaction is 1 like 2,1 or 6,1 then only 1st time
compaction is done subsequent
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2785
[CARBONDATA-2986] Table Properties are lost when multiple driver concâ¦
Issue :- When concurrently multiple driver is creating table , for same
table table properties are lost .
Root
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2703#discussion_r217060418
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/BlockletDetailInfo.java
---
@@ -51,7 +51,8 @@
private short
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2703
[CARBONDATA-2925]Wrong data displayed for spark file format if carbonâ¦
Issue:- if Carbon file has multiple blocklet ,in select query wrong data
displayed.
Root Cause :- it is showing
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2658
[Carbondata 2885]Broadcast Issue and Small file distribution Issue
Issue :-
1. In External Table Carbon Relation sizeInByte is wrong (always 0)
because of this Join Queries
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2606#discussion_r208515484
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/CarbonRowDataWriterProcessorStepImpl.java
---
@@ -169,24 +171,36
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2606
[CARBONDATA-2817]Thread Leak in Update and in No sort flow
Issue :- After Update Command is finished , Loading threads are not
getting stopped.
Root Cause :-
1. In Update flow
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2542
[CARBONDATA-2772] Size based dictionary fallback is failing even threâ¦
Issue:- Size Based Fallback happened even threshold is not reached.
RootCause:- Current size calculation is wrong
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2508
[CARBONDATA-2744]Streaming lock is not released even Batch processingâ¦
Issue Detail :- if Streaming Application is running , DDLs like finish
streaming ,close streaming are blocked
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r201340590
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/createTable/TestNonTransactionalCarbonTable.scala
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r201340518
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -284,6 +286,29 @@ public CarbonWriterBuilder
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820929
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVNonTransactionalCarbonWriterTest.java
---
@@ -284,4 +284,137 @@ public void
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820921
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVNonTransactionalCarbonWriterTest.java
---
@@ -284,4 +284,137 @@ public void
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820912
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -284,6 +286,28 @@ public CarbonWriterBuilder
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820916
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -284,6 +286,28 @@ public CarbonWriterBuilder
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820906
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableSchemaBuilder.java
---
@@ -108,6 +122,20 @@ public TableSchema
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820900
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableSchemaBuilder.java
---
@@ -108,6 +122,20 @@ public TableSchema
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2433#discussion_r200820897
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableSchemaBuilder.java
---
@@ -76,6 +78,18 @@ public TableSchemaBuilder
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2317
[CARBONDATA-2492]Fixed thread leak issue in case of data load failure
Problem: Executor service is not getting shutdown after data load failure.
root cause: When sort step is failing
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2227#discussion_r187087122
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/model/CarbonLoadModelBuilder.java
---
@@ -81,9 +81,8 @@ public
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2227
Retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2227
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2224
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2224
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2227
retest this please
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2245#discussion_r184945715
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -335,10 +335,12 @@ private CarbonTable
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2245
[CARBONDATA-2411] infinite loop when sdk writer throws Exception
Issue & root Cause :- Exception is not handled when Object is cast to
String[] in writer method of CarbonWriter
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2227
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2097
already merged in master PR #2121
---
Github user BJangir closed the pull request at:
https://github.com/apache/carbondata/pull/2097
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2121
retest sdv please
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2227
[CARBONDATA-2401] Date and Timestamp options are not working in SDK
Issue:- Date and Timestamp format is passed in options of SDK but data load
is failed even correct data is set as per format
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2224
retest please
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2224#discussion_r183814030
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVCarbonWriterTest.java
---
@@ -205,4 +205,70 @@ public void testSchemaPersistence
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2224
[CARBONDATA-2393]TaskNo is not working for SDK
Issue:- Task No is not getting reflected in the Carbon Data file and in
index file .
Cause :- Task No is getting overwritten
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2121
[CARBONDATA-2275]Query Failed for 0 byte deletedelta file
When delete is failed on write step because of any exception from hdfs .
Currently 0 bye deletedelta file is created and not getting
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2097
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2097
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/2097
retest this please
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2097#discussion_r176946937
--- Diff:
core/src/main/java/org/apache/carbondata/core/writer/CarbonDeleteDeltaWriterImpl.java
---
@@ -110,7 +111,9 @@ public
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2097#discussion_r176946943
--- Diff:
core/src/main/java/org/apache/carbondata/core/writer/CarbonDeleteDeltaWriterImpl.java
---
@@ -81,7 +80,9 @@ public
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2097
[CARBONDATA-2275]Query Failed for 0 byte deletedelta file
When delete is failed on write step because of any exception from hdfs .
Currently 0 bye deletedelta file is created and not getting
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/2075
[CARBONDATA-2261] Support Set segment command for Streaming Table
Problem Statement :- Currently Set Segment is not supported for Streaming
Segments .
This PR is to support Set segment
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1985
[CARBONDATA-2185]add InputMetrics for Streaming Reader
Issue:- Record count in Input Metrics is always 0 for Stream Reader.
Solution :- InputMetricsStats object need to update
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1971
retest sdv please
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1971
[CARBONDATA-2161]Compacted Segment of Streaming Table should update mâ¦
Problem statement:-
When Handoff is trigger , ROW file format will be converted into COLUMNAR
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1915
[CARBONDATA-1454]false expression handling and block pruning for the â¦
Issue :- In case of wrong value/invalid for time-stamp and date data
type. all blocks are identified for scan
Github user BJangir closed the pull request at:
https://github.com/apache/carbondata/pull/1878
---
Github user BJangir commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1878#discussion_r164813794
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/table/ShowTablesCommand.scala
---
@@ -0,0 +1,72
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1878
[CARBONDATA-2094] Filter DataMap Tables in Show Table Command
Currently Show Table command shows datamap tables (agg tablels) but show
table command should not show aggregate tables
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1861
@xubo245 ,
if agg table is already created and create datamap command is fired again
with "if not exists" syntax then processMetadata will not return any Error (as
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1824
[CARBONDATA-2046]agg Query failed when non supported aggregate is present
in Query
Root Cause :- isValidPlan variable was getting overwritten by
CarbonReflectionUtils.hasPredicateSubquery
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1807
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1807
retest this please
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1807
[CARBONDATA-2030]avg with Aggregate table for double data type is failed.
avg with Aggregate table for double data type is failed.
Root Cause:- for avg column , agg table returns sum
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1804
[CARBONDATA-2028] Select Query failed with preagg having timeseries and
normal agg table together
Select Query failed with preagg having timeseries and normal agg table
together
Root
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1794
@ravipesala Rebase done.. Please review
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1794
[CARBONDATA-2022]Fixed table alias issue in PreAggregate
**Problem:**Query with table alias is Not hitting pre Aggregate table.
**Solution:** Problem is table alias query is plan
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1724
@geetikagupta16 , please handle filter columns also . for example . select
country ,sum( case when year=2010 salary else 0 end) from tbl where year=2010
or year=2011 group by country
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1742
retest this please
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1742
retest please
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1742
[CARBONDATA-1953]Pre-aggregate Should inherit sort
column,sort_scope,dictionary encodiâ¦
Pre-aggregate should inherit the main table properties , sort column order
, sort scope ,table block
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1712
Merged
---
Github user BJangir closed the pull request at:
https://github.com/apache/carbondata/pull/1712
---
GitHub user BJangir opened a pull request:
https://github.com/apache/carbondata/pull/1712
[CARBONDATA-1931]DataLoad failed for Aggregate table when measure is â¦
Root Cause , during data loading ,dataframe columns are sorted based on
selection order but in RowConverter fileds
Github user BJangir closed the pull request at:
https://github.com/apache/carbondata/pull/1115
---
Github user BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1115
please retest
---
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 BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1115
please retest
---
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 BJangir commented on the issue:
https://github.com/apache/carbondata/pull/1115
please retest
---
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
1 - 100 of 110 matches
Mail list logo