Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13337
to look at the new patch set (#3).
Change subject: IMPALA-8545: Test Ldap authentication
......................................................................
IMPALA-8545: Test Ldap authentication
Currently, Impala does not have any automated tests for LDAP auth
functionality, due to the challenge of setting up an LDAP server
for use by the minicluster.
This patch adds LDAP tests by utilizing the ApacheDS project's
unit testing functionality, which works with JUnit to setup up
a local LDAP server for the duration of a test suite.
This requires running an Impala cluster with custom arguments to
set up LDAP auth. This patch introduces a concept of FE custom cluster
tests which must be in the package org.apache.impala.customcluster.
These tests are filtered out from the other FE tests in
bin/run-all-tests.sh and run with the other custom cluster tests so
that they don't affect other tests that expect Impala to have been
started with particular flags.
Testing:
- Ran a full core run and confirmed that new tests run as expected.
Change-Id: I92b5e60860c60209c1bd8afe5b3ea201fb7a7513
---
M bin/rat_exclude_files.txt
M bin/run-all-tests.sh
M fe/pom.xml
A fe/src/test/java/org/apache/impala/customcluster/CustomClusterRunner.java
A fe/src/test/java/org/apache/impala/customcluster/LdapJdbcTest.java
M fe/src/test/java/org/apache/impala/service/JdbcTest.java
A fe/src/test/java/org/apache/impala/service/JdbcTestBase.java
M fe/src/test/java/org/apache/impala/testutil/ImpalaJdbcClient.java
A fe/src/test/resources/users.ldif
9 files changed, 329 insertions(+), 89 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/37/13337/3
--
To view, visit http://gerrit.cloudera.org:8080/13337
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I92b5e60860c60209c1bd8afe5b3ea201fb7a7513
Gerrit-Change-Number: 13337
Gerrit-PatchSet: 3
Gerrit-Owner: Thomas Marshall <tmarsh...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
