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

Specific Data generated class missing Decimal Conversion

Add voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 1.8.2
    • None
    • java
    • None

    Description

      Using 1.8.2-rc3

      Given a class generated with -bigDecimal, the generated class defines the DECIMAL_CONVERSION but does not set it to a BigDecimal field index.

      Fields for illustration:

      @Deprecated public java.lang.String id;
      @Deprecated public org.joda.time.DateTime timestamp;
      @Deprecated public java.lang.String applicationId;
      @Deprecated public java.math.BigDecimal amount;;
      
      ...
      
      protected static final org.apache.avro.data.TimeConversions.TimestampConversion TIMESTAMP_CONVERSION = new org.apache.avro.data.TimeConversions.TimestampConversion();
      protected static final org.apache.avro.Conversions.DecimalConversion DECIMAL_CONVERSION = new org.apache.avro.Conversions.DecimalConversion();
      
      private static final org.apache.avro.Conversion<?>[] conversions =
            new org.apache.avro.Conversion<?>[] {
            null,
            TIMESTAMP_CONVERSION,
            null,
            null, // Should be DECIMAL_CONVERSION
            null,
            null,
            null,
            null,
            null
        };
      

      I am currently unsure of the impact of this.

      Attachments

        Activity

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

          People

            Unassigned Unassigned
            amccague Adrian McCague

            Dates

              Created:
              Updated:

              Time Tracking

                Estimated:
                Original Estimate - 4h
                4h
                Remaining:
                Remaining Estimate - 4h
                4h
                Logged:
                Time Spent - Not Specified
                Not Specified

                Slack

                  Issue deployment