-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55511/
-----------------------------------------------------------
Review request for Ambari, DIPAYAN BHOWMICK, Gaurav Nagar, Pallav Kulshreshtha,
Prabhjyot Singh, Rohit Choudhary, Renjith Kamath, and Ashwin Rajeev.
Bugs: AMBARI-19528
https://issues.apache.org/jira/browse/AMBARI-19528
Repository: ambari
Description
-------
Before this patch :
Right now when new file-system support has to be added to views, we need to add
code to include several more key value pairs into the conf to make it work.
Usually these key value pairs are file-system class names, secret key and
access key.
In this patch:
1. load the complete core-site and hdfs-site into the conf for creating
filesystem. This will load any secret key, access key configured. It will also
load any other necessary parameters.
2. Create a new view setting's parameter per view which will have semi colon
separated key=value pairs that needs to be loaded into conf for connecting to
the filesystem. This will cover any extra parameters.
3. changes in pom.xml -> included Amazon S3 related jars and excluded
conflicting jars.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/view/ClusterImpl.java
10f216e
ambari-server/src/main/java/org/apache/ambari/server/view/RemoteAmbariCluster.java
6a4f04a
ambari-views/src/main/java/org/apache/ambari/view/cluster/Cluster.java
7f7c19b
contrib/views/commons/pom.xml 4b7ee3a
contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/FileOperationService.java
bc3d11d
contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/HdfsService.java
91eebcf
contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/UploadService.java
97253ad
contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/UserService.java
dc303ce
contrib/views/commons/src/main/java/org/apache/ambari/view/commons/hdfs/ViewPropertyHelper.java
PRE-CREATION
contrib/views/files/pom.xml ad9af4f
contrib/views/files/src/main/java/org/apache/ambari/view/filebrowser/DownloadService.java
96d3541
contrib/views/files/src/main/java/org/apache/ambari/view/filebrowser/FileBrowserService.java
adaa6c9
contrib/views/files/src/main/java/org/apache/ambari/view/filebrowser/FilePreviewService.java
051e40d
contrib/views/files/src/main/java/org/apache/ambari/view/filebrowser/HelpService.java
92af2d5
contrib/views/files/src/main/resources/view.xml 640cee9
contrib/views/hive-next/pom.xml 2e7beab
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/internal/HdfsApiSupplier.java
1cc1211
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/resources/files/FileService.java
654bfcc
contrib/views/hive-next/src/main/java/org/apache/ambari/view/hive2/utils/SharedObjectsFactory.java
dd0b715
contrib/views/hive-next/src/main/resources/view.xml 1107cd1
contrib/views/hive20/pom.xml 6fc30f0
contrib/views/hive20/src/main/java/org/apache/ambari/view/hive20/internal/HdfsApiSupplier.java
e66b9ab
contrib/views/hive20/src/main/java/org/apache/ambari/view/hive20/resources/browser/FileService.java
96e9554
contrib/views/hive20/src/main/java/org/apache/ambari/view/hive20/resources/files/FileService.java
a3623e9
contrib/views/hive20/src/main/java/org/apache/ambari/view/hive20/utils/SharedObjectsFactory.java
5bc6070
contrib/views/hive20/src/main/resources/view.xml 315d0a9
contrib/views/jobs/src/main/resources/ui/.gitignore 503a8a9
contrib/views/pig/pom.xml d7d2de9
contrib/views/pig/src/main/java/org/apache/ambari/view/pig/resources/files/FileService.java
509b20e
contrib/views/pig/src/main/java/org/apache/ambari/view/pig/utils/UserLocalObjects.java
8293899
contrib/views/pig/src/main/resources/view.xml da8ffdd
contrib/views/pom.xml 13cfb23
contrib/views/utils/pom.xml c3cdab5
contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/ConfigurationBuilder.java
d505e9a
contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/HdfsApi.java
c7ae952
contrib/views/utils/src/main/java/org/apache/ambari/view/utils/hdfs/HdfsUtil.java
8bc3ca5
contrib/views/wfmanager/pom.xml 95b988f
contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/FileServices.java
ab286b2
contrib/views/wfmanager/src/main/java/org/apache/oozie/ambari/view/HDFSFileUtils.java
327d8fc
contrib/views/wfmanager/src/main/resources/view.xml c5d4a5b
Diff: https://reviews.apache.org/r/55511/diff/
Testing
-------
manual testing done on amazon s3 cluster and normal hdfs cluster.
Thanks,
Nitiraj Rathore
