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

RoaringBitmap.toString produces large strings which create problems for debuggers #514

Open
richardstartin opened this issue Aug 21, 2021 · 4 comments

Comments

@richardstartin
Copy link
Member

The only good way to figure out what container types are used during development is to inspect in memory objects with a debugger, but the string returned by RoaringBitmap.toString is too large for this to be ergonomic, causing IDEs GC problems. This could be fixed with a much smaller string size.

@lemire
Copy link
Member

lemire commented Aug 21, 2021

I think that's more of a possible enhancement than a bug, no?

@richardstartin
Copy link
Member Author

richardstartin commented Aug 21, 2021

The label was added by accident. It’s neither an enhancement nor a bug in my opinion, just a removal of an ergonomic irritation.

@simeonpilgrim
Copy link
Contributor

simeonpilgrim commented Aug 21, 2021

too much memory, we limit the size

it almost seems like it there on purpose. But perhaps it print too much. I know i was finding if helpful the other day (for very small sets)

ah. My bad the above is from Roaring64Bitmap.

@richardstartin
Copy link
Member Author

There is a limit but it is too high

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

3 participants