Uploaded image for project: 'Apache Avro'
  1. Apache Avro
  2. AVRO-971

IDL Import from project classpath

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.6.1
    • 1.6.2
    • java
    • Maven java projects

    Description

      Currently, it looks like the only option to importing another schema in IDL is to place the file being imported in the same directory as that of the importing avdl. In a setup where there are avdl's that are spread among several maven projects that are owned by different teams, this is logistically difficult to manage.

      When using the avro-maven-plugin, I would like to be able to just create a dependency from my project on another jar that contains the avdl I am want to import and have Avro be smart enough to look for it in the classpath of the project containing the avdl when compiling my avdl.

      Attached is a working patch that will:
      1. Change the IDLProtocolMojo class to lookup the current project's classpath and create a new ClassLoader.
      2. Give the Idl compiler class the ClassLoader before parsing the avdl.
      3. If the Idl class encounters an import that it cannot resolve to the local directory while parsing, it will try to use the ClassLoader to load up the file being imported.

      The patch spans the Avro 1.6.1 tag of the Java avro, avro-compiler, and avro-maven-plugin projects.

      Attachments

        1. AVRO-971.patch
          12 kB
          Doug Cutting
        2. AVRO-971.patch
          10 kB
          Doug Cutting
        3. ImportFromClassPath_v3.patch
          12 kB
          Victor Chau
        4. ImportFromClassPath_v2.patch
          7 kB
          Victor Chau
        5. ImportFromClassPath.patch
          6 kB
          Victor Chau

        Issue Links

          Activity

            People

              vchau Victor Chau
              vchau Victor Chau
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: