[
https://issues.apache.org/jira/browse/BEAM-12628?focusedWorklogId=635014&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-635014
]
ASF GitHub Bot logged work on BEAM-12628:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 06/Aug/21 09:27
Start Date: 06/Aug/21 09:27
Worklog Time Spent: 10m
Work Description: iemejia commented on pull request #15292:
URL: https://github.com/apache/beam/pull/15292#issuecomment-894131835
I merged it manually to squash the commits. Since we already cut the 2.32.0
release branch if you want this to go into the 2.32.0 version we will require
to cherry pick it. Can you please do this @clairemcginty ?
@angoenka I put this in the list for 2.32.0 I think the regression is
important enough to deserve to be cherry picked.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 635014)
Time Spent: 50m (was: 40m)
> AvroCoder changed underlying String class for SpecificRecords
> -------------------------------------------------------------
>
> Key: BEAM-12628
> URL: https://issues.apache.org/jira/browse/BEAM-12628
> Project: Beam
> Issue Type: Bug
> Components: io-java-avro
> Affects Versions: 2.30.0
> Reporter: Ryan Skraba
> Assignee: Claire McGinty
> Priority: P2
> Fix For: 2.32.0
>
> Time Spent: 50m
> Remaining Estimate: 0h
>
> The AvroCoder changes for BEAM-2303 changes the reader/writer from the Avro
> {{ReflectDatum*}} classes to the {{SpecificDatum*}} classes.
> Because of the way Avro handles Strings, however, the underlying instances
> for String data are deserialised as {{org.apache.avro.util.Utf8}} instances
> instead of {{java.lang.String}}.
> This causes:
> 1. an unexpected behaviour change when migrating to Beam 2.30.0
> 2. potential serialization issues when using these String instances (Utf8
> instances don't implement Serializable)
> 3. an inconsistent API between {{AvroCoder}} and {{AvroSink}}/{{AvroSource}}
> (the latter still use {{ReflectDatum*}})
> (Original report on the [mailing
> list|https://lists.apache.org/x/thread.html/r5d0b975926cc4761f025ecd8df58a31e3f99e522296cc47d82ed5943@%3Cdev.beam.apache.org%3E]
> and [PR|https://github.com/apache/beam/pull/14410#issuecomment-880838488])
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
