[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_r118310359 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/ScanningResultIterator.java --- @@ -38,10 +38,12 @@ public class

[GitHub] phoenix pull request #249: PHOENIX-3878 Add license headers missed in PHOENI...

2017-05-23 Thread bijugs
GitHub user bijugs opened a pull request: https://github.com/apache/phoenix/pull/249 PHOENIX-3878 Add license headers missed in PHOENIX-3572 Add Apache license headers missed in ``PHOENIX-3572``. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-19 Thread churrodog
Github user churrodog commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r117566388 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/AlterTableWithViewsIT.java --- @@ -107,12 +107,14 @@ public void

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread churrodog
Github user churrodog commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117542085 --- Diff: phoenix-queryserver/pom.xml --- @@ -147,6 +147,14 @@ commons-logging commons-logging + +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117437481 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancerConfiguration.java --- @@ -0,0 +1,91 @@ +/** +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117437408 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,165 @@ +/** + * + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117437437 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,165 @@ +/** + * + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117437387 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,165 @@ +/** + * + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117437336 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,165 @@ +/** + * + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117436975 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancer.java --- @@ -0,0 +1,165 @@ +/** + * + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117436956 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/exception/NoPhoenixQueryServerRegisteredException.java --- @@ -0,0 +1,15

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-19 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117436598 --- Diff: phoenix-queryserver/pom.xml --- @@ -147,6 +147,14 @@ commons-logging commons-logging + +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117405732 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/register/Registry.java --- @@ -0,0 +1,31 @@ +/** + * + * Licensed

[GitHub] phoenix pull request #236: Loadbalancer

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

[GitHub] phoenix pull request #236: Loadbalancer

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

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117391613 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/PhoenixQueryServerNode.java --- @@ -0,0 +1,79 @@ +/** + *

[GitHub] phoenix pull request #236: Loadbalancer

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

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117377275 --- Diff: pom.xml --- @@ -127,6 +128,9 @@ UTF-8 UTF-8 + --- End diff -- removed. --- If your project is

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117376467 --- Diff: phoenix-queryserver/pom.xml --- @@ -147,6 +147,14 @@ commons-logging commons-logging + +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117372183 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancerConfiguration.java --- @@ -0,0 +1,91 @@ +/** +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117372190 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancerConfiguration.java --- @@ -0,0 +1,91 @@ +/** +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-18 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r117371944 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/LoadBalancerConfiguration.java --- @@ -0,0 +1,91 @@ +/** +

[GitHub] phoenix pull request #222: PHOENIX-3355 Register Phoenix built-in functions ...

2017-05-18 Thread lomoree
Github user lomoree closed the pull request at: https://github.com/apache/phoenix/pull/222 --- 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 #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-05-17 Thread bijugs
Github user bijugs closed the pull request at: https://github.com/apache/phoenix/pull/229 --- 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 #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-17 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r117042628 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,105 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-17 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r117041326 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsFileSink.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116896889 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/QueryServicesOptions.java --- @@ -272,6 +278,11 @@ public static final String

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116895997 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsFileSink.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116874151 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,105 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116866907 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,105 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116866233 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -84,8 +114,11 @@ public static void afterClass()

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116869974 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsFileSink.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116868081 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/PhoenixMetaFactoryImpl.java --- @@ -68,7 +73,18 @@ public Meta

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116865709 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,105 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116870173 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,105 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116869793 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/QueryServicesOptions.java --- @@ -272,6 +278,11 @@ public static final String

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-16 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116866567 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -161,5 +196,45 @@ public void smokeTest() throws

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r116869437 --- Diff: phoenix-load-balancer/src/main/java/org/apache/phoenix/loadbalancer/service/PhoenixQueryServerNode.java --- @@ -0,0 +1,79 @@ +/** + *

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r116865790 --- Diff: phoenix-load-balancer/pom.xml --- @@ -0,0 +1,58 @@ + + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] phoenix pull request #236: Loadbalancer

2017-05-16 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/236#discussion_r116863959 --- Diff: phoenix-load-balancer/pom.xml --- @@ -0,0 +1,58 @@ + + + +http://maven.apache.org/POM/4.0.0; +

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116659785 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/ViewFinder.java --- @@ -0,0 +1,241 @@ +/** + * Licensed to the Apache Software

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116659570 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java --- @@ -535,19 +539,115 @@ public void

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread twdsilva
Github user twdsilva commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116659343 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/TableViewFinderResult.java --- @@ -0,0 +1,50 @@ +/** + * Licensed to the

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116626201 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java --- @@ -381,7 +383,8 @@

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116626026 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PColumnImpl.java --- @@ -159,15 +199,6 @@ public SortOrder getSortOrder() {

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116625769 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java --- @@ -2324,96 +2488,97 @@ else if (pkCount <=

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116625600 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/AlterTableWithViewsIT.java --- @@ -107,12 +107,14 @@ public void

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/248#discussion_r116625014 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/schema/PTableImpl.java --- @@ -500,8 +510,20 @@ private void init(PName tenantId, PName

[GitHub] phoenix pull request #248: Phoenix 3534

2017-05-15 Thread churrodog
GitHub user churrodog opened a pull request: https://github.com/apache/phoenix/pull/248 Phoenix 3534 I'll squash these down after a review You can merge this pull request into a Git repository by running: $ git pull https://github.com/churrodog/phoenix PHOENIX-3534

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

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r116564003 --- Diff: phoenix-core/src/test/java/org/apache/phoenix/query/BaseTest.java --- @@ -592,7 +592,7 @@ private static String

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

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

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

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r116562096 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/monitoring/MetricType.java --- @@ -25,6 +25,7 @@ QUERY_TIME("Query times"),

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

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

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

2017-05-15 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/239#discussion_r116560730 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/BaseResultIterators.java --- @@ -496,6 +496,17 @@ public

[GitHub] phoenix pull request #148: bugfix: PHOENIX-2454 Upsert with Double.NaN retur...

2017-05-14 Thread chuxi
Github user chuxi closed the pull request at: https://github.com/apache/phoenix/pull/148 --- 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 #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116331710 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsGlobalMetrics.java --- @@ -0,0 +1,76 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116330910 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsGlobalMetrics.java --- @@ -0,0 +1,76 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116330576 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsConfiguration.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116330598 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsConfiguration.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116323604 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -161,5 +184,19 @@ public void smokeTest() throws

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116309789 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/PhoenixMetaFactoryImpl.java --- @@ -68,7 +70,10 @@ public Meta

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-12 Thread rahulsIOT
Github user rahulsIOT commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r116309097 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsSlf4jSink.java --- @@ -0,0 +1,44 @@ +/* + * Licensed

[GitHub] phoenix pull request #247: PHOENIX-3825 Mutable Index rebuild does not write...

2017-05-11 Thread vincentpoon
GitHub user vincentpoon opened a pull request: https://github.com/apache/phoenix/pull/247 PHOENIX-3825 Mutable Index rebuild does not write an index version fo… …r each data row version You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
GitHub user ohadshacham reopened a pull request: https://github.com/apache/phoenix/pull/246 Connect tal to phoenix You can merge this pull request into a Git repository by running: $ git pull https://github.com/ohadshacham/phoenix ConnectTALtoPhoenix Alternatively you can

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
Github user ohadshacham closed the pull request at: https://github.com/apache/phoenix/pull/246 --- 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

[GitHub] phoenix pull request #246: Connect tal to phoenix

2017-05-09 Thread ohadshacham
GitHub user ohadshacham opened a pull request: https://github.com/apache/phoenix/pull/246 Connect tal to phoenix You can merge this pull request into a Git repository by running: $ git pull https://github.com/ohadshacham/phoenix ConnectTALtoPhoenix Alternatively you can

[GitHub] phoenix pull request #245: Harden Phoenix - distributed big link list and at...

2017-05-05 Thread anirudha
GitHub user anirudha opened a pull request: https://github.com/apache/phoenix/pull/245 Harden Phoenix - distributed big link list and atomicity tests over phoenix Adds: -a new testing utility for test with multiple threads. -a new test driver to execute tests against an

[GitHub] phoenix pull request #244: PHOENIX-3824 Mutable Index partial rebuild adds m...

2017-05-04 Thread vincentpoon
GitHub user vincentpoon opened a pull request: https://github.com/apache/phoenix/pull/244 PHOENIX-3824 Mutable Index partial rebuild adds more than one index r… …ow for updated data row You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #243: PHOENIX-3806 IndexUpdateManager spending a lot of...

2017-05-04 Thread vincentpoon
GitHub user vincentpoon opened a pull request: https://github.com/apache/phoenix/pull/243 PHOENIX-3806 IndexUpdateManager spending a lot of time sorting mutati… …ons on Index rebuild You can merge this pull request into a Git repository by running: $ git pull

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132550 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/PhoenixMetaFactoryImpl.java --- @@ -68,7 +70,10 @@ public Meta

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133315 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsGlobalMetrics.java --- @@ -0,0 +1,76 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132061 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,82 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132936 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsConfiguration.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132800 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -161,5 +184,19 @@ public void smokeTest() throws

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132118 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,82 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133063 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsConfiguration.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132345 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsSlf4jSink.java --- @@ -0,0 +1,44 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133420 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsGlobalMetrics.java --- @@ -0,0 +1,76 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133655 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsFileSink.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132207 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/sink/PqsFileSink.java --- @@ -0,0 +1,66 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132854 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -161,5 +184,19 @@ public void smokeTest() throws

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133169 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsConfiguration.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114132643 --- Diff: phoenix-queryserver/src/it/java/org/apache/phoenix/end2end/QueryServerBasicsIT.java --- @@ -57,10 +64,17 @@ private static

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/phoenix/pull/242#discussion_r114133555 --- Diff: phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/metrics/PqsMetricsSystem.java --- @@ -0,0 +1,82 @@ +/* + * Licensed

[GitHub] phoenix pull request #242: PQS metrics - https://issues.apache.org/jira/brow...

2017-05-01 Thread rahulsIOT
GitHub user rahulsIOT opened a pull request: https://github.com/apache/phoenix/pull/242 PQS metrics - https://issues.apache.org/jira/browse/PHOENIX-3655 Phoenix Query Server metrics https://issues.apache.org/jira/browse/PHOENIX-3655 The following properties is added

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

2017-04-28 Thread zinking
GitHub user zinking opened a pull request: https://github.com/apache/phoenix/pull/241 PHOENIX-3556 upgrade guava library to version 20 PHOENIX-3556 upgrade guava library to version 20 mvn test run pass You can merge this pull request into a Git repository by running: $ git

[GitHub] phoenix pull request #192: Phoenix Cursors implementation

2017-04-27 Thread anirudha
Github user anirudha closed the pull request at: https://github.com/apache/phoenix/pull/192 --- 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 #240: PHOENIX-3808 Implement chaos tests using HBase's ...

2017-04-26 Thread apurtell
GitHub user apurtell opened a pull request: https://github.com/apache/phoenix/pull/240 PHOENIX-3808 Implement chaos tests using HBase's hbase-it facility You can merge this pull request into a Git repository by running: $ git pull https://github.com/apurtell/phoenix

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-24 Thread bijugs
Github user bijugs commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r113034318 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorWithRowValueConstructorIT.java --- @@ -0,0 +1,687 @@ +/* + * Licensed to the

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

2017-04-24 Thread akshita-malhotra
GitHub user akshita-malhotra opened a pull request: https://github.com/apache/phoenix/pull/239 Phoenix-3744: Support snapshot scanners for MR-based queries - Parallel Scan grouper is extended to differentiate the functionality for getting region boundaries - Added

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-24 Thread bijugs
Github user bijugs commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r113033881 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorWithRowValueConstructorIT.java --- @@ -0,0 +1,687 @@ +/* + * Licensed to the

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-24 Thread bijugs
Github user bijugs commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r113032806 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/QueryPlan.java --- @@ -59,6 +60,8 @@ * Returns projector used to formulate

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-24 Thread bijugs
Github user bijugs commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r113032237 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/CursorResultIterator.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r110057029 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/QueryPlan.java --- @@ -59,6 +60,8 @@ * Returns projector used to formulate

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r11317 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorWithRowValueConstructorIT.java --- @@ -0,0 +1,687 @@ +/* + * Licensed to

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r10908 --- Diff: phoenix-core/src/it/java/org/apache/phoenix/end2end/CursorWithRowValueConstructorIT.java --- @@ -0,0 +1,687 @@ +/* + * Licensed to

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r10394 --- Diff: phoenix-core/src/main/antlr3/PhoenixSQL.g --- @@ -742,7 +751,26 @@ upsert_column_refs returns [Pair ret] :

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r10101 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/compile/QueryPlan.java --- @@ -59,6 +60,8 @@ * Returns projector used to formulate

[GitHub] phoenix pull request #229: PHOENIX-3572 Support FETCH NEXT|n ROWS query on c...

2017-04-05 Thread JamesRTaylor
Github user JamesRTaylor commented on a diff in the pull request: https://github.com/apache/phoenix/pull/229#discussion_r109997991 --- Diff: phoenix-core/src/main/java/org/apache/phoenix/iterate/CursorResultIterator.java --- @@ -0,0 +1,75 @@ +/* + * Licensed to the Apache

[GitHub] phoenix pull request #236: Loadbalancer

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

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