[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Resolution: Fixed Status: Resolved (was: Patch Available) Patch committed. We did a manual unit test. All test pass. For the findbug warning, it is due to a dead variable. The line reference to the variable is commented out in the patch. > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch, PIG-881-3.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Status: Patch Available (was: In Progress) > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch, PIG-881-3.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Status: In Progress (was: Patch Available) > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch, PIG-881-3.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Attachment: PIG-881-3.patch Get all unit test pass. > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch, PIG-881-3.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Status: Patch Available (was: Open) > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Attachment: PIG-881-2.patch One unit test fail, resubmit the patch. > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch, PIG-881-2.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Attachment: PIG-881-1.patch > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Attachment: (was: PIG-881-1.patch) > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (PIG-881) Pig should ship load udfs to the backend
[ https://issues.apache.org/jira/browse/PIG-881?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Daniel Dai updated PIG-881: --- Attachment: PIG-881-1.patch In the patch, I also modify JarManager. One statement in JarManager prevent all classes which is load from classpath to be included. I currently remove this but I am not sure if I am doing it right. > Pig should ship load udfs to the backend > > > Key: PIG-881 > URL: https://issues.apache.org/jira/browse/PIG-881 > Project: Pig > Issue Type: Bug > Components: impl >Affects Versions: 0.3.0 >Reporter: Daniel Dai >Assignee: Daniel Dai > Fix For: 0.4.0 > > Attachments: PIG-881-1.patch > > > Currently, when we use load udfs, we have to use "register" statement. It is > ideal that if user put udf jars in classpath, we can omit register statement, > Pig can pick the udf from classpath automatically. > However, Pig do not ship load udfs currently, the classpath approach does not > work. "register" works because Pig ship that entire jar. Pig do ship eval > udfs and storage udfs, we should ship load udfs as well. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.