Mailing-List: contact issues-help@flink.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@flink.apache.org
From: rmetzger <git@git.apache.org>
To: issues@flink.incubator.apache.org
Reply-To: issues@flink.incubator.apache.org
References: <git-pr-1524-flink@git.apache.org>
In-Reply-To: <git-pr-1524-flink@git.apache.org>
Subject: [GitHub] flink pull request: FLINK-2380: allow to specify the default
 files...
Content-Type: text/plain
Message-Id: <20160212113329.4B831E03CD@git1-us-west.apache.org>
Date: Fri, 12 Feb 2016 11:33:29 +0000 (UTC)

Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/1524#issuecomment-183288223
  
    Thank you.
    
    - ` ./bin/yarn-session.sh -tm 1024 -n 7 -s 2 -Dfs.default-scheme=hdfs:///` is now working
    - Setting the fs.default-scheme to `thisIsWrong` and trying to read from `/user/robert/trashaa` leads to:
    ```
    Caused by: java.io.IOException: The URI '/user/robert/trashaa' is invalid. Hint: Did you specify only the PATH to your file (without the scheme) but forgot the initial slash (/) in the beginning???
    ```
    I don't think putting three question marks there is a good idea. I think it implicates that the user is too stupid to properly specify a file path.
    And I would like to see the default scheme there. Right now, its quite hard to see what's wrong.
    
    the error reporting for the default scheme: `thisIsWrong:///` and the path `/user/robert/trashaa` is good again: 
    ```
    Caused by: java.io.IOException: No file system found with scheme thisIsWrong, referenced in file URI 'thisIsWrong:/user/robert/trashaaa'.
    ```
    I would expect a similar error reporting for `fs.default-scheme=thisIsWrong`
    
    
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---