[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15015160#comment-15015160
]
Akshay Rai commented on PIG-4417:
-
Thanks for the commit [~daijy]
> Pig's register command should support
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15011261#comment-15011261
]
Akshay Rai commented on PIG-4417:
-
[~daijy], the latest patch PIG-4417.5.patch is up to date. All issues are
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15011640#comment-15011640
]
Daniel Dai commented on PIG-4417:
-
Thanks, I will take a look shortly.
> Pig's register command should
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15005501#comment-15005501
]
Akshay Rai commented on PIG-4417:
-
[~rdsr], thanks for pointing it out. The latest patch(5) will stream the
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14995147#comment-14995147
]
Ratandeep Ratti commented on PIG-4417:
--
The current approach for finding the ivysettings.xml from the
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14995135#comment-14995135
]
Akshay Rai commented on PIG-4417:
-
[~daijy], I have addressed your last comment.
Here is the ticket to
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14994322#comment-14994322
]
Daniel Dai commented on PIG-4417:
-
Another minor comment, now we wrap ivysettings.xml into pig.jar, we don't
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14991146#comment-14991146
]
Akshay Rai commented on PIG-4417:
-
[~daijy], I have addressed your first 3 comments. I also added a test
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14982965#comment-14982965
]
Akshay Rai commented on PIG-4417:
-
Thanks [~daijy]. I will the update the patch soon. I might need some help
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14977083#comment-14977083
]
Daniel Dai commented on PIG-4417:
-
Ok, I am fine to add this. Several minor comments:
1. Config
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14972414#comment-14972414
]
Anthony Hsu commented on PIG-4417:
--
[~daijy], Hive also supports this syntax. See:
* HIVE-9664
*
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14900505#comment-14900505
]
Akshay Rai commented on PIG-4417:
-
Updated patch: PIG-4417.2.patch
- Added apache license to the ivysettings
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14378244#comment-14378244
]
Daniel Dai commented on PIG-4417:
-
[~akshayrai09], try to understand why you need this.
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14378240#comment-14378240
]
Ratandeep Ratti commented on PIG-4417:
--
Thanks [~alangates] for the quick feedback.
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14378198#comment-14378198
]
Alan Gates commented on PIG-4417:
-
A couple of comments:
# Review board is great for
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14378313#comment-14378313
]
Ratandeep Ratti commented on PIG-4417:
--
Some of the reasons I can think of for this
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14354793#comment-14354793
]
Akshay Rai commented on PIG-4417:
-
[~rdsr], I have removed the optional property,
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14354790#comment-14354790
]
Akshay Rai commented on PIG-4417:
-
5. Group/Organization and Version are optional fields. In
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14344711#comment-14344711
]
Ratandeep Ratti commented on PIG-4417:
--
I think we should refrain from configuring the
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14344709#comment-14344709
]
Akshay Rai commented on PIG-4417:
-
RB Link: https://reviews.apache.org/r/31662/
Pig's
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14344645#comment-14344645
]
Akshay Rai commented on PIG-4417:
-
To add to the above doc,
4. The location of the
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14338578#comment-14338578
]
Akshay Rai commented on PIG-4417:
-
h3.Updates and changes: (Ignore my first comment)
All
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14319654#comment-14319654
]
Akshay Rai commented on PIG-4417:
-
Hi [~nielsbasjes],
Thanks for your input. I would like
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14318426#comment-14318426
]
Niels Basjes commented on PIG-4417:
---
At first glance this seems like a nice feature. I do
[
https://issues.apache.org/jira/browse/PIG-4417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14317675#comment-14317675
]
Akshay Rai commented on PIG-4417:
-
Pig Register Command currently supports 3 types of
25 matches
Mail list logo