[ 
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

Reply via email to