Skip to content

Remove size_on_disk from Table and Schema#1395

Merged
dimitri-yatsenko merged 1 commit intomasterfrom
remove-size-on-disk
Feb 17, 2026
Merged

Remove size_on_disk from Table and Schema#1395
dimitri-yatsenko merged 1 commit intomasterfrom
remove-size-on-disk

Conversation

@dimitri-yatsenko
Copy link
Member

@dimitri-yatsenko dimitri-yatsenko commented Feb 11, 2026

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>
Copy link

@mweitzel mweitzel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 negative diffs

@dimitri-yatsenko dimitri-yatsenko merged commit 6327a82 into master Feb 17, 2026
7 checks passed
@dimitri-yatsenko dimitri-yatsenko deleted the remove-size-on-disk branch February 17, 2026 18:53
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.

2 participants