[
https://issues.apache.org/jira/browse/PIG-1918?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13014880#comment-13014880
]
Daniel Dai commented on PIG-1918:
---------------------------------
Some comments:
1. When I enter "a = load '1.txt' (a0, a1, a2);" in grunt, I get "ERROR 0:
<file null, line 1, column 17> mismatched input '(' expecting SEMI_COLON". It
is better to hide "file null"
2. We introduce PlanValidationException. It inherits VisitorException, which is
fine. But we need to clarify when to use PlanValidationException. I saw some
place we use PlanValidationException and FrontendException mixed.
3. May relate to 2, seems not all exceptions will include line number in
logical plan. How do we decide which one to include, not one does not?
4. I guess we will not include line number in physical/MR plan in 0.9, right?
> Line number should be give for logical plan failures
> ----------------------------------------------------
>
> Key: PIG-1918
> URL: https://issues.apache.org/jira/browse/PIG-1918
> Project: Pig
> Issue Type: Bug
> Reporter: Olga Natkovich
> Assignee: Xuefu Zhang
> Fix For: 0.9.0
>
> Attachments: PIG-1918-2.patch, PIG-1918.patch
>
>
> Currently, the line is only given for the cases where we encounter problem in
> AST but not on the logical plan. It would be much more meaningful to users if
> it covered both
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
