Skip to content

Feature request: Use iter.Seq to iterate over ScanIterator #3347

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

Open
khasanovbi opened this issue Apr 17, 2025 · 1 comment
Open

Feature request: Use iter.Seq to iterate over ScanIterator #3347

khasanovbi opened this issue Apr 17, 2025 · 1 comment

Comments

@khasanovbi
Copy link
Contributor

Support for native iterators has been added in go 1.23.
I suggest to add new Vals method that return keys iterator directly instead of receiving keys through Next, Val methods pair.

@ndyakov
Copy link
Member

ndyakov commented Apr 17, 2025

Hello @khasanovbi, I did respond in the PR. I am not against this, but let's not deprecate the current approach. Also, updating the go version results in failing tests. If you have the time to review and fix whatever is broken, we will gladly accept the version update. We were planning to do this with the next major version, but if we can figure out what is broken, I don't see why we can't update sooner.

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

No branches or pull requests

2 participants