You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@beam.apache.org by "Beam JIRA Bot (Jira)" <ji...@apache.org> on 2022/04/25 17:28:00 UTC

[jira] [Commented] (BEAM-12877) Beam Row Avro conversion: fixed and bytes decimals

    [ https://issues.apache.org/jira/browse/BEAM-12877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17527664#comment-17527664 ] 

Beam JIRA Bot commented on BEAM-12877:
--------------------------------------

This issue is P2 but has been unassigned without any comment for 60 days so it has been labeled "stale-P2". If this issue is still affecting you, we care! Please comment and remove the label. Otherwise, in 14 days the issue will be moved to P3.

Please see https://beam.apache.org/contribute/jira-priorities/ for a detailed explanation of what these priorities mean.


> Beam Row Avro conversion: fixed and bytes decimals
> --------------------------------------------------
>
>                 Key: BEAM-12877
>                 URL: https://issues.apache.org/jira/browse/BEAM-12877
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-java-core
>    Affects Versions: 2.32.0
>            Reporter: Koen Egelink
>            Priority: P2
>              Labels: stale-P2
>         Attachments: beam-sdks-java-core-2.32.0-fixed-decimal.patch
>
>
> An Avro decimal logical type annotates Avro bytes or fixed types.
> Current Row to Avro conversion is limited to bytes type and in addition hardcodes precision to MAX_INT and scale to 0. 
> I have attached a patch that adds support for decimal bytes and fixed types.
> I could think of 2 possible ways to solve this:
>  # Change Row decimal type to a logical type
>  # Use Beam field options to pass additional metadata required to serialize decimals
> I felt that overhauling Beam schema types might not be a good idea. Instead I went with option 2.
> Passes ./gradlew check
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)