-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38387/
-----------------------------------------------------------
(Updated Sept. 21, 2015, 11:38 p.m.)
Review request for Falcon, Ajay Yadava and Sowmya Ramesh.
Changes
-------
Incorporated feedback from Sowmya Ramesh and Venkat Ranganathan. This patch
contains
- Basic validation to ensure readonly and read-write end point are accessible.
- relevant unit tests
Bugs: FALCON-1343
https://issues.apache.org/jira/browse/FALCON-1343
Repository: falcon-git
Description
-------
A read/write endpoint is currently validated by creating a filesystem with the
endpoint url.
{code}
HadoopClientFactory.get().createProxiedFileSystem(conf);
{code}
I confirmed with a HDFS team member that it is not sufficient validation.
Ideally check if the end user has access by doing atleast a list /tmp call
after creating proxiedFileSystem.
Diffs (updated)
-----
common/src/main/java/org/apache/falcon/entity/parser/ClusterEntityParser.java
6bfcb98
common/src/test/java/org/apache/falcon/entity/AbstractTestBase.java 6179855
common/src/test/java/org/apache/falcon/entity/parser/ClusterEntityParserTest.java
2bafac9
common/src/test/resources/config/cluster/cluster-bad-write-endpoint.xml
PRE-CREATION
Diff: https://reviews.apache.org/r/38387/diff/
Testing
-------
Tested end2end, added a unit test.
Thanks,
Balu Vellanki
