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

Support for entityId in legacy managers #456

Closed
DaanVanYperen opened this issue Sep 3, 2016 · 2 comments
Closed

Support for entityId in legacy managers #456

DaanVanYperen opened this issue Sep 3, 2016 · 2 comments

Comments

@DaanVanYperen
Copy link
Collaborator

DaanVanYperen commented Sep 3, 2016

First class int support for TagManager and GroupManager. PlayerManager, UuidEntityManager. Right now internally still use entities. GroupManager allocates bag for int representation, which is definitely not what we want.

non-runtime groups and tags might be better served with subscriptions and aspects, we can reflect that in the docs perhaps? @junkdog?

@DaanVanYperen
Copy link
Collaborator Author

IntBag getEntityIds(group) is no longer expensive by creating a shadow registry for the ids. Does make add/remove a bit more expensive.

@DaanVanYperen DaanVanYperen added this to the artemis-3.0.0 milestone Jul 15, 2018
@DaanVanYperen
Copy link
Collaborator Author

Closing, very little benefit and open since 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant