Rs.exec() doesn't reject with error messages when using streams about clickhouse HOT 8 CLOSED

Can you give me a example code with reproduce? You can use system.number as data

from clickhouse.

An easy way to reproduce is to try and insert to a table that doesn't exist:

const {ClickHouse} = require('clickhouse');

const ch = new ClickHouse({host: 'localhost'});

const stream = ch.insert('INSERT INTO BAD_TABLE').stream();

for (let i = 0; i < 10; i++) {
    stream.writeRow([i, 'test: '+i]);
}

stream.exec().catch(err => {
    console.log(err.code, err.message);
    // outputs: undefined 404: Not Found
});

stream.ws.on('response', res => {
    let body = '';
    res.on('data', data => body += data);
    res.on('end', () => {
        console.log(body);
        // outputs: Code: 60, e.displayText() = DB::Exception: Table default.BAD_TABLE doesn't exist. (version 20.3.8.53 (official build))
    });
})

from clickhouse.

If you exec insert as stream then Clickhouse doesn't return body, only error code 500. In last realises Clickhouse sends error code in header, but text still missed

from clickhouse.

When I run the example above it returns 404 and the body is:
Code: 60, e.displayText() = DB::Exception: Table default.BAD_TABLE doesn't exist. (version 20.3.8.53 (official build))

I can get the error message by manually listening for the response and processing the body, but it would be good if this was rejected by exec() like other errors.

from clickhouse.

Ok, I will patch code like your example

from clickhouse.

That's great, thank you!

from clickhouse.

I published [email protected] with your suggestion. Enjoy!

from clickhouse.

That's fantastic, thank you so much!

from clickhouse.