LuckPerms icon indicating copy to clipboard operation
LuckPerms copied to clipboard

New SQL schema

Open lucko opened this issue 6 years ago • 6 comments

Not going to be an "upgrade by default" thing - more of a manual upgrade.

Requirements:

  • sane design
  • not store server/world as a field - we just have contexts now
  • not store contexts as a json string

lucko avatar Sep 06 '19 18:09 lucko

I'm curious. What's wrong with the current schema?

BrainStone avatar Sep 06 '19 19:09 BrainStone

Main issue is the way contexts are stored.

lucko avatar Sep 06 '19 19:09 lucko

I see. Might as well store UUIDs as binary for additional optimization. Saves space and performance from my experience (binary indexes are a lot smaller and faster than text based ones)

BrainStone avatar Sep 06 '19 19:09 BrainStone

Yes, planning to as well!

lucko avatar Sep 06 '19 19:09 lucko

Maybe also instead of using uuids per each permission of a player, use the luckperms player table with ID

voodootje0 avatar Sep 06 '19 21:09 voodootje0

That would be even better. In principle at least.

I’d check which performs better. Because at that point I think performance is better than normalization.

BrainStone avatar Sep 07 '19 07:09 BrainStone