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
tbls
handles 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.users
is displayed as justusers
analytics.page_views
is displayed as justpage_views
Proposed Improvement
Display PostgreSQL table names with their schema prefix, similar to how
tbls
handles it:auth.users
would be displayed asauth.users
analytics.page_views
would be displayed asanalytics.page_views
public.orders
would be displayed aspublic.orders
(or potentially justorders
if it's the default schema)Technical Analysis
From our codebase analysis:
Current Implementation:
converter.ts
Available Data Sources:
"public.users"
,"administrator.blogs"
Discussion Points
public
schema?schema.table
, highlighting, grouping)Potential Benefits
Potential Drawbacks
Questions for Discussion
public
schema - 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