Implement WITH TOTALS supportSee original GitHub issue
What I’m suggesting is we add a getter
ClickHouseResultSet for those that really need it. This way,
WITH TOTALS is supported and at the same time is not among the actual ResultSet results, so no unexpected information or behaviour there.
Supposedly, it’ll be populated during one of
next() calls (if you have any better ideas - let me know) and the user will be able to extract it after iterating over ResultSet.
I’ll be happy to have a go at this and, if successful, send a PR.
- Created 6 years ago
- Comments:7 (7 by maintainers)
Top GitHub Comments
I think first is better since it’s more intuitive and scalable. I’ll start working on it tonight.
I see this issue has now been resolved, awesome 😃
Sorry that I wasn’t able to provide an implementation myself, it’s been a rough time.