Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve exception message for FieldExtractor #228

Closed
costin opened this issue Jul 10, 2014 · 1 comment
Closed

Improve exception message for FieldExtractor #228

costin opened this issue Jul 10, 2014 · 1 comment
Labels
:Cascading enhancement :MR :Pig v2.0.1 v2.1.0.Beta1

Comments

@costin
Copy link
Member

costin commented Jul 10, 2014

When encountering an error, the FieldExtractor implementations should be more helpful in displaying the field request and the target value, where possible.
Messages like
Caused by: org.elasticsearch.hadoop.EsHadoopIllegalArgumentException: [org.elasticsearch.hadoop.pig.PigFieldExtractor@4e5bcff7] cannot extract value from object [org.elasticsearch.hadoop.pig.PigTuple@6db28951]

are not really helpful.
costin added a commit that referenced this issue Jul 14, 2014
@costin
Add explanation about failing field extractions
c14137b 
Explain what fields are not found and also the target data set to
improve debugging.

related to #228
costin added a commit that referenced this issue Jul 31, 2014
@costin
Add explanation about failing field extractions
a39d9bb 
Explain what fields are not found and also the target data set to
improve debugging.

related to #228

(cherry picked from commit c14137b)
@costin
Copy link
Member Author

costin commented Jul 31, 2014

Fixed in master and 2.x

@costin costin closed this as completed Jul 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Cascading enhancement :MR :Pig v2.0.1 v2.1.0.Beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@costin