GitHub user Neilk1021 added a comment to the discussion: Proposal - Migrating 
User Authentication to a new table to support multiple log-in sources.

I favor keeping it to 2 tables (`users` and `auth_credentials`). From an 
application perspective, `local auth` and `OAuth` are heavily coupled in 
responsibility: both serve to verify credentials and yield a `user_id` so our 
servlets can issue a JWT. Modeling local auth as just another provider 
(provider_type = 'local') gives us a single, uniform interface across all 
authentication strategies.

While a 3-table design avoids NULL columns, Postgres represents NULLs via a 
tuple header bitmap, making them virtually free in terms of storage and 
performance. The minor tradeoff of using a DB CHECK constraint to enforce field 
rules is worth avoiding the schema fragmentation imo. 

GitHub link: 
https://github.com/apache/texera/discussions/6716#discussioncomment-17757961

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to