zhaoyongjie commented on PR #27631: URL: https://github.com/apache/superset/pull/27631#issuecomment-2025796543
@betodealmeida > This PR introduces a new table called database_user_oauth2_tokens. Maybe a stupid question, but I think we should consider the implementation details. Why introduce a new table and a super complex foreign key topology? I believe this requirement is straightforward. We should store some tokens in the user table and keep the corresponding token in the user session. The metadata in Superset is extremely messy right now. Why do you guys always create new tables and create foreign keys for a super simple requirement? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
