Mailing-List: contact issues-help@drill.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@drill.apache.org
Date: Wed, 22 Jul 2015 21:45:04 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: issues@drill.apache.org
Message-ID: <JIRA.12822039.1429315935000.264816.1437601504791@Atlassian.JIRA>
In-Reply-To: <JIRA.12822039.1429315935000@Atlassian.JIRA>
References: <JIRA.12822039.1429315935000@Atlassian.JIRA>
 <JIRA.12822039.1429315935142@arcas>
Subject: [jira] [Commented] (DRILL-2818) Error message must be updated when
 query fails with FileNotFoundException
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/DRILL-2818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14637692#comment-14637692 ] 

ASF GitHub Bot commented on DRILL-2818:
---------------------------------------

Github user jaltekruse commented on a diff in the pull request:

    https://github.com/apache/drill/pull/93#discussion_r35268073
  
    --- Diff: common/src/main/java/org/apache/drill/common/config/DrillConfig.java ---
    @@ -138,34 +154,83 @@ public static DrillConfig create(Properties testConfigurations) {
         return create(null, testConfigurations, true);
       }
     
    -  public static DrillConfig create(String overrideFileName, boolean enableServerConfigs) {
    -    return create(overrideFileName, null, enableServerConfigs);
    +  /**
    +   * ...
    +   * @param overrideFileResourcePathname
    +   *          see {@link #create(String)}'s {@code overrideFileResourcePathname}
    +   */
    +  public static DrillConfig create(String overrideFileResourcePathname, boolean enableServerConfigs) {
    +    return create(overrideFileResourcePathname, null, enableServerConfigs);
       }
     
    -  private static DrillConfig create(String overrideFileName, Properties overriderProps, boolean enableServerConfigs) {
    -    overrideFileName = overrideFileName == null ? CommonConstants.CONFIG_OVERRIDE : overrideFileName;
    -
    -    // first we load defaults.
    +  /**
    +   * ...
    --- End diff --
    
    I think if you should just put in explicit todo. However I'm not even sure that it makes sense to document these methods individually. There is a comment that is very descriptive about what all of these do, it's just above the wrong version of the method. If you move it above the one that takes all of the parameters, I think it would be fine to even leave out the parameter descriptions on the other versions and have a common message directing people to the one descriptive comment (which also describes all possible parameters). No need for a todo, not a bunch of work up front. 


> Error message must be updated when query fails with FileNotFoundException
> -------------------------------------------------------------------------
>
>                 Key: DRILL-2818
>                 URL: https://issues.apache.org/jira/browse/DRILL-2818
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: SQL Parser
>    Affects Versions: 0.9.0
>         Environment: exception branch
>            Reporter: Abhishek Girish
>            Assignee: Deneche A. Hakim
>            Priority: Minor
>              Labels: error_message_must_fix
>             Fix For: 1.3.0
>
>
> When user specifies a non-existent file/directory in a query, the following error is being thrown:
> {code:sql}
> > show files from dfs.tmp.`tpch`;
> Query failed: SYSTEM ERROR: Failure handling SQL.
> [9184097e-8339-42d3-96ce-1fba51c6bc78 on 192.168.158.107:31010]
> Error: exception while executing query: Failure while executing query. (state=,code=0)
> {code}
> This should be updated to 
> {code:sql}
> > show files from dfs.tmp.`tpch`;
> Query failed: File /tmp/tpch does not exist. 
> {code}


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)