profile
viewpoint

nvartolomei/dotfiles 5

nv ♥️ ~/

nvartolomei/ant-design 0

:ant: One design language

nvartolomei/awesome-bigdata 0

A curated list of awesome big data frameworks, ressources and other awesomeness.

nvartolomei/awesome-data-engineering 0

A curated list of data engineering tools for software developers

nvartolomei/awesome-interview-questions 0

:octocat: A curated awesome list of lists of interview questions. Feel free to contribute! :mortar_board:

nvartolomei/backoff 0

The exponential backoff algorithm in Go (Golang).

nvartolomei/bimg 0

Small Go package for fast high-level image processing using libvips via C bindings

nvartolomei/clef-employee-handbook 0

An employee handbook built for inclusion

nvartolomei/ClickHouse 0

https://github.com/ClickHouse/ClickHouse

nvartolomei/clickhouse-grafana 0

Clickhouse datasource for grafana

startedplasma-umass/coz

started time in 19 hours

issue commentClickHouse/ClickHouse

clickhouse Distributed ENGINE ALTER corresponding local tables?

There is no support for this. Distributed tables are independent from the tables it points to.

2416210017

comment created time in a day

startedquickwit-inc/quickwit

started time in 4 days

issue commentClickHouse/ClickHouse

Automatically choose the size of bloom filter index

@alexey-milovidov could you give some pointers how a bloom filter can be resized and resulting false positivity rate obtained from it?

alexey-milovidov

comment created time in 6 days

issue commentClickHouse/ClickHouse

how to find out the last access_time or query_time of a table?

system.query_log is probably your best bet.

DESCRIBE TABLE  system.query_log FORMAT JSONEachRow;

...
{"name":"databases","type":"Array(LowCardinality(String))","default_type":"","default_expression":"","comment":"","codec_expression":"","ttl_expression":""}
{"name":"tables","type":"Array(LowCardinality(String))","default_type":"","default_expression":"","comment":"","codec_expression":"","ttl_expression":""}
...
windylcx

comment created time in 6 days

issue commentClickHouse/ClickHouse

How can I force to handle all rows in one time execution of IExecutableFunction::executeImpl()

This sounds like an aggregate function rather than an executable function https://github.com/ClickHouse/ClickHouse/tree/master/src/AggregateFunctions.

Alex-Cheng

comment created time in 8 days

delete branch nvartolomei/ClickHouse

delete branch : nv/test-odbc

delete time in 8 days

issue commentClickHouse/ClickHouse

clickhouse data cannot move part

Are you able to reproduce the issue? Can you provide an example?

sunny19930321

comment created time in 11 days

startedlogseq/logseq

started time in 12 days

issue commentClickHouse/ClickHouse

clickhouse data cannot move part

Duplicate #33428

sunny19930321

comment created time in 12 days

issue commentClickHouse/ClickHouse

clickhouse data cannot move part

It looks like you are using a tool to automatically move parts? Curious what it is? Is it on GitHub? Wondering if it is missing some documentation.

Parts don’t have uuids by default yet. I haven’t figured out how to enable this by default without a breaking change yet.

First you need to enable the assign part uuids setting in clickhouse config and then new parts will have uuids and will become movable. To force assignment of uuids to old parts you need to trigger a mutation on the table and wait for it to recreate the parts. The mutation could be something really simple like ‘update set col = col where 1 = 1’. I think a simple ‘alter add column’ could work as well, don’t remember where this recreates parts today.

sunny19930321

comment created time in 12 days

push eventnvartolomei/ClickHouse

Nicolae Vartolomei

commit sha 50e6f729d1e01fc3d3a1e3e31cbbafe5e84d81e7

Try ping connection on unexpected errors Workaround for https://www.postgresql.org/message-id/CAC5UznEsaG75-Q89z4Ypz1q48UT7O%2B1U7drUPt6Xs%2Bma9_hEGQ%40mail.gmail.com

view details

push time in 13 days

Pull request review commentClickHouse/ClickHouse

Adding /replicas_status request

 Web UI can be accessed here: `http://localhost:8123/play`. ![Web UI](../images/play.png)  -In health-check scripts use `GET /ping` request. This handler always returns “Ok.” (with a line feed at the end). Available from version 18.12.13.+In health-check scripts use `GET /ping` or `GET /replicas_status` request. This handler always returns “Ok.” (with a line feed at the end). Available from version 18.12.13.

replica_status does a bit more than just returning Ok, and doesn't always return Ok

https://github.com/ClickHouse/ClickHouse/blob/04be5437d94e397c0d379bc648c56950f90f09ef/src/Server/ReplicasStatusHandler.cpp#L25

peter279k

comment created time in 13 days

PullRequestReviewEvent

pull request commentClickHouse/ClickHouse

Add test for broken connection

Reported the issue to psqlodbc mailing list. Maybe we’ll get some feedback there. https://www.postgresql.org/message-id/CAC5UznEsaG75-Q89z4Ypz1q48UT7O%2B1U7drUPt6Xs%2Bma9_hEGQ%40mail.gmail.com

nvartolomei

comment created time in 16 days

delete branch nvartolomei/ClickHouse

delete branch : nv/allow-build-wo-libraries

delete time in 19 days

delete branch nvartolomei/ClickHouse

delete branch : nv/fix-old-ccache-again

delete time in 19 days

push eventnvartolomei/ClickHouse

Nicolae Vartolomei

commit sha fd1e9ffa664a4eb7f9bdd3184308d17a069b9d4e

Return early if azure blob storage is not used, otherwise it might with fail with irrelevant errors

view details

push time in 20 days

pull request commentClickHouse/ClickHouse

Add test for broken connection

Suggestions for fix are welcome. Fixing psqlodbc would be great, but too time-consuming and unlikely to reach the masses soon. There is SQL_ATTR_CONNECTION_DEAD but this one doesn't seem to be set/returned correctly by psqlodbc either.

Workaround would be something like this or matching on exception string message...

if (e.state().starts_with("HY00"))
        {
            try 
            {
                just_execute(connection_holder->get(), "SELECT 1");
            } catch (...)
            {
                connection_holder->updateConnection();
                return query_func(connection_holder->get());
            }
        }
nvartolomei

comment created time in 20 days

PR opened ClickHouse/ClickHouse

Return early if azure blob storage is not used, otherwise it might with fail with irrelevant errors

Like missing internal ssl library when ENABLE_LIBRARIRES=0

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)
+2 -0

0 comment

1 changed file

pr created time in 20 days

create barnchnvartolomei/ClickHouse

branch : nv/allow-build-wo-libraries

created branch time in 20 days

Pull request review commentClickHouse/ClickHouse

Fix ccache with ENABLE_CHECK_HEAVY_BUILDS (ccache 4.0 and 4.1 only affected)

 if (CCACHE_FOUND AND NOT COMPILER_MATCHES_CCACHE)       # - 4.0+ will ignore SOURCE_DATE_EPOCH environment variable.       if (CCACHE_VERSION VERSION_GREATER_EQUAL "4.2")          message(STATUS "ccache is 4.2+ no quirks for SOURCE_DATE_EPOCH required")+         set(LAUNCHER ${CCACHE_FOUND})

Hope this works https://github.com/ClickHouse/ClickHouse/pull/33306

azat

comment created time in 20 days

PullRequestReviewEvent

PR opened ClickHouse/ClickHouse

Fix launcher not being defined for ccache < 4.0 and slightly cleanup

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)
+3 -7

0 comment

1 changed file

pr created time in 20 days

create barnchnvartolomei/ClickHouse

branch : nv/fix-old-ccache-again

created branch time in 20 days

create barnchnvartolomei/ClickHouse

branch : nv/test-odbc

created branch time in 20 days

PR opened ClickHouse/ClickHouse

Add test for broken connection

This seems to be failing. It seems that psqlodbc driver doesn't correctly propagate communication failure error status after connection already was established.

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)
+2 -0

0 comment

1 changed file

pr created time in 20 days

PullRequestReviewEvent

Pull request review commentClickHouse/ClickHouse

Fix ccache with ENABLE_CHECK_HEAVY_BUILDS (ccache 4.0 and 4.1 only affected)

 if (CCACHE_FOUND AND NOT COMPILER_MATCHES_CCACHE)       # - 4.0+ will ignore SOURCE_DATE_EPOCH environment variable.       if (CCACHE_VERSION VERSION_GREATER_EQUAL "4.2")          message(STATUS "ccache is 4.2+ no quirks for SOURCE_DATE_EPOCH required")+         set(LAUNCHER ${CCACHE_FOUND})

I think this needs to be moved to line 34, otherwise LAUNCHER is undefined for ccache < 4.0

azat

comment created time in 20 days

startedlinuxerwang/goroutine-inspect

started time in 21 days

more