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

2017-08-14 Thread joshelser
Github user joshelser commented on the issue:

https://github.com/apache/phoenix/pull/236
  
Thanks, Rahul. I'll add this to my list to try to merge in so we can keep 
moving forward!


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-08-14 Thread rahulsIOT
Github user rahulsIOT commented on the issue:

https://github.com/apache/phoenix/pull/236
  
@joshelser  - I have rebased the code,  and file PHOENIX-4085 for SASL 
issue. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-08-07 Thread joshelser
Github user joshelser commented on the issue:

https://github.com/apache/phoenix/pull/236
  
Thanks for the fixes in f042ca0, @rahulsIOT.

Github is telling me that you have some conflicts that need resolving. If 
you can do that and file the JIRA case about firming up the ACL support, I'll 
try to test+merge this today before the week ramps up.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-07-24 Thread joshelser
Github user joshelser commented on the issue:

https://github.com/apache/phoenix/pull/236
  
> The LoadBalancer.getLoadBalancer().getSingleServiceLocation() throw 
exception that client needs to catch. Usually it shows that there is some issue 
with connecting to zookeeper or there is not PQS registered with the zookeeper.

Also thinking about this after the review comment about configuring the 
digest auth ACL for ZK, we'll need to get together documentation for this 
feature. This is another detail to add to the list of things to cover :)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-06-20 Thread joshelser
Github user joshelser commented on the issue:

https://github.com/apache/phoenix/pull/236
  
Ok, thanks for the pointer, Rahul.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-06-20 Thread rahulsIOT
Github user rahulsIOT commented on the issue:

https://github.com/apache/phoenix/pull/236
  
Thin client can call " 
LoadBalancer.getLoadBalancer().getSingleServiceLocation() " and get hold of 
host and port for PQS location. It can user DriverManager.getConnection to get 
connection object. The 
LoadBalancer.getLoadBalancer().getSingleServiceLocation() throw exception that 
client needs to catch. Usually it shows that there is some issue with 
connecting to zookeeper or there is not PQS registered with the zookeeper. 


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-06-06 Thread joshelser
Github user joshelser commented on the issue:

https://github.com/apache/phoenix/pull/236
  
Also, maybe I'm just not seeing it, but how does this load balancer get 
used by clients? It seems like this is only the advertisement component, and 
there is nothing for the thin client to use it. Is that intended? (in other 
words, you would do the rest later)


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---