[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-12-07 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r91459367 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -167,50 +180,152 @@ private void printHelpAndExit(Options

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-12-07 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r91454910 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -85,7 +102,7 @@ private static final Option

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-12-07 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r91454867 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/index/MutableIndexFailureIT.java --- @@ -385,6 +383,8 @@ private void

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-12-07 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r91454865 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexMetadataIT.java --- @@ -216,6 +219,15 @@ public void testIndexCreateDrop()

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-11-30 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r90192895 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolForPartialBuildIT.java --- @@ -217,39 +223,47 @@ public void

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-11-30 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r90192647 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -167,50 +180,152 @@ private void printHelpAndExit(Options

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-11-30 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r90188400 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/MetaDataClient.java --- @@ -305,16 +305,27 @@ TENANT_ID + "," +

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-11-16 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r88209639 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolForPartialBuildWithNamespaceEnabled.java --- @@ -0,0 +1,71 @@ +/* + *

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83832270 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/MetaDataClient.java --- @@ -305,16 +305,27 @@ TENANT_ID + "," +

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83827883 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -167,50 +180,152 @@ private void printHelpAndExit(Options

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83827393 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -167,50 +180,152 @@ private void printHelpAndExit(Options

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83824633 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexTool.java --- @@ -85,7 +102,7 @@ private static final Option

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83795678 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexMetadataIT.java --- @@ -568,4 +580,52 @@ public void testAsyncCreatedDate()

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83795282 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexMetadataIT.java --- @@ -568,4 +580,52 @@ public void testAsyncCreatedDate()

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83795010 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/index/IndexMetadataIT.java --- @@ -216,6 +219,15 @@ public void testIndexCreateDrop()

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83793592 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/OrderByIT.java --- @@ -500,14 +500,12 @@ public void testOrderByWithExpression() throws

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83793514 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolForPartialBuildWithNamespaceEnabled.java --- @@ -0,0 +1,71 @@ +/* + *

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83792778 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolForPartialBuildIT.java --- @@ -0,0 +1,264 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #210: PHOENIX-2890 Extend IndexTool to allow incrementa...

2016-10-18 Thread chrajeshbabu
Github user chrajeshbabu commented on a diff in the pull request: https://github.com/apache/phoenix/pull/210#discussion_r83792475 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/IndexToolForPartialBuildIT.java --- @@ -0,0 +1,264 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #210: PHOENIX 2890 Extend IndexTool to allow incrementa...

2016-09-27 Thread ankitsinghal
GitHub user ankitsinghal opened a pull request: https://github.com/apache/phoenix/pull/210 PHOENIX 2890 Extend IndexTool to allow incremental index rebuilds You can merge this pull request into a Git repository by running: $ git pull https://github.com/ankitsinghal/phoenix