[GitHub] carbondata issue #2991: [CARBONDATA-3043] Add build script and add test case...

2019-01-11 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2991 retest this please ---

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-01 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-01 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-29 Thread BJangir
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] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2018-12-25 Thread BJangir
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] carbondata pull request #2897: [CARBONDATA-3080] Supporting local dictionary...

2018-12-24 Thread BJangir
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] carbondata pull request #2897: [CARBONDATA-3080] Supporting local dictionary...

2018-12-24 Thread BJangir
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] carbondata pull request #2987: [CARBONDATA-3167] Add a example for DataFrame...

2018-12-19 Thread BJangir
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] carbondata pull request #2897: [CARBONDATA-3080] Supporting local dictionary...

2018-12-18 Thread BJangir
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] carbondata pull request #2991: [WIP] Add build script and add test case with...

2018-12-16 Thread BJangir
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] carbondata pull request #2991: [WIP] Add build script and add test case with...

2018-12-16 Thread BJangir
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] carbondata pull request #2991: [WIP] Add build script and add test case with...

2018-12-15 Thread BJangir
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] carbondata pull request #2944: [CARBONDATA-3122]CarbonReader memory leak

2018-11-23 Thread BJangir
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] carbondata pull request #2944: [CARBONDATA-3122]CarbonReader memory leak

2018-11-22 Thread BJangir
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] carbondata issue #2935: [HOTFIX] Intializing the CSDK object reference to NU...

2018-11-20 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2935 LGTM ---

[GitHub] carbondata pull request #2897: [WIP] Supporting local dictionary enable by d...

2018-11-05 Thread BJangir
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] carbondata pull request #2897: [WIP] Supporting local dictionary enable by d...

2018-11-05 Thread BJangir
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] carbondata pull request #2794: [CARBONDATA-2985]Fix issues in Table level co...

2018-10-04 Thread BJangir
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] carbondata pull request #2796: [CARBONDATA-2991]NegativeArraySizeException d...

2018-10-03 Thread BJangir
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] carbondata pull request #2794: [CARBONDATA-2985]Fix issues in Table level co...

2018-10-01 Thread BJangir
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] carbondata pull request #2785: [CARBONDATA-2986] Table Properties are lost w...

2018-09-28 Thread BJangir
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] carbondata pull request #2703: [CARBONDATA-2925]Wrong data displayed for spa...

2018-09-12 Thread BJangir
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] carbondata pull request #2703: [CARBONDATA-2925]Wrong data displayed for spa...

2018-09-10 Thread BJangir
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] carbondata pull request #2658: [Carbondata 2885]Broadcast Issue and Small fi...

2018-08-24 Thread BJangir
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] carbondata pull request #2606: [CARBONDATA-2817]Thread Leak in Update and in...

2018-08-08 Thread BJangir
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] carbondata pull request #2606: [CARBONDATA-2817]Thread Leak in Update and in...

2018-08-02 Thread BJangir
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] carbondata pull request #2542: [CARBONDATA-2772] Size based dictionary fallb...

2018-07-23 Thread BJangir
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] carbondata pull request #2508: [CARBONDATA-2744]Streaming lock is not releas...

2018-07-15 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-10 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-10 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2433: [CARBONDATA-2676]Support local Dictionary for...

2018-07-07 Thread BJangir
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] carbondata pull request #2317: [CARBONDATA-2492]Fixed thread leak issue in c...

2018-05-18 Thread BJangir
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] carbondata pull request #2227: [CARBONDATA-2401] Date and Timestamp options ...

2018-05-09 Thread BJangir
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] carbondata issue #2227: [CARBONDATA-2401] Date and Timestamp options are not...

2018-05-02 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2227 Retest this please ---

[GitHub] carbondata issue #2227: [CARBONDATA-2401] Date and Timestamp options are not...

2018-04-30 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2227 retest this please ---

[GitHub] carbondata issue #2224: [CARBONDATA-2393]TaskNo is not working for SDK

2018-04-30 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2224 retest this please ---

[GitHub] carbondata issue #2224: [CARBONDATA-2393]TaskNo is not working for SDK

2018-04-30 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2224 retest this please ---

[GitHub] carbondata issue #2227: [CARBONDATA-2401] Date and Timestamp options are not...

2018-04-30 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2227 retest this please ---

[GitHub] carbondata pull request #2245: [CARBONDATA-2411] infinite loop when sdk writ...

2018-04-30 Thread BJangir
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] carbondata pull request #2245: [CARBONDATA-2411] infinite loop when sdk writ...

2018-04-27 Thread BJangir
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] carbondata issue #2227: [CARBONDATA-2401] Date and Timestamp options are not...

2018-04-26 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2227 retest this please ---

[GitHub] carbondata issue #2097: [CARBONDATA-2275]Query Failed for 0 byte deletedelta...

2018-04-26 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2097 already merged in master PR #2121 ---

[GitHub] carbondata pull request #2097: [CARBONDATA-2275]Query Failed for 0 byte dele...

2018-04-26 Thread BJangir
Github user BJangir closed the pull request at: https://github.com/apache/carbondata/pull/2097 ---

[GitHub] carbondata issue #2121: [CARBONDATA-2275]Query Failed for 0 byte deletedelta...

2018-04-25 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2121 retest sdv please ---

[GitHub] carbondata pull request #2227: [CARBONDATA-2401] Date and Timestamp options ...

2018-04-25 Thread BJangir
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] carbondata issue #2224: [CARBONDATA-2393]TaskNo is not working for SDK

2018-04-24 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2224 retest please ---

[GitHub] carbondata pull request #2224: [CARBONDATA-2393]TaskNo is not working for SD...

2018-04-24 Thread BJangir
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] carbondata pull request #2224: [CARBONDATA-2393]TaskNo is not working for SD...

2018-04-24 Thread BJangir
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] carbondata pull request #2121: [CARBONDATA-2275]Query Failed for 0 byte dele...

2018-03-30 Thread BJangir
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] carbondata issue #2097: [CARBONDATA-2275]Query Failed for 0 byte deletedelta...

2018-03-29 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2097 retest this please ---

[GitHub] carbondata issue #2097: [CARBONDATA-2275]Query Failed for 0 byte deletedelta...

2018-03-26 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2097 retest this please ---

[GitHub] carbondata issue #2097: [CARBONDATA-2275]Query Failed for 0 byte deletedelta...

2018-03-25 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/2097 retest this please ---

[GitHub] carbondata pull request #2097: [CARBONDATA-2275]Query Failed for 0 byte dele...

2018-03-25 Thread BJangir
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] carbondata pull request #2097: [CARBONDATA-2275]Query Failed for 0 byte dele...

2018-03-25 Thread BJangir
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] carbondata pull request #2097: [CARBONDATA-2275]Query Failed for 0 byte dele...

2018-03-23 Thread BJangir
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] carbondata pull request #2075: [CARBONDATA-2261] Support Set segment command...

2018-03-18 Thread BJangir
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] carbondata pull request #1985: [CARBONDATA-2185]add InputMetrics for Streami...

2018-02-19 Thread BJangir
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] carbondata issue #1971: [CARBONDATA-2161]Compacted Segment of Streaming Tabl...

2018-02-13 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1971 retest sdv please ---

[GitHub] carbondata pull request #1971: [CARBONDATA-2161]Compacted Segment of Streami...

2018-02-11 Thread BJangir
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] carbondata pull request #1915: [CARBONDATA-1454]false expression handling an...

2018-02-02 Thread BJangir
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] carbondata pull request #1878: [CARBONDATA-2094] Filter DataMap Tables in Sh...

2018-02-01 Thread BJangir
Github user BJangir closed the pull request at: https://github.com/apache/carbondata/pull/1878 ---

[GitHub] carbondata pull request #1878: [CARBONDATA-2094] Filter DataMap Tables in Sh...

2018-01-30 Thread BJangir
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] carbondata pull request #1878: [CARBONDATA-2094] Filter DataMap Tables in Sh...

2018-01-29 Thread BJangir
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] carbondata issue #1861: [CARBONDATA-2078][CARBONDATA-1516] Add 'if not exist...

2018-01-25 Thread BJangir
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] carbondata pull request #1824: [CARBONDATA-2046]agg Query failed when non su...

2018-01-17 Thread BJangir
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] carbondata issue #1807: [CARBONDATA-2030]avg with Aggregate table for double...

2018-01-16 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1807 retest this please ---

[GitHub] carbondata issue #1807: [CARBONDATA-2030]avg with Aggregate table for double...

2018-01-16 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1807 retest this please ---

[GitHub] carbondata pull request #1807: [CARBONDATA-2030]avg with Aggregate table for...

2018-01-15 Thread BJangir
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] carbondata pull request #1804: [CARBONDATA-2028] Select Query failed with pr...

2018-01-15 Thread BJangir
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] carbondata issue #1794: [CARBONDATA-2022]Fixed table alias issue in PreAggre...

2018-01-12 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1794 @ravipesala Rebase done.. Please review ---

[GitHub] carbondata pull request #1794: [CARBONDATA-2022]Fixed table alias issue in P...

2018-01-11 Thread BJangir
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] carbondata issue #1724: [CARBONDATA-1940][PreAgg] Fixed bug for creation of ...

2017-12-30 Thread BJangir
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] carbondata issue #1742: [CARBONDATA-1953]Pre-aggregate Should inherit sort c...

2017-12-29 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1742 retest this please ---

[GitHub] carbondata issue #1742: [CARBONDATA-1953]Pre-aggregate Should inherit sort c...

2017-12-29 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1742 retest please ---

[GitHub] carbondata pull request #1742: [CARBONDATA-1953]Pre-aggregate Should inherit...

2017-12-29 Thread BJangir
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] carbondata issue #1712: [CARBONDATA-1931]DataLoad failed for Aggregate table...

2017-12-26 Thread BJangir
Github user BJangir commented on the issue: https://github.com/apache/carbondata/pull/1712 Merged ---

[GitHub] carbondata pull request #1712: [CARBONDATA-1931]DataLoad failed for Aggregat...

2017-12-26 Thread BJangir
Github user BJangir closed the pull request at: https://github.com/apache/carbondata/pull/1712 ---

[GitHub] carbondata pull request #1712: [CARBONDATA-1931]DataLoad failed for Aggregat...

2017-12-22 Thread BJangir
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] carbondata pull request #1115: [CARBONDATA-1247]Block pruning not working fo...

2017-12-13 Thread BJangir
Github user BJangir closed the pull request at: https://github.com/apache/carbondata/pull/1115 ---

[GitHub] carbondata issue #1115: [CARBONDATA-1247]Block pruning not working for date ...

2017-07-11 Thread BJangir
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] carbondata issue #1115: [CARBONDATA-1247]Block pruning not working for date ...

2017-07-10 Thread BJangir
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] carbondata issue #1115: [CARBONDATA-1247]Block pruning not working for date ...

2017-07-09 Thread BJangir
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   2   >