Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document missing system variables #3155

Closed
20 of 22 tasks
ghost opened this issue Jul 5, 2020 · 15 comments · Fixed by #10872, #3907 or #9943
Closed
20 of 22 tasks

Document missing system variables #3155

ghost opened this issue Jul 5, 2020 · 15 comments · Fixed by #10872, #3907 or #9943
Assignees

Comments

@ghost
Copy link

ghost commented Jul 5, 2020

Change Request

This repository is ONLY used to solve issues related to DOCS.
For other issues (related to TiDB, PD, etc), please move to other repositories.

Please answer the following questions before submitting your issue. Thanks!

  1. Describe what you find is inappropriate or missing in the existing docs.

In #3152 I compared the current output of SHOW GLOBAL VARIABLES to what is in the documentation. The following system variables are not documented:

  1. Describe your suggestion or addition.

Please explain what they do :-)

  1. Provide some reference materials (documents, websites, etc) if you could.
@yikeke
Copy link
Contributor

yikeke commented Jul 7, 2020

This PR in docs-cn might cover some of the variables of your list. I'm translating this PR and the English version is expected to be merged by the end of this week.

@yikeke yikeke self-assigned this Jul 7, 2020
@ghost
Copy link
Author

ghost commented Jul 7, 2020

Yes, it does look like there are a lot in that PR! Let me know if there is anything I can do to help merge the two sets of changes better.

@yikeke
Copy link
Contributor

yikeke commented Jul 7, 2020

Yes, it does look like there are a lot in that PR! Let me know if there is anything I can do to help merge the two sets of changes better.

Sure, I'll see what I can do to reduce the conflicts between the two sets of changes.

@yikeke yikeke removed their assignment Jul 9, 2020
@yikeke
Copy link
Contributor

yikeke commented Jul 10, 2020

Yes, it does look like there are a lot in that PR! Let me know if there is anything I can do to help merge the two sets of changes better.

The English version of that PR is opened: #3215. Please take a look and let's figure out a way to merge the two sets of changes better. @nullnotnil

@ghost
Copy link
Author

ghost commented Jul 10, 2020

The English version of that PR is opened: #3215. Please take a look and let's figure out a way to merge the two sets of changes better. @nullnotnil

I will let #3215 merge first and then rebase my changes. I think this makes the most sense.

@ghost
Copy link
Author

ghost commented Jul 10, 2020

I have updated this issue to remove what was documented in #3215

@ghost
Copy link
Author

ghost commented Sep 4, 2020

It looks like documenting tidb_dml_batch_size is important because of a behavior change in master. Previously I used LOAD DATA to import the example database in the docs. Because of pingcap/tidb#18807 the example now breaks by default due to a transaction too large error.

I can increase the transaction size, or set tidb_dml_batch_size to a non-zero value, but the error message is not entirely clear. I think the new behavior is better, so I suggest updating https://docs.pingcap.com/tidb/dev/sql-statement-load-data (which is now out of date for master as well) to say that there was a behavior change.

I will send a PR shortly.

cc @tina77fritz @jackysp @cfzjywxk

@github-actions
Copy link

This issue is stale because it has been open for 60 days with no activity. If no comment has been made and the lifecycle/stale label is not removed, this issue will be closed in 15 days.

@github-actions github-actions bot added the lifecycle/stale This issue has been open for 60 days with no activity. label Nov 11, 2020
@ghost ghost removed the lifecycle/stale This issue has been open for 60 days with no activity. label Nov 11, 2020
@github-actions
Copy link

This issue is stale because it has been open for 60 days with no activity. If no comment has been made and the lifecycle/stale label is not removed, this issue will be closed in 15 days.

@github-actions github-actions bot added the lifecycle/stale This issue has been open for 60 days with no activity. label Jan 11, 2021
@morgo
Copy link
Contributor

morgo commented Jan 11, 2021

/unlabel lifecycle/stale

@ti-srebot ti-srebot removed the lifecycle/stale This issue has been open for 60 days with no activity. label Jan 11, 2021
@morgo
Copy link
Contributor

morgo commented Jan 11, 2021

The options for character set and collation need documentation:

| character_set_client                                   | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_connection                               | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_database                                 | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_filesystem                               | binary                                                                                                                                                                                                                                                                                                                                           |
| character_set_results                                  | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_server                                   | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_system                                   | utf8                                                                                                                                                                                                                                                                                                                                             |
| collation_connection                                   | utf8mb4_bin                                                                                                                                                                                                                                                                                                                                      |
| collation_database                                     | utf8mb4_bin                                                                                                                                                                                                                                                                                                                                      |
| collation_server                                       | utf8mb4_bin                                                                                                                                                                                                                

@TomShawn
Copy link
Contributor

The options for character set and collation need documentation:

| character_set_client                                   | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_connection                               | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_database                                 | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_filesystem                               | binary                                                                                                                                                                                                                                                                                                                                           |
| character_set_results                                  | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_server                                   | utf8mb4                                                                                                                                                                                                                                                                                                                                          |
| character_set_system                                   | utf8                                                                                                                                                                                                                                                                                                                                             |
| collation_connection                                   | utf8mb4_bin                                                                                                                                                                                                                                                                                                                                      |
| collation_database                                     | utf8mb4_bin                                                                                                                                                                                                                                                                                                                                      |
| collation_server                                       | utf8mb4_bin                                                                                                                                                                                                                

@wjhuang2016 PTAL

@dveeden
Copy link
Contributor

dveeden commented Jun 24, 2021

@morgo is this still an issue after your recent improvements?

@morgo
Copy link
Contributor

morgo commented Jun 24, 2021

Yes. It is worse than this :( This list was created by filtering on SHOW VARIABLES LIKE 'tidb_%'.

Here is a more accurate list of what is not documented but should be:

case variable.BlockEncryptionMode, variable.CharacterSetClient, variable.CharacterSetConnection, variable.CharsetDatabase,
		variable.CharacterSetFilesystem, variable.CharacterSetResults, variable.CharacterSetServer, variable.CollationConnection, variable.CollationDatabase,
		variable.CollationServer, variable.CTEMaxRecursionDepth, variable.DataDir, variable.DefaultWeekFormat, variable.ErrorCount, variable.GroupConcatMaxLen,
		"have_openssl", "have_ssl", variable.InitConnect, "last_insert_id", variable.LowerCaseTableNames, variable.MaxAllowedPacket, variable.MaxConnections,
		variable.MaxPreparedStmtCount, variable.PluginDir, variable.PluginLoad, variable.SQLLogBin, "ssl_ca", "ssl_cert", "ssl_key", variable.TiDBAnalyzeVersion,
		variable.TiDBBatchCommit, variable.TiDBBatchDelete, variable.TiDBBatchInsert, variable.TiDBEnableChangeColumnType, variable.TiDBEnableChangeMultiSchema,
		variable.TiDBEnableDynamicPrivileges, variable.TiDBEnableExchangePartition, variable.TiDBEnableExtendedStats, variable.TiDBEnablePointGetCache,
		variable.TiDBEnableStreaming, variable.TiDBGuaranteeLinearizability, variable.WarningCount, variable.TiDBTxnScope, variable.TiDBTxnReadTS,
		variable.TxnIsolationOneShot, variable.Timestamp, variable.TiDBLastQueryInfo, variable.TiDBLastTxnInfo,
		variable.TiDBMemQuotaHashJoin, variable.TiDBStreamAggConcurrency, variable.TiDBTrackAggregateMemoryUsage, variable.TiDBOptBCJ,
		variable.TiDBOptConcurrencyFactor, variable.TiDBOptCopCPUFactor, variable.TiDBEnableIndexMergeJoin,
		variable.TiDBMemQuotaIndexLookupJoin, variable.TiDBMemQuotaIndexLookupReader, variable.TiDBMemQuotaMergeJoin,
		variable.TiDBEnableAlterPlacement, variable.TiDBSlowLogMasking, variable.TiDBShardAllocateStep, variable.TiDBMemQuotaTopn,
		variable.TiDBMemQuotaSort, variable.TiDBMergeJoinConcurrency, variable.TiDBOptCPUFactor, variable.TiDBOptDescScanFactor,
		variable.TiDBOptDiskFactor, variable.TiDBOptJoinReorderThreshold, variable.TiDBOptMemoryFactor, variable.TiDBOptNetworkFactor,
		variable.TiDBOptScanFactor, variable.TiDBOptTiFlashConcurrencyFactor, variable.TiDBOptimizerSelectivityLevel, variable.TiDBPartitionPruneMode,
		variable.TiDBOptSeekFactor, variable.LogBin, "license", variable.TiDBEnableTopSQL, variable.TiDBTopSQLAgentAddress, variable.TiDBTopSQLPrecisionSeconds,
		variable.TiDBTopSQLMaxStatementCount, variable.TiDBEnableGlobalTemporaryTable, variable.TiDBEnablePipelinedWindowFunction, variable.TiDBOptCartesianBCJ:

@shichun-0415
Copy link
Contributor

shichun-0415 commented Feb 2, 2023

@till now, all the system variables mentioned above have been documented and the related PRs have been attached for reference. Two variables tidb_enable_radix_join and tidb_slow_log_masking are not found in https://github.com/pingcap/tidb/blob/master/sessionctx/variable/sysvar.go and might have been removed. Therefore, this issue can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants