Hi Ivan,
Thank you for contributing.

> that my changes requires also change specification for BigDecimal
> <https://avro.apache.org/docs/current/spec.html#Decimal> to accept string
> as an underlying format for encoding/decoding.


Could you elaborate on what the string representation would be for
decimal?  On the surface this change seems undesirable as it potentially
breaks backwards compatibility of the format.

Thanks,
Micah

On Tue, Mar 29, 2022 at 2:21 PM Ivan Zemlyanskiy <
[email protected]> wrote:

> Hello, avro developers!
> First of all thank you all for your hard work and the project you move
> forward. The result is marvellous and it was my pleasure to make a small
> adjustment for it.
>
> Some time ago I opened an issue
> https://issues.apache.org/jira/browse/AVRO-3408 and made a pull-request
> with my understanding how my ideas should be implemented
> https://github.com/apache/avro/pull/1584
> During the discussion, Ryan Skraba noticed
> <
> https://issues.apache.org/jira/browse/AVRO-3408?focusedCommentId=17508373&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17508373
> >
> that my changes requires also change specification for BigDecimal
> <https://avro.apache.org/docs/current/spec.html#Decimal> to accept string
> as an underlying format for encoding/decoding. He recommended sending a
> note here to discuss if someone has any objections to doing so.
>
> I'm looking forward to reading any ideas/comments/concerns about my work
> and this specification change.
>
> Thank you in advance.
>

Reply via email to