[jira] [Comment Edited] (OOZIE-2701) Oozie to support Multiple HCatalog URIs

2017-04-04 Thread Peter Cseh (JIRA) 

[ 
https://issues.apache.org/jira/browse/OOZIE-2701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15954737#comment-15954737
 ] 

Peter Cseh edited comment on OOZIE-2701 at 4/4/17 7:49 AM:
---

Thanks for the update [~abhishekbafna]!
I'm a bit confused with the docs:
When you have multiple HCat servers, you were defining the scheme for each of 
them in the tests like
{noformat}
"hcat://hostname1:1000,hcat://hostname2:2000/mydb/clicks/datastamp=12;region=us,scheme://hostname3:3000,"
{noformat}
In the documentation description and in the example, you're defining the scheme 
only once:
{noformat}
hcat://[metastore_server]:[port],[metastore_server]:[port]/[database_name]/[table_name]/[partkey1]=[value];[partkey2]=[value];
hcat://myhcatmetastore:9080,myhcatmetastore:9080/database1/table1/datestamp=${YEAR}${MONTH}${DAY}${HOUR};region=USA
{noformat}
I don't think the pattern in oozie-default.xml would match the URIs in the 
documentation.


was (Author: gezapeti):
Thanks for the update [~abhishekbafna]!
I'm a bit confused with the docs:
When you have multiple HCat servers, you were defining the scheme for each of 
them in the tests like
{quote}
"hcat://hostname1:1000,hcat://hostname2:2000/mydb/clicks/datastamp=12;region=us,scheme://hostname3:3000,"
{quote}
In the documentation description and in the example, you're defining the scheme 
only once:
{quote}
hcat://[metastore_server]:[port],[metastore_server]:[port]/[database_name]/[table_name]/[partkey1]=[value];[partkey2]=[value];
hcat://myhcatmetastore:9080,myhcatmetastore:9080/database1/table1/datestamp=${YEAR}${MONTH}${DAY}${HOUR};region=USA
{quote}
I don't think the pattern in oozie-default.xml would match the URIs in the 
documentation.

> Oozie to support Multiple HCatalog URIs
> ---
>
> Key: OOZIE-2701
> URL: https://issues.apache.org/jira/browse/OOZIE-2701
> Project: Oozie
>  Issue Type: New Feature
>  Components: core
>Reporter: Abhishek Bafna
>Assignee: Abhishek Bafna
> Fix For: 5.0.0
>
> Attachments: OOZIE-2701-00.patch, OOZIE-2701-01.patch, 
> OOZIE-2701-02.patch, OOZIE-2701-03.patch, OOZIE-2701-04.patch, 
> OOZIE-2701-05.patch, OOZIE-2701-06.patch, OOZIE-2701-07.patch, 
> OOZIE-2701-08.patch, OOZIE-2701-09.patch, OOZIE-2701-10.patch, 
> OOZIE-2701-11.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Comment Edited] (OOZIE-2701) Oozie to support Multiple HCatalog URIs

2017-03-28 Thread Abhishek Bafna (JIRA) 

[ 
https://issues.apache.org/jira/browse/OOZIE-2701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944828#comment-15944828
 ] 

Abhishek Bafna edited comment on OOZIE-2701 at 3/28/17 9:09 AM:


[~gezapeti] There was change in regex used for identifying the URLs and moved 
the test cases also into the core module. Can you take another look, for 
resolving this. Thanks.



was (Author: abhishekbafna):
[~gezapeti] There was change in regex used of identifying the URLs and moved 
the test cases also into the core module. Can you take another look, for 
resolving this. Thanks.


> Oozie to support Multiple HCatalog URIs
> ---
>
> Key: OOZIE-2701
> URL: https://issues.apache.org/jira/browse/OOZIE-2701
> Project: Oozie
>  Issue Type: New Feature
>  Components: core
>Reporter: Abhishek Bafna
>Assignee: Abhishek Bafna
> Fix For: 5.0.0
>
> Attachments: OOZIE-2701-00.patch, OOZIE-2701-01.patch, 
> OOZIE-2701-02.patch, OOZIE-2701-03.patch, OOZIE-2701-04.patch, 
> OOZIE-2701-05.patch, OOZIE-2701-06.patch, OOZIE-2701-07.patch, 
> OOZIE-2701-08.patch, OOZIE-2701-09.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Comment Edited] (OOZIE-2701) Oozie to support Multiple HCatalog URIs

2017-03-24 Thread Peter Cseh (JIRA) 

[ 
https://issues.apache.org/jira/browse/OOZIE-2701?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15940232#comment-15940232
 ] 

Peter Cseh edited comment on OOZIE-2701 at 3/24/17 12:25 PM:
-

We should make sure that the regex in the {{oozie-default.xml}} and in 
{{TestHCatURIParser}} are the same.
They are not quite the same in patch-07.
This is the one in {{oozie-default.xml}} and in the test after removing the 
escaping characters:
{noformat}
([a-z]+://[\w\.\-]+:\d+[,]*)+/\w+/\w+[/\w+=;\-]*
([a-z]+://[\w\.\-]+:\d+[,]*)+/\w+/\w+/[\w+=;\-]*
{noformat}
Notice the extra "/" before the last pair of brackets. [~abhishekbafna], could 
you check which one is correct?
Also, I think moving the test to core would worth it to ensure the tested regex 
is the same everywhere.



was (Author: gezapeti):
We should make sure that the regex in the {{oozie-default.xml}} and in 
{{TestHCatURIParser}} are the same.
They are not quite the same in patch-07.
This is the one in {{oozie-default.xml}} and in the test after removing the 
escaping characters:
([a-z]+://[\w\.\-]+:\d+[,]*)+/\w+/\w+[/\w+=;\-]*
([a-z]+://[\w\.\-]+:\d+[,]*)+/\w+/\w+/[\w+=;\-]*

Notice the extra "/" before the last pair of brackets. [~abhishekbafna], could 
you check which one is correct?
Also, I think moving the test to core would worth it to ensure the tested regex 
is the same everywhere.


> Oozie to support Multiple HCatalog URIs
> ---
>
> Key: OOZIE-2701
> URL: https://issues.apache.org/jira/browse/OOZIE-2701
> Project: Oozie
>  Issue Type: New Feature
>  Components: core
>Reporter: Abhishek Bafna
>Assignee: Abhishek Bafna
> Fix For: 5.0.0
>
> Attachments: OOZIE-2701-00.patch, OOZIE-2701-01.patch, 
> OOZIE-2701-02.patch, OOZIE-2701-03.patch, OOZIE-2701-04.patch, 
> OOZIE-2701-05.patch, OOZIE-2701-06.patch, OOZIE-2701-07.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)