Tsv totals extremes 2
Fix for ClickHouseTabSeparatedProcessor with extremes and/or totals.
Thanks for your contribution @radmirnovii. That was indeed a missing feature. Will you also consider RowBinary processor?
By the way, the build break is irrelevant. It's caused by ClickHouse server, which was reported in #906. I'll enhance corresponding integration test as a workaround.
Will you also consider RowBinary processor?
Is it possible without parsing query?
Is it possible without parsing query?
Maybe not ;) Would be nice if we can support both formats.
Anyway, do you have time for additional changes? If not, I'll follow up to add test along with a few more changes to get it merged.
Anyway, do you have time for additional changes?
There is no empty spaces or any special values. Cannot find solution without query parsing. But i can add support for grpc version.
Sorry I think it's better to revisit this in 0.3.3 release, mainly for 3 reasons:
- it's a bit tricky for user to call the new methods to get totals and extremes - they have to know how many rows returned from the server first, and then call them in sequence right after reading the last row
- we'd better leverage progress stats for a more generic implementation for all protocols and formats
- unit tests and integration tests to prove the function works well