[GitHub] incubator-carbondata pull request #71: CARBONDATA-155 Code refactor to avoid...

2016-08-10 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/71#discussion_r74250267 --- Diff: core/src/main/java/org/carbondata/core/carbon/datastore/block/Distributable.java --- @@ -19,7 +19,7

[GitHub] incubator-carbondata pull request #56: blocklets distribution for query perf...

2016-08-09 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/56#discussion_r74184028 --- Diff: core/src/main/java/org/carbondata/core/carbon/datastore/block/TableBlockInfo.java --- @@ -185,6 +207,16 @@ public long

[GitHub] incubator-carbondata pull request #56: blocklets distribution for query perf...

2016-08-10 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/56#discussion_r74208162 --- Diff: core/src/main/java/org/carbondata/core/carbon/datastore/block/TableBlockInfo.java --- @@ -185,6 +207,16 @@ public long

[GitHub] incubator-carbondata pull request #71: CARBONDATA-155 Code refactor to avoid...

2016-08-10 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-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] incubator-carbondata pull request #77: CARBONDATA-153 Record count is not ma...

2016-08-19 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/77 CARBONDATA-153 Record count is not matching while loading the data when one data node went down in HA setup Record count is not matching while loading the data

[GitHub] incubator-carbondata pull request #87: [CARBONDATA-171] Block distribution n...

2016-08-23 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/87 [CARBONDATA-171] Block distribution not proper when the number of active executors more than the node size Secenario: suppose block size : 10 Node size = 3

[GitHub] incubator-carbondata pull request #134: [CARBONDATA-220] TimeStampDirectDict...

2016-09-07 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/134 [CARBONDATA-220] TimeStampDirectDictionaryGenerator_UT.java is not running in the build Refactored the class name and added boundary test cases You can merge this pull

[GitHub] incubator-carbondata pull request #144: [CARBONDATA-220] TimeStampDirectDict...

2016-09-08 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/144 [CARBONDATA-220] TimeStampDirectDictionaryGenerator_UT.java is not running in the build # **Why raise this pr?** Our build script does not execute the test case

[GitHub] incubator-carbondata pull request #140: [CARBONDATA-227] In block distributi...

2016-09-07 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/140 [CARBONDATA-227] In block distribution parralelism is decided initial… In block distribution parralelism is decided initially and not re initialized after requesting new

[GitHub] incubator-carbondata pull request #114: [CARBONDATA-195] Select query with A...

2016-08-31 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/114 [CARBONDATA-195] Select query with AND filter failing for empty '' operand value of numeric column Affected Query syntax: select * from Table Int_col='' and str_col

[GitHub] incubator-carbondata pull request #218: [CARBONDATA-288] In hdfs bad record ...

2016-10-08 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/218 [CARBONDATA-288] In hdfs bad record logger is failing in writing the bad records in log file **Poblem** For HDFS file system CarbonFile logFile

[GitHub] incubator-carbondata pull request #98: [CARBONDATA-183] Blocks are allocated...

2016-08-26 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/98 [CARBONDATA-183] Blocks are allocated to single node when Executors configured is based on the ip address. You can merge this pull request into a Git repository by running

[GitHub] incubator-carbondata pull request #218: [CARBONDATA-288] In hdfs bad record ...

2016-10-14 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/218#discussion_r83391617 --- Diff: processing/src/main/java/org/apache/carbondata/processing/surrogatekeysgenerator/csvbased/BadRecordslogger.java

[GitHub] incubator-carbondata pull request #218: [CARBONDATA-288] In hdfs bad record ...

2016-10-14 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/218#discussion_r83391382 --- Diff: integration/spark/src/main/java/org/apache/carbondata/spark/load/CarbonLoadModel.java --- @@ -117,9 +117,9

[GitHub] incubator-carbondata pull request #218: [CARBONDATA-288] In hdfs bad record ...

2016-10-14 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/218#discussion_r83391717 --- Diff: processing/src/main/java/org/apache/carbondata/processing/surrogatekeysgenerator/csvbased/CarbonCSVBasedSeqGenStep.java

[GitHub] incubator-carbondata pull request #237: [CARBONDATA-317] - CSV having only s...

2016-10-13 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/237 [CARBONDATA-317] - CSV having only space char is throwing NullPointer… Problem: Data loading fails if csv is having only empty chars Analysis: During data load

[GitHub] incubator-carbondata pull request #235: [CARBONDATA-316] Change BAD_RECORDS_...

2016-10-13 Thread mohammadshahidkhan
GitHub user mohammadshahidkhan opened a pull request: https://github.com/apache/incubator-carbondata/pull/235 [CARBONDATA-316] Change BAD_RECORDS_LOGGER_ACTION to BAD_RECORDS_ACTION **Poblem** the name BAD_RECORDS_LOGGER_ACTION is not related to logging the bad records, its