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

Daisy Chen commented on STORM-2821:
-----------------------------------

[~revans2], I still see there are a bunch of places in Trident using 
TransactionAttempt class. In the comment of TransactionAttempt, it mentioned 
"This is dead code. It is retained to avoid breaking Kryo registration order." 

Will the change of order in SerializationFactory.java break anything?

And do you know why TransactionAttempt is deprecated and which class should be 
used to replace TransactionAttempt if there is one?

> Remove TransactionalTopologySupport.
> ------------------------------------
>
>                 Key: STORM-2821
>                 URL: https://issues.apache.org/jira/browse/STORM-2821
>             Project: Apache Storm
>          Issue Type: Task
>          Components: storm-client
>    Affects Versions: 2.0.0
>            Reporter: Robert Joseph Evans
>            Priority: Major
>
> Transactional Topologies have been in storm for a long time, but Trident 
> overrides it and the former is deprecated.  We should remove all of the code 
> that supports it and from the code base.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to