ClickHouseExceptionSpecifier returns ClickHouseUnknownException while underlying DB::Exception is actually known.

[PavelRuban]

As far as I understand ClickHouseExceptionSpecifier is trying to extract the code of DB::Exception from the error message generated by clickhouse-server. It should return new ClickHouseException initialized with that extracted code if extraction was successful.

Unfortunately it looks like actual error message format is different from the one getErrorCode() method is expecting:

So getErrorCode() returns -1 instead of actually received code. That means exception was not recognized, so specify() returns ClickHouseUnknownException instead of ClickHouseException(code).

Also I found the string of such format could be returned from Exception::getExceptionMessage (https://github.com/ClickHouse/ClickHouse/blob/master/src/Common/Exception.cpp)
Line 476: stream << "Code: " << e.code() << ". " << text;

So I suggest to look for dot as well as for comma in order to extract error code correctly.

[zhicwu]

This actually caused failures in integration test as well. It's been fixed and I'll come up with a PR to get it merged into develop branch.

[zhicwu]

Fixed in develop branch, will release in 0.3.2.