- Notifications
You must be signed in to change notification settings - Fork975
Add streaming of :many viachan and/or callbackfunc#1229
-
I noticed that Another idea is a |
BetaWas this translation helpful?Give feedback.
All reactions
Given the newiter package proposal (golang/go#61897) is likely to move forward, I think we won't want to add functionality until we know how to integrate with that package.
Replies: 7 comments 4 replies
-
A |
BetaWas this translation helpful?Give feedback.
All reactions
-
True, the callback would allow one to implement streaming. I thought it's a bit much ceremony at the moment to do it in every callback, with |
BetaWas this translation helpful?Give feedback.
All reactions
-
Today I was looking for a way to make sqlc to support streaming a large amount of data from a query to gRPC. I found the related issues and this discussion. To me a simple callback option seems like the best way to do it. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I've just ran into this. I'm working on some maintenance work on a database and I don't know how many rows I get back. Could be 1K or 1M. I really love sqlc and the correctness it brings. It would be really amazing to have a callback that would get fired off for each row. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Given the new |
BetaWas this translation helpful?Give feedback.
All reactions
👍 1🎉 1❤️ 1
-
Hi, what do you think about the updates from go 1.23? (Found this discussion here via google, not sure if I should create a new one?) |
BetaWas this translation helpful?Give feedback.
All reactions
👍 4
-
I see there is a PR for iterator#3631 |
BetaWas this translation helpful?Give feedback.
All reactions
-
What's the status here? |
BetaWas this translation helpful?Give feedback.
All reactions
-
imho, I think this ticket should be closed. Iterators is the obvious way forward and since there is quite good progress in a PR, we should direct ppl there to evaluate the PR. |
BetaWas this translation helpful?Give feedback.
All reactions
-
Since theiterator PR hasn't been merged yet, I'm curious about current best practices for handling large table queries. |
BetaWas this translation helpful?Give feedback.
All reactions
-
I think pagination is the way to go for now. |
BetaWas this translation helpful?Give feedback.
All reactions
This discussion was converted from issue #1218 on October 09, 2021 17:53.