Skip to content

{175900538}: Fixing stat-size misreporting#4556

Closed
riverszhang89 wants to merge 1 commit into
bloomberg:mainfrom
riverszhang89:rename_stat_size
Closed

{175900538}: Fixing stat-size misreporting#4556
riverszhang89 wants to merge 1 commit into
bloomberg:mainfrom
riverszhang89:rename_stat_size

Conversation

@riverszhang89

Copy link
Copy Markdown
Contributor

After RENAME, the file-version-cache would hold onto the old file versions, which would cause stat-size to misreport the table size. This patch fixes it.

@roborivers roborivers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Coding style check: Error. ⚠.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 9/562 tests failed ⚠.

The first 10 failing tests are:
sc_timepart_logicalsc_generated
sc_resume
sc_transactional_rowlocks_generated
online_compaction
op_makerecord
unionpar_maxqueue
truncatesc
selectv_rcode

After RENAME, the file-version-cache would hold onto the old file
versions, which would cause stat-size to misreport the table size.
This patch fixes it.

Signed-off-by: Rivers Zhang <hzhang320@bloomberg.net>

@roborivers roborivers left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 2/402 tests failed ⚠.

The first 10 failing tests are:
analyze_exit_immediately

@riverszhang89

Copy link
Copy Markdown
Contributor Author

see #6042

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