Discussion: naming convention of session variables

[BugenZhao]

off-topic: We should really review the naming of these variables. 😕

• Some are prefixed with RW while others are not.
• Some are prefixed with STREAMING (or BATCH) while others are not.

For example, why SINK_DECOUPLE but not RW_STREAMING_SINK_DECOUPLE?

Originally posted by @BugenZhao in #13262 (comment)

[TennyZhuang]

We need to unify them, but before that, we have to introduce session variable alias to keep backward compatibility.

[fuyufjh]

Sounds good to me. Can you take a look? @yuhao-su

[yuhao-su]

Do we have any conclusion about what is the name convention?

[fuyufjh]

Do we have any conclusion about what is the name convention?

Indeed. Not yet, I guess.

Personally, I prefer the ones without RW_ prefix, because most session variables are specific to RW, and we don't plan to port many session variables from PG.

As for STREAMING/BATCH, I think it's optional and depends on the specific variables.

[github-actions]

This issue has been open for 60 days with no activity. Could you please update the status? Feel free to continue discussion or close as not planned.

[xxchan]

Maybe we should prioritize. I feel very confused and don't know which one to choose when I want to add a new config in #18749 🤣