Pavel Kruglov
Pavel Kruglov
> I've added a line 1 in users.xml under users/default, but it doesn't pick up. Did you run `system reload config`? It works in my installation. > As a separate...
> Reading the description I would assume the max_types values should refer to data types across the table, but not in one individual Insert statement. Is this an expected behavior?...
> DB::Exception: There is no supertype for types Dynamic, Dynamic(max_types=4). (NO_COMMON_TYPE) It's a bug, I will fix it. Thanks for testing it.
Yes, it's the same
Seems like it breaks compatibility for case `tuple(NULL)`: https://github.com/ClickHouse/ClickHouse/issues/66636 Before: ``` :) select toTypeName(tuple(NULL)) ┌─toTypeName(tuple(NULL))──┐ │ Tuple(Nullable(Nothing)) │ └──────────────────────────┘ ``` Now: ``` :) select toTypeName(tuple(NULL)) ┌─toTypeName((NULL))──────────────┐ │ Tuple(`NULL` Nullable(Nothing)) │...
Can you create a fix for it?
Failed tests are just flaky and not related to the changes in this PR
Test `03036_dynamic_read_subcolumns` was failing in master. It's already fixed so I updated the branch with recent changes to avoid its failing in the CI
The test `03036_dynamic_read_subcolumns` was broken in master, it's fixed now. Please, update the branch with recent changes
Test `03036_dynamic_read_subcolumns` was failing in master. It's already fixed so I updated the branch with recent changes to avoid its failing in the CI