[GitHub] incubator-carbondata pull request #790: [CARBONDATA-919]result_size in query...

2017-04-13 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/790#discussion_r111356217 --- Diff: integration/spark2/src/main/java/org/apache/carbondata/spark/vectorreader/VectorizedCarbonRecordReader.java --- @@ -147,7 +148,11

[GitHub] incubator-carbondata issue #730: [CARBONDATA-898]NullPointerException is get...

2017-04-12 Thread nareshpr
Github user nareshpr commented on the issue: https://github.com/apache/incubator-carbondata/pull/730 Updated comments --- 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] incubator-carbondata pull request #730: [WIP]NullPointerException is getting...

2017-04-05 Thread nareshpr
Github user nareshpr closed the pull request at: https://github.com/apache/incubator-carbondata/pull/730 --- 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] incubator-carbondata pull request #730: [WIP]NullPointerException is getting...

2017-04-05 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/730 [WIP]NullPointerException is getting thrown when rename table and select query is run concurrently You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-carbondata pull request #713: [WIP]add decimal column without scal...

2017-03-30 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/713 [WIP]add decimal column without scale and precision is failing. You can merge this pull request into a Git repository by running: $ git pull https://github.com/nareshpr

[GitHub] incubator-carbondata issue #681: [CARBONDATA-802]Select query is throwing ex...

2017-03-21 Thread nareshpr
Github user nareshpr commented on the issue: https://github.com/apache/incubator-carbondata/pull/681 Fixed review comments. Pls review and merge --- 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] incubator-carbondata pull request #681: [CARBONDATA-802]Select query is thro...

2017-03-21 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/681 [CARBONDATA-802]Select query is throwing exception when newly column is added without any default value If user does not provide default value in add column, dictionary & sort i

[GitHub] incubator-carbondata pull request #674: [CARBONDATA-795]rename table is chan...

2017-03-20 Thread nareshpr
GitHub user nareshpr reopened a pull request: https://github.com/apache/incubator-carbondata/pull/674 [CARBONDATA-795]rename table is changing provided db to current db When user trigger rename table command not on current database, table is getting renamed from provided db

[GitHub] incubator-carbondata issue #674: [CARBONDATA-795]rename table is changing pr...

2017-03-20 Thread nareshpr
Github user nareshpr commented on the issue: https://github.com/apache/incubator-carbondata/pull/674 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] incubator-carbondata pull request #675: [CARBONDATA-796]Drop database comman...

2017-03-20 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/675 [CARBONDATA-796]Drop database command clears all carbonfiles from carbon.store/dbfolder When user trigger drop database command, Carbon is deleted all the files from /database

[GitHub] incubator-carbondata issue #674: [CARBONDATA-795]rename table is changing pr...

2017-03-20 Thread nareshpr
Github user nareshpr commented on the issue: https://github.com/apache/incubator-carbondata/pull/674 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] incubator-carbondata pull request #674: [CARBONDATA-795]rename table is chan...

2017-03-20 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/674 [CARBONDATA-795]rename table is changing provided db to current db When user trigger rename table command not on current database, table is getting renamed from provided db

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106192438 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -313,6 +307,100

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106131227 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSpark2SqlParser.scala --- @@ -129,4 +134,80 @@ class

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106129635 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSpark2SqlParser.scala --- @@ -129,4 +134,80 @@ class

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106125641 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonMetastore.scala --- @@ -304,38 +341,76 @@ class CarbonMetastore

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106112796 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/CarbonScalaUtil.scala --- @@ -194,4 +196,102 @@ object

[GitHub] incubator-carbondata pull request #641: [CARBONDATA-767] Alter table support...

2017-03-15 Thread nareshpr
Github user nareshpr commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/641#discussion_r106112761 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/DataTypeConverterUtil.scala --- @@ -77,4 +77,40

[GitHub] incubator-carbondata pull request #558: [CARBONDATA-562] Supporting Spark 1....

2017-01-19 Thread nareshpr
GitHub user nareshpr opened a pull request: https://github.com/apache/incubator-carbondata/pull/558 [CARBONDATA-562] Supporting Spark 1.6.3 Version in CarbonData Implemented support for Spark 1.6.3 in CarbonData You can merge this pull request into a Git repository by running

[GitHub] incubator-carbondata issue #491: [CARBONDATA-583] Add replace function suppo...

2017-01-04 Thread nareshpr
Github user nareshpr commented on the issue: https://github.com/apache/incubator-carbondata/pull/491 Can you check regexp_replace spark function? This function replace all substrings of the specified string that matches the pattern --- If your project is set up for it, you can reply

[GitHub] incubator-carbondata pull request #308: [CARBONDATA-398] In DropCarbonTable ...

2016-11-22 Thread nareshpr
Github user nareshpr closed the pull request at: https://github.com/apache/incubator-carbondata/pull/308 --- 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