Thomas Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/13338
Change subject: IMPALA-8545 (part 2): Add tests for JDBC + LDAP + SSL ...................................................................... IMPALA-8545 (part 2): Add tests for JDBC + LDAP + SSL A previous commit added LdapJdbcTest, which tests that JDBC connections work as expected when ldap auth is in use. This patch parameterizes LdapJdbcTest to run both with and without SSL enabled. TODO: This patch relies on @BeforeParam, which is only available in JUnit 4.13 which has only been released in beta. I expect that JUnit 4.13 will be released soon, based on this thread: https://github.com/junit-team/junit4/issues/1496 We can probably wait until then to commit this patch, so that we don't have a beta dependency in our pom. Change-Id: I2af89cd374f8b8783fc29fc91ceac5513d857cb5 --- M fe/src/test/java/org/apache/impala/service/LdapJdbcTest.java M fe/src/test/java/org/apache/impala/testutil/ImpalaJdbcClient.java M impala-parent/pom.xml 3 files changed, 40 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/38/13338/1 -- To view, visit http://gerrit.cloudera.org:8080/13338 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2af89cd374f8b8783fc29fc91ceac5513d857cb5 Gerrit-Change-Number: 13338 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Marshall <[email protected]>
