[
https://issues.apache.org/jira/browse/BEAM-12628?focusedWorklogId=658583&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-658583
]
ASF GitHub Bot logged work on BEAM-12628:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 30/Sep/21 19:58
Start Date: 30/Sep/21 19:58
Worklog Time Spent: 10m
Work Description: udim commented on pull request #15622:
URL: https://github.com/apache/beam/pull/15622#issuecomment-931621498
Thank you for running the postcommit!
At least one of the test failures is due to another cherrypick (pubsublite),
which I'll probably be reverting.
--
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: 658583)
Time Spent: 5.5h (was: 5h 20m)
> 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, 2.33.0
> Reporter: Ryan Skraba
> Assignee: Claire McGinty
> Priority: P1
> Fix For: 2.34.0
>
> Time Spent: 5.5h
> 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)
