Skip to content

Conversation

@dimitri-yatsenko
Copy link
Member

Summary

  • Remove Table.size_on_disk property
  • Remove Schema.size_on_disk property
  • Remove associated tests

Rationale

  • MySQL-specific: Uses SHOW TABLE STATUS and information_schema queries incompatible with PostgreSQL support
  • Incomplete in 2.0: Reports only relational DB size, not object storage where large scientific data resides
  • Accuracy issues: MySQL statistics can be stale without ANALYZE TABLE
  • Low utility: Trivial for users to implement directly if needed
  • Undocumented: No documentation in datajoint-docs

Test plan

  • Removed tests for both methods
  • CI passes

🤖 Generated with Claude Code

- Remove Table.size_on_disk property
- Remove Schema.size_on_disk property
- Remove associated tests

Rationale:
- MySQL-specific queries (SHOW TABLE STATUS, information_schema)
  incompatible with PostgreSQL support
- Incomplete picture in 2.0: reports only relational DB size,
  not object storage where large scientific data resides
- MySQL statistics can be stale without ANALYZE TABLE
- Trivial for users to implement directly if needed
- Undocumented feature with limited utility

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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 this pull request may close these issues.

1 participant