jaegertracing / jaeger-clickhouse

Jaeger ClickHouse storage plugin implementation
Apache License 2.0
233 stars 50 forks source link

[Feature]: Allow changing TTL configuration on existing tables #120

Open ferpizza opened 1 year ago

ferpizza commented 1 year ago

Requirement

As a Jaeger Operator I want to be able to modify the TTL configuration of my tables/databases So that I can change these settings after the initial database creation

Problem

Currently, TTL is set ONLY on database creation. A change on TTL config values, after database creation, will not get propagated to the ddbb nor tables

Proposal

We can add sqlscripts to perform the TTL adjustment independenty from ddbb creation.

For the spans table, this new script will look similar to

ALTER TABLE {{.SpansTable}}
    MODIFY {{.TTLTimestamp}}

and we should make sure we run this script AFTER the one that creates the table, so it wont fail on new installs.

Open questions

No response