cockroach icon indicating copy to clipboard operation
cockroach copied to clipboard

Role members table migration

Open RichardJCai opened this issue 3 years ago • 1 comments

RichardJCai avatar Aug 10 '22 22:08 RichardJCai

This change is Reviewable

cockroach-teamcity avatar Aug 10 '22 22:08 cockroach-teamcity

Bump @ajwerner @rafiss, thoughts on merging this? Given that it's the same as the system.users and system.role_options ones, I figure it's fine. Also it's somewhat strange to have the other two with id columns and not this.

RichardJCai avatar Aug 22 '22 18:08 RichardJCai

Given nothing uses this, do we get benefit from having migrated all the tables? Is it correct that nothing uses it?

Reviewed 6 of 24 files at r1, all commit messages. Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @rafiss and @rhu713)

Yeah we don't use it for reads right now.

RichardJCai avatar Aug 23 '22 15:08 RichardJCai

Superseded by #91517

andyyang890 avatar Nov 15 '22 15:11 andyyang890