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

Fix Javadoc Warnings in Generated Records

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 1.8.2
    • 1.9.0
    • java
    • None

    Description

      I've been seeing a number of javadoc errors for the generated records using the 1.8.2 version of SpecificCompiler.

      For example, if I generate code from the following schema:

      {
          "type":"record",
          "name":"Test",
          "namespace":"foo.bar",
          "fields":[
              {"name":"name","type":"string"}
           ],
          "version":1
      }
      

      And then run the javadoc tool on it, I get the following javadoc warnings:

      [warn] src_managed/main/Test.java:30: warning: no @return
      [warn]   public static BinaryMessageDecoder<Test> getDecoder() {
      [warn]                                            ^
      [warn] src_managed/main/Test.java:38: warning: no @return
      [warn]   public static BinaryMessageDecoder<Test> createDecoder(SchemaStore resolver) {
      [warn]                                            ^
      [warn] src_managed/main/Test.java:43: warning: no @return
      [warn]   public java.nio.ByteBuffer toByteBuffer() throws java.io.IOException {
      [warn]                              ^
      [warn] src_managed/main/Test.java:43: warning: no @throws for java.io.IOException
      [warn]   public java.nio.ByteBuffer toByteBuffer() throws java.io.IOException {
      [warn]                              ^
      [warn] src_managed/main/Test.java:48: warning: no @param for b
      [warn]   public static Test fromByteBuffer(
      [warn]                      ^
      [warn] src_managed/main/Test.java:48: warning: no @return
      [warn]   public static Test fromByteBuffer(
      [warn]                      ^
      [warn] src_managed/main/Test.java:48: warning: no @throws for java.io.IOException
      [warn]   public static Test fromByteBuffer(
      [warn]                      ^
      

      Since this is a simple enough problem, I'll see if I can make a quick patch in a bit.

      Attachments

        1. Test.java
          7 kB
          Bridger Howell

        Issue Links

          Activity

            People

              howellbridger Bridger Howell
              howellbridger Bridger Howell
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: