Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Store table row size in system table #1161

Closed
joshua-spacetime opened this issue Apr 26, 2024 · 3 comments · Fixed by #1319
Closed

Store table row size in system table #1161

joshua-spacetime opened this issue Apr 26, 2024 · 3 comments · Fixed by #1319
Assignees

Comments

@joshua-spacetime
Copy link
Collaborator

Ideally this would be a column in st_table, but I don't think we want to change the schema of st_table. It could also be a view over st_columns, but in that case we'd need to add an index to st_columns. The fastest, in terms of deriving the row size, is probably just to store it in a new system table. Perhaps the same one as #1158.

@joshua-spacetime joshua-spacetime self-assigned this Apr 26, 2024
@cloutiertyler
Copy link
Contributor

NOTE: This is presumably only the fixed len portion of the row?

@Shubham8287
Copy link
Contributor

Should we have ST_STATS for storing row size, row count, etc about a table?

@joshua-spacetime
Copy link
Collaborator Author

NOTE: This is presumably only the fixed len portion of the row?

Correct, specifically the BFLATN size.

Should we have ST_STATS for storing row size, row count, etc about a table?

Yes I think so.

@Shubham8287 Shubham8287 linked a pull request May 31, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants