TimonKK / clickhouse

NodeJS client for ClickHouse
Apache License 2.0
221 stars 122 forks source link

Mismatch "database" field in docs #53

Closed AnatoliiVorobiov closed 4 years ago

AnatoliiVorobiov commented 4 years ago

You have written that field "database" should be in the inner object "config" in docs, but actually It is false.

In docs

const clickhouse = new ClickHouse({
         ....
    config: {
               .....
        database                                : 'my_database_name',
    },
});

Must be

const clickhouse = new ClickHouse({
         ....
         database                                : 'my_database_name',
    config: {
               .....
    },
});
TimonKK commented 4 years ago

There is not a bug. You can set database as you wish

AnatoliiVorobiov commented 4 years ago
const clickhouse = new ClickHouse({
         ....
    config: {
               .....
        database                                : 'my_database_name',
    },
});

This case is not working

TimonKK commented 4 years ago

Fix in 2.1.4