[GitHub] drill issue #1042: DRILL-5261: Expose REST endpoint in zookeeper

2018-05-01 Thread xhochy
Github user xhochy commented on the issue: https://github.com/apache/drill/pull/1042 @kkhatua Thanks, I'll have a look at that PR. ---

[GitHub] drill issue #1042: DRILL-5261: Expose REST endpoint in zookeeper

2018-04-30 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1042 @xhochy I recently had a PR #1203 for DRILL-6289 and realized your PR's solution was a subset of that. I've already incorporated changes to the C++ client's protobuf as well. If it addresses the

[GitHub] drill issue #1042: DRILL-5261: Expose REST endpoint in zookeeper

2017-12-28 Thread kkhatua
Github user kkhatua commented on the issue: https://github.com/apache/drill/pull/1042 @paul-rogers , since you've responded to the original JIRA, perhaps you can review this for @xhochy ---

[GitHub] drill issue #1042: DRILL-5261: Expose REST endpoint in zookeeper

2017-11-19 Thread xhochy
Github user xhochy commented on the issue: https://github.com/apache/drill/pull/1042 This still needs the protobufs to be re-compiled for C++, I would do this once someone signals me that I have taken the correct approach. ---