[
https://issues.apache.org/jira/browse/PIG-1918?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13014891#comment-13014891
]
Xuefu Zhang commented on PIG-1918:
----------------------------------
Responses for comments above:
1.The message is probably due to Richard's change. If you try with a semantic
error case, you will not see the null file name. This issue can be addressed
separately.
2. FrontendException is a more general exception and PlanValidationException is
more specific. The mixed use of them isn't introduced by this patch. In
principle, a as specific as possible exception should be thrown, even the
method signature may throw a more general exception.
3. User facing exceptions should have line numbers. This excludes exceptions
due to bug (mostly).
4. We will not have line number in physical/MR plans.
> 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