Avro
  1. Avro
  2. AVRO-988

Java: Add public, private, and public-deprecated options for SpecificCompiler output

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2
    • Fix Version/s: 1.7.3
    • Component/s: java
    • Labels:
      None
    • Release Note:
      Adds field visibility options for SpecificCompiler output

      Description

      The specific compiler produces deprecation warnings in both client code and the specific generated classes themselves. The field visibility should be configurable by the user.
      I propose that the SpecificCompiler for the default template allow for a fieldVisibility option that is one of the below:

      • 'public' fields are public, and not deprecated, this is the 1.5.x behavior
      • 'public-deprecated' fields are public, and marked @Deprecated. This is the default behavior in 1.6.x
      • 'private' fields are private. This may be the default in the future.
      1. AVRO-988.patch
        13 kB
        Jeff Kolesky

        Activity

        Scott Carey created issue -
        Scott Carey made changes -
        Field Original Value New Value
        Component/s java [ 12312780 ]
        Jeff Kolesky made changes -
        Attachment AVRO-988.patch [ 12554269 ]
        Jeff Kolesky made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Release Note Adds field visibility options for SpecificCompiler output
        Affects Version/s 1.7.3 [ 12323298 ]
        Affects Version/s 1.6.1 [ 12318847 ]
        Doug Cutting made changes -
        Assignee Jeff Kolesky [ jeffkole ]
        Fix Version/s 1.7.3 [ 12323298 ]
        Affects Version/s 1.7.2 [ 12322476 ]
        Affects Version/s 1.7.3 [ 12323298 ]
        Doug Cutting made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Jeff Kolesky
            Reporter:
            Scott Carey
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development