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

Jonathan Coveney updated PIG-2880:
----------------------------------

        Fix Version/s: 0.12
    Affects Version/s:     (was: 0.10.0)
               Status: Open  (was: Patch Available)

Removing patch status pending response to the following comments:

1. you can in-line the checkIfNumber method, I don't know if there is a huge 
gain from making it a method? Don't really care.
2. Please removed auto-generated IDE comments
3. 2 space indents, no tabs please
4. Since you refer to input.get(0) and input.get(1) multiple times, it's 
probably easier to just save those in local variables
                
> Pig current releases lack a UDF charAt.This UDF returns the char value at the 
> specified index.
> ----------------------------------------------------------------------------------------------
>
>                 Key: PIG-2880
>                 URL: https://issues.apache.org/jira/browse/PIG-2880
>             Project: Pig
>          Issue Type: New Feature
>          Components: piggybank
>            Reporter: Sabir Ayappalli
>              Labels: patch
>             Fix For: 0.12
>
>         Attachments: CharAt.java.patch
>
>
> Pig current releases lack a UDF charAt.This UDF returns the char value at the 
> specified index. An index ranges from 0 to length() - 1. The first char value 
> of the sequence is at index 0, the next at index 1, and so on.

--
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