search

PNixx / clickhouse-activerecord

A Ruby database ActiveRecord driver for ClickHouse
MIT License
202 stars 100 forks source link

Improve DB::Exception error handling #165

Closed kyrylo closed 1 month ago

kyrylo commented 1 month ago

When the returned row contains DB::Exception, the adapter will erroneously raise an ActiveRecord::ActiveRecordError.

The current check for DB::Exception doesn't account for that.

Fixes https://github.com/PNixx/clickhouse-activerecord/issues/164

PNixx commented 1 month ago

Could you please add a test?

kyrylo commented 1 month ago

Where do I add it? Sorry, I am not familiar with the codebase here.