Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1325

Druid adapter requires Guava 14.0 or higher

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.8.0
    • 1.9.0
    • None
    • None

    Description

      Druid adapter requires Guava version 14 or higher. With guava-13.0.1 compile fails as follows, due to the missing Maps.asMap method, which was added in 14.0:

      [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.2:compile (default-compile) on project calcite-druid: Compilation failure
      [ERROR] /home/jhyde/regress/calcite/druid/src/main/java/org/apache/calcite/adapter/druid/DruidSchema.java:[66,16] method asMap in class com.google.common.collect.Maps cannot be applied to given types;
      [ERROR] required: java.util.SortedSet<K>,com.google.common.base.Function<? super K,V>
      [ERROR] found: com.google.common.collect.ImmutableSet<java.lang.String>,com.google.common.cache.LoadingCache<java.lang.String,org.apache.calcite.schema.Table>
      [ERROR] reason: cannot infer type-variable(s) K,V
      [ERROR] (argument mismatch; com.google.common.collect.ImmutableSet<java.lang.String> cannot be converted to java.util.SortedSet<K>)
      

      We were incorrect to state that Calcite release 1.8 works with "Guava versions 12.0.1 to 19.0"; it should state "Guava versions 14.0 to 19.0".

      Calcite release 1.7 and earlier do not have this problem.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            julianhyde Julian Hyde
            julianhyde Julian Hyde
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment