[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-07-24 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r129068605 --- Diff: phoenix-load-balancer/pom.xml --- @@ -0,0 +1,84 @@ + + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] phoenix pull request #268: PHOENIX-4010 Hash Join cache may not be send to a...

2017-07-21 Thread ankitsinghal
GitHub user ankitsinghal opened a pull request: https://github.com/apache/phoenix/pull/268 PHOENIX-4010 Hash Join cache may not be send to all regionservers whe… …n we have stale HBase meta cache You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #241: PHOENIX-3556 upgrade guava library to version 20

2017-07-18 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/241#discussion_r128055418 --- Diff: pom.xml --- @@ -84,7 +84,7 @@ 1.2 1.0 1.2.0 -13.0.1 +20.0 --- End diff -- Can you revert

[GitHub] phoenix pull request #241: PHOENIX-3556 upgrade guava library to version 20

2017-07-18 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/241#discussion_r128054952 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -523,7 +523,7 @@ public void close() throws

[GitHub] phoenix pull request #241: PHOENIX-3556 upgrade guava library to version 20

2017-07-18 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/241#discussion_r128054860 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -420,7 +420,7 @@ private void initTxServiceClient()

[GitHub] phoenix pull request #267: 4.10 h base 1.2

2017-07-17 Thread kaisenc
GitHub user kaisenc opened a pull request: https://github.com/apache/phoenix/pull/267 4.10 h base 1.2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/morningzhang/phoenix 4.10-HBase-1.2 Alternatively you can review and apply

[GitHub] phoenix pull request #266: PHOENIX-3769 - Fix OnDuplicateKeyIT failure on pp...

2017-07-14 Thread snehakpersistent
GitHub user snehakpersistent opened a pull request: https://github.com/apache/phoenix/pull/266 PHOENIX-3769 - Fix OnDuplicateKeyIT failure on ppc64le You can merge this pull request into a Git repository by running: $ git pull https://github.com/snehakpersistent/phoenix

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-07-12 Thread Wancy
Github user Wancy closed the pull request at: https://github.com/apache/phoenix/pull/265 --- 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, or if the feature is

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126833346 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/QueryWithTableSampleIT.java --- @@ -0,0 +1,261 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126833276 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/QueryWithTableSampleIT.java --- @@ -0,0 +1,261 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126833016 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/parse/SelectStatement.java --- @@ -267,6 +267,12 @@ public LimitNode getLimit() { }

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread aertoria
Github user aertoria commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126782179 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/QueryCompiler.java --- @@ -539,6 +539,7 @@ protected QueryPlan

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread aertoria
Github user aertoria commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126779607 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/QueryWithTableSampleIT.java --- @@ -0,0 +1,261 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126779550 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/parse/SelectStatement.java --- @@ -267,6 +267,12 @@ public LimitNode getLimit() { }

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126774166 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/QueryCompiler.java --- @@ -539,6 +539,7 @@ protected QueryPlan

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-07-11 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/262#discussion_r126773527 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/QueryWithTableSampleIT.java --- @@ -0,0 +1,261 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-07-07 Thread brfrn169
Github user brfrn169 closed the pull request at: https://github.com/apache/phoenix/pull/261 --- 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, or if the feature is

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread Wancy
Github user Wancy commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124413926 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -228,7 +235,9 @@ public int run(String[] args)

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread Wancy
Github user Wancy commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124413815 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -273,6 +282,54 @@ public int run(String[] args)

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124409282 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -228,7 +235,9 @@ public int run(String[] args)

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124409157 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -273,6 +282,54 @@ public int run(String[]

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124402049 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -273,6 +282,54 @@ public int run(String[]

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-27 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124312271 --- Diff: phoenix-queryserver/src/test/java/org/apache/phoenix/queryserver/server/PhoenixRemoteUserExtractorTest.java --- @@ -0,0 +1,102 @@ +/*

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread Wancy
Github user Wancy commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124153233 --- Diff: phoenix-queryserver/src/test/java/org/apache/phoenix/queryserver/server/PhoenixRemoteUserExtractorTest.java --- @@ -0,0 +1,102 @@ +/* + *

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124113023 --- Diff: phoenix-queryserver/src/test/java/org/apache/phoenix/queryserver/server/PhoenixRemoteUserExtractorTest.java --- @@ -0,0 +1,102 @@ +/*

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124112286 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -274,6 +282,47 @@ public int run(String[]

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124110879 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -274,6 +282,47 @@ public int run(String[]

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124110721 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -274,6 +282,47 @@ public int run(String[]

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/265#discussion_r124113153 --- Diff: phoenix-queryserver/pom.xml --- @@ -147,6 +147,10 @@ commons-logging commons-logging + +

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-26 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123960912 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/util/SchemaUtil.java --- @@ -983,14 +983,21 @@ public static boolean hasRowTimestampColumn(PTable

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-26 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123958036 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java --- @@ -1184,8 +1184,11 @@ public synchronized boolean

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-26 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123956939 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/FromCompiler.java --- @@ -184,9 +184,9 @@ public static ColumnResolver

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-26 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123955685 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/CreateTableIT.java --- @@ -708,4 +705,52 @@ public void

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-26 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123954850 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/util/SchemaUtil.java --- @@ -983,14 +983,21 @@ public static boolean hasRowTimestampColumn(PTable

[GitHub] phoenix pull request #265: PHOENIX-3598

2017-06-26 Thread Wancy
GitHub user Wancy opened a pull request: https://github.com/apache/phoenix/pull/265 PHOENIX-3598 Add two params "phoenix.queryserver.doAs.enabled" and "phoenix.queryserver.doAs.param" to control whether to get enduser from request parameters and what is the parameter key name.

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-23 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123726523 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -1420,7 +1420,7 @@ public

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-23 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123725378 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/FromCompiler.java --- @@ -196,7 +196,8 @@ public static ColumnResolver

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-22 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123495453 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/FromCompiler.java --- @@ -196,7 +196,8 @@ public static ColumnResolver

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-22 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123495103 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/util/SchemaUtil.java --- @@ -1016,7 +1017,8 @@ public static TableName getPhysicalTableName(byte[]

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-22 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123493994 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java --- @@ -1420,7 +1420,7 @@ public MetaDataMutationResult

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22676582 In phoenix-core/src/main/java/org/apache/phoenix/util/CursorUtil.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22676567 In phoenix-core/src/main/java/org/apache/phoenix/iterate/CursorResultIterator.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22676426 In phoenix-core/src/main/java/org/apache/phoenix/iterate/CursorResultIterator.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22676177 In phoenix-core/src/main/java/org/apache/phoenix/iterate/CursorResultIterator.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22674920 In phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorFetchPriorIT.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22674824 In phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorFetchPriorIT.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/c5644c521546efa69b10cf7719fa1c34d2aa12a9#commitcomment-22674761 In phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorFetchPriorIT.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22672590 In phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorFetchPriorIT.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22672273 In phoenix-core/src/main/java/org/apache/phoenix/execute/CursorFetchPlan.java: In

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22672267 In phoenix-core/src/main/antlr3/PhoenixSQL.g: In phoenix-core/src/main/antlr3/PhoenixSQL.g on line 774: can't we use

[GitHub] phoenix pull request #:

2017-06-21 Thread ankitsinghal
Github user ankitsinghal commented on the pull request: https://github.com/apache/phoenix/commit/26d7657f1c2b9921dcbd9c22ccf7d75e8ccf5bb7#commitcomment-22672204 In phoenix-core/src/main/java/org/apache/phoenix/util/CursorUtil.java: In

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-20 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123153895 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/ViewIT.java --- @@ -710,6 +714,27 @@ public void testCreateViewWithUpdateCacheFrquency()

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-20 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123154332 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/util/SchemaUtil.java --- @@ -302,6 +302,10 @@ public static String getTableName(byte[]

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-20 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123154290 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/MetaDataClient.java --- @@ -2135,7 +2135,7 @@ private PTable

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-20 Thread ankitsinghal
Github user ankitsinghal commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r123154068 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/FromCompiler.java --- @@ -196,7 +196,8 @@ public static ColumnResolver

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-20 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r123116803 --- Diff: phoenix-queryserver/src/it/resources/log4j.properties --- @@ -1,63 +0,0 @@ -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-20 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r123116460 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,170 @@ +/** + * + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-20 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r123116500 --- Diff: phoenix-queryserver/src/it/resources/log4j.properties --- @@ -1,63 +0,0 @@ -# Licensed to the Apache Software Foundation (ASF) under one

[GitHub] phoenix pull request #264: PHOENIX-3961 Should delete `tableOutputPath` in `...

2017-06-20 Thread asdf2014
GitHub user asdf2014 opened a pull request: https://github.com/apache/phoenix/pull/264 PHOENIX-3961 Should delete `tableOutputPath` in `completebulkload` For solving the situation that too many `.tmp` files in /tmp directory when processing a huge `bulkload` job, we should consider

[GitHub] phoenix pull request #263: PHOENIX-3817 Verify replication tool

2017-06-19 Thread aaraujo
GitHub user aaraujo opened a pull request: https://github.com/apache/phoenix/pull/263 PHOENIX-3817 Verify replication tool MapReduce tool that compares data accross tables. SQL conditions may be used to optionally compare only a subset of the tables. You can merge this pull

[GitHub] phoenix pull request #262: PHOENIX 153 implement TABLESAMPLE clause

2017-06-16 Thread aertoria
GitHub user aertoria opened a pull request: https://github.com/apache/phoenix/pull/262 PHOENIX 153 implement TABLESAMPLE clause table sampling on each Table basis (at the 'FROM' part of the query). Sample size decided by the input sampling rate applies on Primary Key's frequency.

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122508226 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122505939 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-15 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122323098 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-15 Thread brfrn169
Github user brfrn169 commented on a diff in the pull request: https://github.com/apache/phoenix/pull/261#discussion_r122169866 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/FromCompiler.java --- @@ -196,7 +196,8 @@ public static ColumnResolver

[GitHub] phoenix pull request #261: PHOENIX-3944 ReadOnlyTableException occurs when w...

2017-06-15 Thread brfrn169
GitHub user brfrn169 opened a pull request: https://github.com/apache/phoenix/pull/261 PHOENIX-3944 ReadOnlyTableException occurs when we map Phoenix view t… …o an existing HBase table with Namespace Mapping enabled You can merge this pull request into a Git repository by

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122086038 --- Diff: pom.xml --- @@ -953,6 +961,26 @@ joda-time ${jodatime.version} + +org.apache.curator

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122086056 --- Diff: pom.xml --- @@ -953,6 +961,26 @@ joda-time ${jodatime.version} + +org.apache.curator

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122076328 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122059868 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122060006 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122048729 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConfImpl.java --- @@ -0,0 +1,97 @@ +/** + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122048516 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConfImpl.java --- @@ -0,0 +1,97 @@ +/** + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-14 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r122046406 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConfImpl.java --- @@ -0,0 +1,97 @@ +/** + *

[GitHub] phoenix pull request #260: PHOENIX-3812: Use HBase snapshots in async index ...

2017-06-09 Thread akshita-malhotra
GitHub user akshita-malhotra opened a pull request: https://github.com/apache/phoenix/pull/260 PHOENIX-3812: Use HBase snapshots in async index building M/R job - Index tool creates a snapshot and uses it as a configuration parameter to run index M/R job using HBase snapshot. -

[GitHub] phoenix pull request #257: PHOENIX-3917 Change to calculate query projection...

2017-06-09 Thread bijugs
Github user bijugs closed the pull request at: https://github.com/apache/phoenix/pull/257 --- 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, or if the feature is

[GitHub] phoenix pull request #259: fixed setting of jetty resource base directory in...

2017-06-09 Thread dizzy2
GitHub user dizzy2 opened a pull request: https://github.com/apache/phoenix/pull/259 fixed setting of jetty resource base directory in phoenix-tracing-web… fixed obtaining of resource base for embedded jetty in tracing web appp - original implementation of obtaining webapp root

[GitHub] phoenix pull request #258: PHOENIX-3921 Change the condition checking in Sca...

2017-06-08 Thread bijugs
GitHub user bijugs opened a pull request: https://github.com/apache/phoenix/pull/258 PHOENIX-3921 Change the condition checking in ScanUtil#isReversed The current logic will return ``isReversed`` as ``true`` whether the ``BaseScannerRegionObserver.REVERSE_SCAN`` attribute is set to

[GitHub] phoenix pull request #257: PHOENIX-3917 Change to calculate query projection...

2017-06-07 Thread bijugs
GitHub user bijugs opened a pull request: https://github.com/apache/phoenix/pull/257 PHOENIX-3917 Change to calculate query projection size Change to perform the size estimation of query row projector after all the conditions to add ``column families`` to the query ``scan`` object

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120396712 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConf.java --- @@ -0,0 +1,42 @@ +/** + * + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120395606 --- Diff: pom.xml --- @@ -953,6 +961,26 @@ joda-time ${jodatime.version} + +org.apache.curator

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120395552 --- Diff: pom.xml --- @@ -953,6 +961,26 @@ joda-time ${jodatime.version} + +org.apache.curator

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120392825 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConfImpl.java --- @@ -0,0 +1,97 @@ +/** + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120394867 --- Diff: phoenix-queryserver/pom.xml --- @@ -147,6 +147,10 @@ commons-logging commons-logging + +

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120392963 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/loadbalancer/service/PqsZookeeperConfImpl.java --- @@ -0,0 +1,97 @@ +/** + *

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120394548 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/QueryServer.java --- @@ -233,16 +240,29 @@ public int run(String[]

[GitHub] phoenix pull request #236: PHOENIX-3654 Load Balancer for thin client

2017-06-06 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r120393196 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/register/ZookeeperRegistry.java --- @@ -0,0 +1,72 @@ +/** + * + *

[GitHub] phoenix pull request #256: PHOENIX-3477 patch for 4.x-HBase-1.1

2017-06-05 Thread akshita-malhotra
GitHub user akshita-malhotra opened a pull request: https://github.com/apache/phoenix/pull/256 PHOENIX-3477 patch for 4.x-HBase-1.1 PHOENIX-3477 patch for 4.x-HBase-1.1 You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #255: PHOENIX-3744 for 4.x-HBase-0.98

2017-06-05 Thread akshita-malhotra
GitHub user akshita-malhotra opened a pull request: https://github.com/apache/phoenix/pull/255 PHOENIX-3744 for 4.x-HBase-0.98 PHOENIX-3744 patch for 4.x-HBase-0.98 branch You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #254: PHOENIX-3903 Generate empty javadoc jar to comply...

2017-06-01 Thread ssanthanam-sfdc
GitHub user ssanthanam-sfdc opened a pull request: https://github.com/apache/phoenix/pull/254 PHOENIX-3903 Generate empty javadoc jar to comply with Maven central You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #253: PHOENIX-3903 Generate empty javadoc jar to comply...

2017-06-01 Thread ssanthanam-sfdc
GitHub user ssanthanam-sfdc opened a pull request: https://github.com/apache/phoenix/pull/253 PHOENIX-3903 Generate empty javadoc jar to comply with Maven central You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #252: Generate empty javadoc jar to comply with Maven c...

2017-06-01 Thread ssanthanam-sfdc
GitHub user ssanthanam-sfdc opened a pull request: https://github.com/apache/phoenix/pull/252 Generate empty javadoc jar to comply with Maven central You can merge this pull request into a Git repository by running: $ git pull https://github.com/ssanthanam-sfdc/phoenix

[GitHub] phoenix pull request #251: Generate empty javadoc jar to comply with Maven C...

2017-06-01 Thread ssanthanam-sfdc
GitHub user ssanthanam-sfdc opened a pull request: https://github.com/apache/phoenix/pull/251 Generate empty javadoc jar to comply with Maven Central You can merge this pull request into a Git repository by running: $ git pull https://github.com/ssanthanam-sfdc/phoenix

[GitHub] phoenix pull request #250: Generate empty javadoc jar to comply with Maven C...

2017-05-30 Thread ssanthanam-sfdc
GitHub user ssanthanam-sfdc opened a pull request: https://github.com/apache/phoenix/pull/250 Generate empty javadoc jar to comply with Maven Central Im guessing I need to add same change to the following repositories 4.10-HBase-1.1, 4.10-HBase-1.2, 4.x-HBase-0.98,

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118314856 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/RegionContext.java --- @@ -0,0 +1,15 @@ +package org.apache.phoenix.iterate; +

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118314328 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/util/PhoenixConfigurationUtil.java --- @@ -192,6 +196,18 @@ public static void

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118313746 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/PhoenixInputFormat.java --- @@ -185,7 +184,7 @@ private QueryPlan

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118313368 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/TableMRParallelScanGrouper.java --- @@ -0,0 +1,99 @@ +package

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118312938 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/DefaultParallelScanGrouper.java --- @@ -17,46 +17,79 @@ */ package

[GitHub] phoenix pull request #239: PHOENIX-3744: Support snapshot scanners for MR-ba...

2017-05-24 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r118311801 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/mapreduce/util/PhoenixMapReduceUtil.java --- @@ -63,6 +67,66 @@ public static void

<    2   3   4   5   6   7   8   9   10   11   >