sql: implement CREATE SCHEMA ... AUTHORIZATION

[rohany]

Fixes #53559.

This commit adds the CREATE SCHEMA ... AUTHORIZATION command. When
authorization is provided, the target user is given ownership of the
schema. If the schema name is not provided, then the schema is named the
same name as the target role.

Release justification: low risk updates to new functionality
Release note (sql change): Support the CREATE SCHEMA ... AUTHORIZATION
command.

[cockroach-teamcity]

This change is 

[ajwerner]

Reviewed 6 of 6 files at r1.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @lucy-zhang, @rohany, and @solongordon)

---

pkg/sql/create_schema.go, line 73 at r1 (raw file):

Quoted 4 lines of code…

	// Check validity of the schema name.
	if err := schemadesc.IsSchemaNameValid(schemaName); err != nil {
		return err
	}

Should we do this check before the existence check?

---

pkg/sql/parser/sql.y, line 5297 at r1 (raw file):

// %Category: DDL
// %Text:
// CREATE SCHEMA [IF NOT EXISTS] <schemaname> [AUTHORIZATION <rolename>]

nit: should this be CREATE SCHEMA [IF NOT EXISTS] { <schemaname> | [ [<schemaname>] AUTHORIZATION <rolename>] }

[rohany]

Done, addressed both comments

bors r+

[craig]

Build failed (retrying...):

• GitHub CI (Cockroach)

[ajwerner]

Reviewable status: complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @ajwerner, @lucy-zhang, @rohany, and @solongordon)

---

pkg/sql/create_schema.go, line 59 at r2 (raw file):

	// Check validity of the schema name.
	if err := schemadesc.IsSchemaNameValid(schemaName); err != nil {

seems like re-arranging this broke some tests 😬

[ajwerner]

Reviewable status: complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @ajwerner, @lucy-zhang, @rohany, and @solongordon)

---

pkg/sql/parser/sql.y, line 5297 at r2 (raw file):

// %Category: DDL
// %Text:
// CREATE SCHEMA [IF NOT EXISTS] { <schemaname> | [<schemaname>] [AUTHORIZATION <rolename>] }

also, I don't think this means the same thing as what I wrote. This seems to imply that you could omit both elements of the RHS. I think you could do:

CREATE SCHEMA [IF NOT EXISTS] { <schemaname> | [<schemaname>] AUTHORIZATION <rolename> }

[rohany]

Ah yeah you're right, can't have brackets around the AUTHORIZATION ...

[knz]

this is failing ci - removing from the bors queue

bors r-

[craig]

Canceled.

[rohany]

Updated the help message.

Seems like we need to keep the name check after the existence check.

[rohany]

bors r+

[rohany]

bors r-

[craig]

Canceled.

[rohany]

@ajwerner can you stamp this?

[thoszhang]

Reviewed 4 of 6 files at r1, 2 of 2 files at r3.
Reviewable status: complete! 0 of 0 LGTMs obtained (and 1 stale) (waiting on @rohany and @solongordon)

[ajwerner]

Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @rohany and @solongordon)

[rohany]

bors r+

[craig]

This PR was included in a batch that was canceled, it will be automatically retried

[craig]

Build succeeded:

• GitHub CI (Cockroach)