Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2588#discussion_r208199760
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/sort/unsafe/merger/UnsafeIntermediateMerger.java
---
@@ -111,18
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2588#discussion_r208199601
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/sort/unsafe/merger/UnsafeIntermediateMerger.java
---
@@ -111,18
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2588#discussion_r207162716
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/sort/unsafe/merger/UnsafeIntermediateMerger.java
---
@@ -111,18
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2571#discussion_r206525134
--- Diff:
integration/spark2/src/main/scala/org/apache/carbondata/spark/util/CarbonSparkUtil.scala
---
@@ -87,18 +88,20 @@ object
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2571#discussion_r206503427
--- Diff:
integration/spark2/src/main/scala/org/apache/carbondata/spark/util/CarbonSparkUtil.scala
---
@@ -87,18 +88,20 @@ object
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2571#discussion_r206425523
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/createTable/TestCreateExternalTable.scala
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2571
retest sdv please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2517#discussion_r205752516
--- Diff:
core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentUpdateStatusManager.java
---
@@ -724,6 +724,7 @@ public void
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2553
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2571
[CARBONDATA-2792][schema restructure] Create external table fails post
schema restructure.
â¦
Be sure to do all of the following checklist to help us incorporate
your
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2517
retest sdv please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2553
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2553
[HOTFIX] Fixed random test failure
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [X] Any interfaces
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2517#discussion_r204685081
--- Diff:
core/src/main/java/org/apache/carbondata/core/fileoperations/AtomicFileOperationsImpl.java
---
@@ -70,12 +78,20 @@ public
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
Failled test case for the below build is unrelated and random.
Checked locally its passing.
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/6136/
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2517
retest sdv please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2517#discussion_r203957912
--- Diff:
core/src/main/java/org/apache/carbondata/core/fileoperations/AtomicFileOperationsImpl.java
---
@@ -70,12 +78,20 @@ public
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2517
The failled test case is unrelated and random.
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/5903/
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
Failled test case for the below build is unrelated. The same testcase is
passing locally.
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/7258/
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2517
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2466
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2466
Failled test case for the below build is unrelated. The same testcase is
passing locally.
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/7247/
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2517
[CARBONDATA-2749][dataload] In HDFS Empty tablestatus file is written
during datalaod, iud or compaction when disk is full.
â¦
Be sure to do all of the following
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2466
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2498
[CARBONDATA-2729][file-format] Schema Compatibility problem between version
1.3.0 and 1.4.0
**Problem**:
In TableSchema the field Name schemaEvaluation is changed
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2466
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2466
[WIP][CARBONDATA-2710][Spark Integration] Refactor CarbonSparkSqlParser for
better code reuse.
â¦
Be sure to do all of the following checklist to help us incorporate
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
retest this please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2429#discussion_r199685424
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java
---
@@ -341,21 +343,32 @@ public CarbonTable
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2429#discussion_r199685364
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java
---
@@ -341,21 +343,32 @@ public CarbonTable
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2429#discussion_r199685348
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java
---
@@ -341,21 +343,32 @@ public CarbonTable
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2387
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2429
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2429
[CARBONDATA-2674][Streaming]Streaming with merge index enabled does not
consider the merge index file while pruning.
â¦
Be sure to do all of the following checklist
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2389#discussion_r197026107
--- Diff:
integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala
---
@@ -272,6 +284,13 @@ class
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2389#discussion_r197025960
--- Diff:
integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonTableCompactor.scala
---
@@ -272,6 +284,13 @@ class
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2387#discussion_r197014465
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java ---
@@ -1530,6 +1531,22 @@ public String
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2389
[CARBONDATA-2623][DataMap] Add DataMap Pre and Pevent listener
Added Pre and Post Execution Events for index datamap
- [X] Any interfaces changed?
None
- [X] Any
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2387
[CARBONDATA-2621][BloomDataMap] Lock problem in index datamap
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1942
LGTM
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2014
LGTM
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2263
[WIP] Refactored SegmentPropertiesFetcher to resolve pruning problem post
the carbon schema restructure.
â¦
Be sure to do all of the following checklist to help us
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2243
Failled test is unrelated
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2243
restricted parsed data display in the exception using
parserSettings.setErrorContentLength(0)
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2232#discussion_r185411161
--- Diff:
integration/spark-common/src/main/java/org/apache/carbondata/spark/dictionary/server/SecureDictionaryServer.java
---
@@ -63,7
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2243
retest this please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2243
[CARBONDATA-2410] Error message correction when column value length exceeds
32 character
The raw is displayed in the client, to avoid the same the exception message
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2232
[CARBONDATA-2406][dataload]-Dictionary Server and Dictionary Client MD5
Validation failed with hive.server2.enable.doAs = true
With conf hive.server2.enable.doAs = true
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/2118
---
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/2117
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2118
[CARBONDATA-2289] If carbon merge index is enabled then after IUD opeâ¦
â¦ration if some blocks of a segment is deleted, then during query and IUD
operation the driver is throwing
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2117
[CARBONDATA-2285] Spark integration code refactor
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [X] Any
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/71
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2104#discussion_r178051841
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/ICarbonSessionCatalog.scala
---
@@ -0,0 +1,67
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/2110
retest SDV please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2110
[CARBONDATA-2289] If carbon merge index is enabled then after IUD opeâ¦
â¦ration if some blocks of a segment is deleted, then during query and IUD
operation the driver is throwing
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2104#discussion_r177426177
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/ICarbonSessionCatalog.scala
---
@@ -0,0 +1,59
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2104#discussion_r177423796
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/ICarbonSessionCatalog.scala
---
@@ -0,0 +1,59
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2104#discussion_r177416365
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1297,6 +1297,12 @@
public
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/2104
[CARBONDATA-2285] Spark integration code refactor
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [X] Any
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1942
+1 The streaming issue is handled in PR #2014
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1942
@geetikagupta16
Please handle the same issue in case of Stream Record writer as well.
Currently there no test case for the bucketing flow to validate the issue
handled
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1942#discussion_r168394055
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/DataConverterProcessorStepImpl.java
---
@@ -100,10
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1942#discussion_r168395269
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/DataConverterProcessorStepImpl.java
---
@@ -100,10
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1831
retest this please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165810447
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1264,18 +1231,7
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1831
retest SDV please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1104#discussion_r165763372
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableOutputFormat.java
---
@@ -307,17 +307,11 @@ public static
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/71
---
GitHub user mohammadshahidkhan reopened a pull request:
https://github.com/apache/carbondata/pull/71
[CARBONDATA-155] Code refactor to avoid the Type Casting in compareTo method
â¦
You can merge this pull request into a Git repository by running:
$ git pull https
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165743860
--- Diff: conf/carbon.properties.template ---
@@ -17,29 +17,25 @@
#
System Configuration
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165743651
--- Diff: conf/carbon.properties.template ---
@@ -76,22 +72,16 @@ carbon.enable.quick.filter=false
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165743745
--- Diff: docs/configuration-parameters.md ---
@@ -32,10 +32,10 @@ This section provides the details of all the
configurations required
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165742856
--- Diff: conf/carbon.properties.template ---
@@ -110,7 +100,7 @@ carbon.enable.quick.filter=false
##Percentage to identify whether
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1831#discussion_r165741342
--- Diff: conf/carbon.properties.template ---
@@ -76,22 +72,16 @@ carbon.enable.quick.filter=false
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1834
closing this PR as per @ravipesala this fix will be handled for all the
I/O operation
together with S3 support implementation
---
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/1834
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1822
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1822
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1822
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1831
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1834
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1834
Failed test case is random
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest SDV please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1831
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1834
retest SDV please
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1841#discussion_r163154276
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/CommonUtil.scala
---
@@ -941,4 +943,39 @@ object
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1104
retest this please
---
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1828
retest sdv please
---
GitHub user mohammadshahidkhan opened a pull request:
https://github.com/apache/carbondata/pull/1834
[CARBONDATA-2056] Hadoop Configuration with access key and secret key
should be passed while creating InputStream of distributed carbon file.
â¦
Be sure to do all
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1828#discussion_r162357297
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/util/CleanFiles.scala ---
@@ -33,15 +33,19 @@ object CleanFiles
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1828#discussion_r162357455
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/api/CarbonStore.scala
---
@@ -98,34 +98,44 @@ object CarbonStore
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1828#discussion_r162357211
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/CarbonCleanFilesCommand.scala
---
@@ -70,11
1 - 100 of 298 matches
Mail list logo
