----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38695/#review100320 -----------------------------------------------------------
Ship it! Ship It! - Srimanth Gunturi On Sept. 23, 2015, 10:46 p.m., Pallav Kulshreshtha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38695/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2015, 10:46 p.m.) > > > Review request for Ambari, Srimanth Gunturi, Sid Wagle, and Yusaku Sako. > > > Bugs: AMBARI-13217 > https://issues.apache.org/jira/browse/AMBARI-13217 > > > Repository: ambari > > > Description > ------- > > Implemented LDAP authentication for HIVE view. > > > Diffs > ----- > > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/Connection.java > c30b269 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/ConnectionFactory.java > 5a4406a > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/HiveAuthCredentials.java > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/HiveAuthRequiredException.java > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/HiveClientAuthRequiredException.java > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/IConnectionFactory.java > 7a3df06 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/UserLocalConnection.java > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/client/UserLocalHiveAuthCredentials.java > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/browser/HiveBrowserService.java > 55919a7 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/browser/HiveBrowserService.java.orig > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/JobService.java > ad46e33 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/JobService.java.orig > PRE-CREATION > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/OperationHandleController.java > 8156c29 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/OperationHandleControllerFactory.java > 4297078 > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobControllerFactory.java > 12d1cdb > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/resources/jobs/viewJobs/JobControllerImpl.java > 32d64ec > > contrib/views/hive/src/main/java/org/apache/ambari/view/hive/utils/SharedObjectsFactory.java > df67985 > > contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/databases.js > e6bc7f1 > contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/index.js > 2519df1 > > contrib/views/hive/src/main/resources/ui/hive-web/app/controllers/modal-save.js > 6dc1349 > contrib/views/hive/src/main/resources/ui/hive-web/app/initializers/i18n.js > 8f9f72d > contrib/views/hive/src/main/resources/ui/hive-web/app/routes/application.js > 11b93a1 > contrib/views/hive/src/main/resources/ui/hive-web/app/services/database.js > 6b4df6f > > contrib/views/hive/src/main/resources/ui/hive-web/app/templates/modal-save.hbs > f327e3b > > contrib/views/hive/src/test/java/org/apache/ambari/view/hive/BaseHiveTest.java > 2b915f0 > > contrib/views/hive/src/test/java/org/apache/ambari/view/hive/client/ConnectionTest.java > PRE-CREATION > > contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobLDAPServiceTest.java > PRE-CREATION > > contrib/views/hive/src/test/java/org/apache/ambari/view/hive/resources/jobs/JobServiceTest.java > 897d951 > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/UserLocal.java > PRE-CREATION > > contrib/views/utils/src/main/java/org/apache/ambari/view/utils/UserLocalFactory.java > PRE-CREATION > > contrib/views/utils/src/test/java/org/apache/ambari/view/utils/ViewUserLocalTest.java > c82df62 > > Diff: https://reviews.apache.org/r/38695/diff/ > > > Testing > ------- > > ------------------------------------------------------- > T E S T S > ------------------------------------------------------- > Running org.apache.ambari.view.hive.backgroundjobs.BackgroundJobControllerTest > Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.421 sec > Running org.apache.ambari.view.hive.client.ConnectionTest > Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.388 sec > Running org.apache.ambari.view.hive.resources.udfs.UDFServiceTest > Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.598 sec > Running org.apache.ambari.view.hive.resources.jobs.LogParserTest > Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.069 sec > Running org.apache.ambari.view.hive.resources.jobs.ATSParserTest > Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.162 sec > Running org.apache.ambari.view.hive.resources.jobs.JobLDAPServiceTest > Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.87 sec > Running org.apache.ambari.view.hive.resources.jobs.JobServiceTest > Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.905 sec > Running org.apache.ambari.view.hive.resources.jobs.AggregatorTest > Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.213 sec > Running org.apache.ambari.view.hive.resources.files.FileServiceTest > Tests run: 8, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 6.358 sec > Running > org.apache.ambari.view.hive.resources.savedQueries.SavedQueryResourceManagerTest > Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.177 sec > Running > org.apache.ambari.view.hive.resources.savedQueries.SavedQueryServiceTest > Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 4.735 sec > Running > org.apache.ambari.view.hive.resources.resources.FileResourceServiceTest > Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.609 sec > Running org.apache.ambari.view.hive.PropertyValidatorTest > Tests run: 4, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.109 sec > > Results : > > Tests run: 54, Failures: 0, Errors: 0, Skipped: 0 > > [INFO] > [INFO] --- apache-rat-plugin:0.11:check (default) @ hive --- > [INFO] 51 implicit excludes (use -debug for more details). > [INFO] Exclude: .git/ > [INFO] Exclude: **/.gitignore > [INFO] Exclude: **/.gitattributes > [INFO] Exclude: .idea/ > [INFO] Exclude: pass.txt > [INFO] Exclude: .DS_Store > [INFO] Exclude: .iml/ > [INFO] Exclude: .classpath > [INFO] Exclude: .project > [INFO] Exclude: .settings > [INFO] Exclude: **/target/** > [INFO] Exclude: **/.gitkeep > [INFO] Exclude: **/.bowerrc > [INFO] Exclude: **/.editorconfig > [INFO] Exclude: **/.jshintrc > [INFO] Exclude: **/.tmp/** > [INFO] Exclude: **/tmp/** > [INFO] Exclude: **/*.json > [INFO] Exclude: **/*.svg > [INFO] Exclude: **/vendor/** > [INFO] Exclude: **/bin > [INFO] Exclude: **/node_modules/** > [INFO] Exclude: **/vendor/** > [INFO] Exclude: **/public/** > [INFO] Exclude: **/dist/** > [INFO] Exclude: **/bower_components/** > [INFO] Exclude: **/bower/** > [INFO] Exclude: **/node/** > [INFO] Exclude: **/runner.js > [INFO] Exclude: **/assets/javascripts/** > [INFO] Exclude: **/assets/stylesheets/** > [INFO] Exclude: **/assets/static/javascripts/** > [INFO] Exclude: **/assets/static/stylesheets/** > [INFO] Exclude: storm/src/main/resources/** > [INFO] 321 resources included (use -debug for more details) > Warning: org.apache.xerces.jaxp.SAXParserImpl$JAXPSAXParser: Property > 'http://www.oracle.com/xml/jaxp/properties/entityExpansionLimit' is not > recognized. > Compiler warnings: > WARNING: 'org.apache.xerces.jaxp.SAXParserImpl: Property > 'http://javax.xml.XMLConstants/property/accessExternalDTD' is not recognized.' > Warning: org.apache.xerces.parsers.SAXParser: Feature > 'http://javax.xml.XMLConstants/feature/secure-processing' is not recognized. > Warning: org.apache.xerces.parsers.SAXParser: Property > 'http://javax.xml.XMLConstants/property/accessExternalDTD' is not recognized. > Warning: org.apache.xerces.parsers.SAXParser: Property > 'http://www.oracle.com/xml/jaxp/properties/entityExpansionLimit' is not > recognized. > [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 > approved: 320 licence. > [INFO] > [INFO] --- maven-jar-plugin:2.4:jar (default-jar) @ hive --- > [INFO] Building jar: > /shared/submodules/ambari/contrib/views/hive/target/hive-1.0.0-SNAPSHOT.jar > [INFO] > [INFO] --- maven-assembly-plugin:2.2-beta-5:single (make-assembly) @ hive --- > [INFO] Reading assembly descriptor: > /shared/submodules/ambari/ambari-project/src/main/assemblies/empty.xml > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Ambari View Utils .................................. SUCCESS [ 8.437 > s] > [INFO] Hive ............................................... SUCCESS [ 42.402 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 51.272 s > [INFO] Finished at: 2015-09-23T22:43:39+00:00 > [INFO] Final Memory: 68M/1202M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Pallav Kulshreshtha > >
