Add upgrade step for changes in Id serialization

Description

None

Release Notes

None

relates to

Activity

Show:

Ali Anwar January 5, 2016 at 11:23 PM

Ok, I think we should do that as a part of this JIRA (since its a fairly small change), but its also necessary.

Terence Yim January 5, 2016 at 10:19 PM

The usage of {Id.Stream.toString()} in StreamService is easy to replace. It's just a string representation for the stream ID. To be backward compatible, it can be just a simply static function in StreamUtils to do so as the name needs to be shared between StreamService and the client.

Ali Anwar January 5, 2016 at 10:10 PM

We also use Id.Stream.toString() in the StreamService (I think, for stream scheduler):
https://github.com/caskdata/cdap/blob/release/3.3/cdap-data-fabric/src/main/java/co/cask/cdap/data/stream/service/DistributedStreamService.java#L479

Do you have some idea what change that would require?

Alvin Wang December 15, 2015 at 10:46 PM

Alvin Wang December 8, 2015 at 9:55 PM

StreamConsumerStateStore uses Id.Stream.toString(), so we'll have to perform an upgrade for that.

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Components

Fix versions

Priority

Created October 10, 2015 at 12:31 AM
Updated September 22, 2016 at 12:40 AM
Resolved January 14, 2016 at 1:16 AM