dev/reference: update the error code

[jackysp]

Signed-off-by: Shuaipeng Yu jackysp@gmail.com

What is changed, added or deleted?

Update the error code of dev/reference

What is the related PR or file link(s)?

• Reference link(s):
• This PR is to align with:
• N/A (not applicable)

Which TiDB version(s) does your changes apply to?

• All active versions: dev, v3.0, v2.1, v3.1
• dev (the latest development version)
• v3.0 (TiDB 3.0 versions)
• v2.1 (TiDB 2.1 versions)
• v3.1 (TiDB 3.1 versions)
• v4.0 (TiDB 4.0 versions)
• All active and inactive versions
• N/A (not applicable)

Note: If your changes apply to multiple TiDB versions, make sure you update the documents in the corresponding version folders such as "dev", "v3.0", "v2.1" and "v3.1" in this PR.

• Updated one version first. Will update other versions after I get two LGTMs.

[ichn-hu]

Perhaps s/错误/不匹配 will be more informative

[ichn-hu]

I suggest remove 错误

[ichn-hu]

It seems that if the plugin doesn't have version number will also cause this error

// plugin.go:122
		for component, reqVer := range p.RequireVersion {
			if ver, ok := tiPlugins.versions[component]; !ok || ver < reqVer {
				return errRequireVersionCheckFail.GenWithStackByArgs(p.Name, component, reqVer, ver)
			}
		}

So 载入的插件未指定版本或指定的版本过低 would be more clear

[ichn-hu]

I doubt this error is caused by system variable, it seems it is because of empty result set, see

// getExecRet executes restricted sql and the result is one column.
// It returns a string value.
func (s *session) getExecRet(ctx sessionctx.Context, sql string) (string, error) {
	rows, fields, err := s.ExecRestrictedSQL(sql)
	if err != nil {
		return "", err
	}
	if len(rows) == 0 {
		return "", executor.ErrResultIsEmpty
	}
	d := rows[0].GetDatum(0, &fields[0].Column.FieldType)
	value, err := d.ToString()
	if err != nil {
		return "", err
	}
	return value, nil
}

[jackysp]

I'll remove it, it will be replaced by ErrUnknownSystemVar in GetGlobalSysVar.

[ichn-hu]

s/算子/器

[ichn-hu]

Or 执行器构建失败

[ichn-hu]

Also ErrBatchInsertFail = 8119 is missing, is it intentional?

[jackysp]

Also ErrBatchInsertFail = 8119 is missing, is it intentional?

The docs of batch insert have already deleted in #2194. So we should not expose its error code.

[ichn-hu]

LGTM

[bb7133]

Why 8208 and 8209 is not listed? Is there any reason? I think we can simply list them all here.

[jackysp]

They are not used anywhere.

[lilin90]

LGTM