Avro
  1. Avro
  2. AVRO-1441

Avro Class Files Output Directory

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.6
    • Fix Version/s: 1.7.7
    • Component/s: doc
    • Labels:
      None

      Description

      The current documentation shows that Avro class files can be generated to the src/ directory. They should be generated to the target/ directory. Having files go to the src/ directory means they are not cleaned during a mvn clean. Guess who has two thumbs and ran into this issue?

      1. targetdirectory_withdefaults.diff
        1 kB
        Jesse Anderson
      2. targetdirectory.diff
        0.6 kB
        Jesse Anderson

        Activity

        Jesse Anderson created issue -
        Hide
        Jesse Anderson added a comment -

        Doc fix.

        Show
        Jesse Anderson added a comment - Doc fix.
        Jesse Anderson made changes -
        Field Original Value New Value
        Attachment targetdirectory.diff [ 12623530 ]
        Hide
        Doug Cutting added a comment -

        The default is to find sources in src/main/avro and place generated files in target/generated-sources, so couldn't we actually leave this empty?

        Show
        Doug Cutting added a comment - The default is to find sources in src/main/avro and place generated files in target/generated-sources, so couldn't we actually leave this empty?
        Hide
        Jesse Anderson added a comment -

        Here's a new patch. This one adds a paragraph talking about the defaults. I think the full settings should be shown here because this is the page that shows up when you Google "avro pom.xml". Others may have different configurations and want to see how to configure the avro-maven-plugin quickly.

        Show
        Jesse Anderson added a comment - Here's a new patch. This one adds a paragraph talking about the defaults. I think the full settings should be shown here because this is the page that shows up when you Google "avro pom.xml". Others may have different configurations and want to see how to configure the avro-maven-plugin quickly.
        Jesse Anderson made changes -
        Attachment targetdirectory_withdefaults.diff [ 12623680 ]
        Hide
        ASF subversion and git services added a comment -

        Commit 1562204 from Doug Cutting in branch 'avro/trunk'
        [ https://svn.apache.org/r1562204 ]

        AVRO-1441. Java: Improve documentation for Maven configuration. Contributed by Jesse Anderson.

        Show
        ASF subversion and git services added a comment - Commit 1562204 from Doug Cutting in branch 'avro/trunk' [ https://svn.apache.org/r1562204 ] AVRO-1441 . Java: Improve documentation for Maven configuration. Contributed by Jesse Anderson.
        Hide
        Doug Cutting added a comment -

        I committed this. Thanks, Jesse!

        Show
        Doug Cutting added a comment - I committed this. Thanks, Jesse!
        Doug Cutting made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Jesse Anderson [ eljefe6a ]
        Fix Version/s 1.7.7 [ 12326041 ]
        Resolution Fixed [ 1 ]
        Hide
        Hudson added a comment -

        ABORTED: Integrated in AvroJava #431 (See https://builds.apache.org/job/AvroJava/431/)
        AVRO-1441. Java: Improve documentation for Maven configuration. Contributed by Jesse Anderson. (cutting: rev 1562204)

        • /avro/trunk/CHANGES.txt
        • /avro/trunk/doc/src/content/xdocs/mr.xml
        Show
        Hudson added a comment - ABORTED: Integrated in AvroJava #431 (See https://builds.apache.org/job/AvroJava/431/ ) AVRO-1441 . Java: Improve documentation for Maven configuration. Contributed by Jesse Anderson. (cutting: rev 1562204) /avro/trunk/CHANGES.txt /avro/trunk/doc/src/content/xdocs/mr.xml
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        11d 20h 32m 1 Doug Cutting 28/Jan/14 20:02
        Resolved Resolved Closed Closed
        177d 13m 1 Doug Cutting 24/Jul/14 21:16

          People

          • Assignee:
            Jesse Anderson
            Reporter:
            Jesse Anderson
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development