Replies: 2 comments
-
|
📝
|
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Background
We received feedback from tokuhirom-san on Twitter suggesting that our current approach to handling PostgreSQL schemas might not be optimal for PostgreSQL users. The feedback indicates that PostgreSQL schemas should be displayed as part of the table names (similar to how
tblshandles it), rather than being stripped out during parsing.Original feedback: https://x.com/tokuhirom/status/1941857822784029145
Reference: tbls approach to PostgreSQL schemas
Current Behavior
Currently, Liam ERD strips schema qualifiers from PostgreSQL table names during parsing:
auth.usersis displayed as justusersanalytics.page_viewsis displayed as justpage_viewsProposed Improvement
Display PostgreSQL table names with their schema prefix, similar to how
tblshandles it:auth.userswould be displayed asauth.usersanalytics.page_viewswould be displayed asanalytics.page_viewspublic.orderswould be displayed aspublic.orders(or potentially justordersif it's the default schema)Technical Analysis
From our codebase analysis:
Current Implementation:
converter.tsAvailable Data Sources:
"public.users","administrator.blogs"Discussion Points
publicschema?schema.table, highlighting, grouping)Potential Benefits
Potential Drawbacks
Questions for Discussion
publicschema - show it or hide it?Related Work
frontend/packages/db-structure/src/parser/sql/postgresql/Looking forward to hearing thoughts from the community, especially from PostgreSQL users who can share their preferences and use cases.
Beta Was this translation helpful? Give feedback.
All reactions