Pradeep Kamath commented on PIG-653:

Not so sure about using RequiredFieldList for subFields - this will mean we 
could ask for all subFields in two ways - 
1. By just asking for the main field (this will imply we need all sub fields)
2. By asking for the main field with a subField which has its allFieldsRequired 
flag set to true.

I think it would be better to keep the subFields as only THE required subfields 
represented as a list. RequiredFieldList is specifically being introduced to 
handle top level information to be given to the loader which may not be 
applicable at a field level. 


> Make fieldsToRead work in loader
> --------------------------------
>                 Key: PIG-653
>                 URL: https://issues.apache.org/jira/browse/PIG-653
>             Project: Pig
>          Issue Type: New Feature
>            Reporter: Alan Gates
>            Assignee: Pradeep Kamath
>         Attachments: PIG-653-2.comment
> Currently pig does not call the fieldsToRead function in LoadFunc, thus it 
> does not provide information to load functions on what fields are needed.  We 
> need to implement a visitor that determines (where possible) which fields in 
> a file will be used and relays that information to the load function.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to