[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-06 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16354309#comment-16354309 ] Vihang Karajgaonkar commented on HIVE-18347: latest patch looks good to me +1 > Allow dynamic

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-06 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16354253#comment-16354253 ] Szehon Ho commented on HIVE-18347: -- [~vihangk1] do you want to have another review on my rebased patch ? 

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-05 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16353207#comment-16353207 ] Hive QA commented on HIVE-18347: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-05 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16353160#comment-16353160 ] Hive QA commented on HIVE-18347: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-05 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16352752#comment-16352752 ] Szehon Ho commented on HIVE-18347: -- Thanks a lot.  Also I guess HIVE-18449 conflicts with this, so

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-02 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16350788#comment-16350788 ] Vihang Karajgaonkar commented on HIVE-18347: Hi [~szehon] Yes you are right. HIVE-18449

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-02-02 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16350780#comment-16350780 ] Szehon Ho commented on HIVE-18347: -- Hi Vihang, thanks for the review!  Sorry about the late response. 

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-26 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16341405#comment-16341405 ] Vihang Karajgaonkar commented on HIVE-18347: Hi [~szehon] Thanks for the changes. I left some

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-23 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16335975#comment-16335975 ] Hive QA commented on HIVE-18347: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-22 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16334604#comment-16334604 ] Szehon Ho commented on HIVE-18347: -- Thanks for the message, [~thejas]!  Included a review request for a

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-18 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16330884#comment-16330884 ] Thejas M Nair commented on HIVE-18347: -- [~szehon] can you also please include a github pull request

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-18 Thread Thejas M Nair (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16330882#comment-16330882 ] Thejas M Nair commented on HIVE-18347: -- I think adding the support for hook and leaving the Consul

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-18 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16330785#comment-16330785 ] Szehon Ho commented on HIVE-18347: -- After reading the discussion on HIVE-18449, this looks like still a

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-17 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16328727#comment-16328727 ] Szehon Ho commented on HIVE-18347: -- Thanks for the response guys.  What Vihang says makes sense, I am

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-16 Thread Alan Gates (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16327541#comment-16327541 ] Alan Gates commented on HIVE-18347: --- cc [~thejas], as he's done work in this area. > Allow dynamic

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-15 Thread Vihang Karajgaonkar (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16326606#comment-16326606 ] Vihang Karajgaonkar commented on HIVE-18347: Hi [~szehon] I like the idea of having a

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2018-01-15 Thread Szehon Ho (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16326348#comment-16326348 ] Szehon Ho commented on HIVE-18347: -- [~alangates] [~vihangk1] any thoughts on whether this is a useful

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2017-12-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16306497#comment-16306497 ] Hive QA commented on HIVE-18347: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2017-12-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16306465#comment-16306465 ] Hive QA commented on HIVE-18347: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2017-12-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16306443#comment-16306443 ] Hive QA commented on HIVE-18347: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-18347) Allow dynamic lookup of Hive Metastores via Consul

2017-12-29 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-18347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16306408#comment-16306408 ] Hive QA commented on HIVE-18347: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||