[client-v2] Fix reading binary streams

[chernser]

Summary

Problem 1

Some streams return 0 when available() called even when data is available. Some streams may throw EOFException while read some may not. So only check for number of read bytes or result of read() really works.

Problem 2

If we want to wrap a stream of data into collection we must properly implement hasNext(). Because of problem#1 we can not do it easily

Solution

• Check read() result and throw EOFException
• Read data up-front. It will give some overhead, but it is simplest solution
• Logic is based on EOFException what is bad, but can be fixed in future with passing error handler to each read or returning tuples (what will be a memory overhead)
  • If EOFException is throw before reading next record - it assumes that if we at the start of row reaching end of stream indicates that no row is sent by server.

Checklist

Delete items not relevant to your PR:

• Unit and integration tests covering the common scenarios were added
• A human-readable description of the changes was provided to include in CHANGELOG
• For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

[Paultagoras]

Does this check to make sure it actually got a value? In theory it could skip this and pass...

[chernser]

Good point. Will fix.

[sonarcloud]

Quality Gate failed

Failed conditions
18.4% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud