[ https://issues.apache.org/jira/browse/BEAM-9502?focusedWorklogId=451886&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-451886 ]
ASF GitHub Bot logged work on BEAM-9502: ---------------------------------------- Author: ASF GitHub Bot Created on: 27/Jun/20 16:36 Start Date: 27/Jun/20 16:36 Worklog Time Spent: 10m Work Description: stale[bot] commented on pull request #11447: URL: https://github.com/apache/beam/pull/11447#issuecomment-650583617 This pull request has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 451886) Time Spent: 1h 40m (was: 1.5h) > SchemaCoder assigns random UUID, causes Dataflow's compatibility check to fail > ------------------------------------------------------------------------------ > > Key: BEAM-9502 > URL: https://issues.apache.org/jira/browse/BEAM-9502 > Project: Beam > Issue Type: Bug > Components: runner-dataflow, sdk-java-core > Reporter: Yaron Neuman > Priority: P3 > Labels: stale-assigned > Time Spent: 1h 40m > Remaining Estimate: 0h > > After fe4b7794, _Schema.equals_ comparing only the UUIDs for faster > comparison. > After 0b3b18c6 _SchemaCoder_ forcing random UUID when schema.uuid is null. > thus, when trying to update (--update) a Dataflow job with row schemas in > user-code, the compatibility check will fail because SchemaCoder produce > another random UUID. > > The user can set the UUID after creating the Schema, but not with > Schema.Builder > and I'm afraid most users, that are not aware to the internal > implementation, won't do that. > > In my branch, I added _.withUUID_ and _.withRandomUUID_ to _Schema.Builder_ > But I think a better solution will be to calculate the UUID based on the > schema itself. > any thoughts? > [~reuvenlax] > -- This message was sent by Atlassian Jira (v8.3.4#803005)