Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/20471


Change subject: IMPALA-12432: Make LdapKerberosImpalaShellTest* work with Guava 
28
......................................................................

IMPALA-12432: Make LdapKerberosImpalaShellTest* work with Guava 28

In the change for IMPALA-11726, we added some usages of
Guava's ImmutableMap.of() with >5 key/value pairs. This
won't compile in older versions of Guava (like Guava 28)
that only support up to 5 key/value pairs in ImmutableMap.of().

Being compatible with older versions of Guava makes it
easier for Impala to match the Guava version of Hadoop,
Hive, etc when compiling against older versions.

This switches to using ImmutableMap.builder() to build
the larger maps.

Testing:
 - Built with Guava 28 and ran the tests

Change-Id: Ic6d9c69fff749bcdb0887a6676574db9526f68e8
---
M 
fe/src/test/java/org/apache/impala/customcluster/LdapKerberosImpalaShellTest.java
M 
fe/src/test/java/org/apache/impala/customcluster/LdapKerberosImpalaShellTestBase.java
2 files changed, 32 insertions(+), 32 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/20471/1
--
To view, visit http://gerrit.cloudera.org:8080/20471
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic6d9c69fff749bcdb0887a6676574db9526f68e8
Gerrit-Change-Number: 20471
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <[email protected]>

Reply via email to