Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecated STM Support for Agents #13921

Closed
patriknw opened this issue Mar 12, 2014 · 9 comments
Closed

Remove deprecated STM Support for Agents #13921

patriknw opened this issue Mar 12, 2014 · 9 comments

Comments

@patriknw
Copy link
Member

@patriknw patriknw commented Mar 12, 2014

imported from https://www.assembla.com/spaces/akka/tickets/3921

@patriknw patriknw added this to the Rollins milestone Apr 13, 2014
@patriknw patriknw added the imported label Apr 13, 2014
@ktoso ktoso added the t:deprecation label Apr 15, 2014
@hepin1989
Copy link
Contributor

@hepin1989 hepin1989 commented Jan 21, 2015

will remove akka-agent?

@patriknw
Copy link
Member Author

@patriknw patriknw commented Jan 21, 2015

No (not that I'm aware of). This is only about removing the STM support from Agents.

@hepin1989
Copy link
Contributor

@hepin1989 hepin1989 commented Jan 21, 2015

then agent become only a safety cell for value right?

@patriknw patriknw modified the milestones: 2.5.0, 2.4.0 Aug 16, 2015
@rkuhn rkuhn removed the 1 - triaged label Mar 9, 2016
@ktoso ktoso mentioned this issue Sep 9, 2016
7 of 9 tasks complete
@ktoso
Copy link
Member

@ktoso ktoso commented Nov 30, 2016

Updated - we'll deprecate agents in 2.5 (not remove yet though)

@ktoso ktoso self-assigned this Nov 30, 2016
@ktoso ktoso added 3 - in progress and removed 1 - triaged labels Nov 30, 2016
@patriknw
Copy link
Member Author

@patriknw patriknw commented Nov 30, 2016

Is it too much work to remove the STM support and we just keep it until we remove all of agents?

@ktoso
Copy link
Member

@ktoso ktoso commented Nov 30, 2016

I'll give it a look - this we could remove right away in 2.5 you mean?

@ktoso
Copy link
Member

@ktoso ktoso commented Nov 30, 2016

I checked, it's a bit involved, not a few hours thing (I wouldn't be confident we provide the same semantics..).
So since we have it for 2.12 anyway (it's not blocking anything) I vote for keeping it until we remove.

@patriknw
Copy link
Member Author

@patriknw patriknw commented Nov 30, 2016

let's keep it, and remove all together

@patriknw patriknw removed this from the 2.5.0 milestone Nov 30, 2016
@patriknw
Copy link
Member Author

@patriknw patriknw commented Nov 30, 2016

meaning we can close this ticket

@patriknw patriknw closed this Nov 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants