Down with `SETTINGS index_granularity = 8192`!

[rschu1ze]

Whether it is

• on StackOverflow
• in Slack,
• or elsewhere

..., people continue to append SETTINGS index_granularity = 8192 to CREATE TABLE statements in technical articles, examples etc. Don't know why this is (perhaps the setting was mandatory in the past) but it is definitely not necessary: 8192 rows is the default already. I actually consider explicitly specifying the index granularity a bad practice as one could think 8192 has a special significance for the current SQL.

It would be nice to check our docs and also clickhouse.com/blog for similar mentions of SETTINGS index_granularity = 8192 and remove it.

[gingerwizard]

Its because people copy the replayed input from the client @rschu1ze and it outputs this. They rely on this for nice formatitng but the result is this setting is printed.

Agreed lets remove.

[rschu1ze]

I was just about to say the same.

create table tab (a String) engine = MergeTree order by tuple();

show table tab;

returns

statement: CREATE TABLE default.tab
(
    `a` String
)
ENGINE = MergeTree
ORDER BY tuple()
SETTINGS index_granularity = 8192

🤦