Remove usage of shaded imports

It seems there are a few spots where maybe an IDE automatically resolved a FE or test import to from a shaded dependency. We should use the direct dependency to remove confusion and reduce the required jars for some usages of the FE code.

Examples being:
fe/src/main/java/org/apache/impala/analysis/MultiAggregateInfo.java uses org.apache.kudu.shaded.com.google.common.collect.Lists
fe/src/main/java/org/apache/impala/catalog/Transaction.java uses org.apache.curator.shaded.com.google.common.base.Preconditions
fe/src/main/java/org/apache/impala/planner/AnalyticPlanner.java use org.apache.curator.shaded.com.google.common.collect.Iterables
fe/src/test/java/org/apache/impala/analysis/ExprCardinalityTest.java uses org.apache.curator.shaded.com.google.common.collect.Sets