[ 
https://issues.apache.org/jira/browse/PIG-2657?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Cheolsoo Park updated PIG-2657:
-------------------------------

    Labels: newbie  (was: )

Additional comments:
- Please remove tabs. Use 4 space instead.
- The property {{jython.version}} shouldn't be hard-coded in {{build.xml}}. It 
is automatically loaded from {{ivy/libraries.properties}} at compile-time, so 
no reason to define it.
- The {{jython.version}} attribute shouldn't be embedded in {{pigunit.jar}}. 
It's useful only in {{pig.jar}} and {{pig-withouthadoop.jar}}.
- Regarding the log message, why don't we print a message like "Pig is tested 
with ${jython.version}, so it may not work with ${runtime.jython.version}"? I 
think that this is flexible and informative at the same time.

Thanks!
                
> Print warning if using wrong jython version
> -------------------------------------------
>
>                 Key: PIG-2657
>                 URL: https://issues.apache.org/jira/browse/PIG-2657
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Fabian Alenius
>              Labels: newbie
>             Fix For: 0.12
>
>         Attachments: PIG-2657.1.patch, PIG-2657.2.patch
>
>
> Hi,
> It would be good if Pig would print a warning (or refuse to run) if you are 
> using an unsupported version of jython. I spent a couple of hours before 
> figuring out that you had to use 2.5.0. I've seen posts indicating that 
> others have run into this problem as well.
> Might write up a patch if others agree this is an issue.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to