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.

Sign up for GitHub

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

Jump to bottom

[FEA] Use BitSet + cleanup column tracking in device/host stores #9121

Open
abellina opened this issue Aug 28, 2023 · 1 comment
Open

[FEA] Use BitSet + cleanup column tracking in device/host stores #9121

abellina opened this issue Aug 28, 2023 · 1 comment
Assignees
@abellina
Labels
tech debt

Comments

@abellina
Copy link
Collaborator

Currently column tracking in device and host stores is done via a ConcurrentHashMap. @jlowe suggested we could use a BitSet instead. The code is duplicated right now between device and host stores, so this task should also clean this up and commonize things as much as possible.
@abellina abellina added feature request New feature or request ? - Needs Triage Need team to review and classify tech debt labels Aug 28, 2023
@abellina abellina self-assigned this Aug 28, 2023
@abellina abellina mentioned this issue Aug 28, 2023
Closed
@abellina
Copy link
Collaborator Author

With this, we should also make clear the deduping that is being done in order to attach event handlers to unique columns (#9098 (comment))

@abellina abellina mentioned this issue Aug 28, 2023
Merged
@sameerz sameerz removed feature request New feature or request ? - Needs Triage Need team to review and classify labels Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abellina abellina

Labels
tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@abellina @sameerz