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

Issue #441: Fix data change on optimize #442

Merged
merged 2 commits into from
Oct 24, 2024

Conversation

JosepSampe
Copy link
Member

Description

Fixes issue #441 by setting dataChange = False during optimize()

Type of change

Bug Fix

@JosepSampe JosepSampe requested a review from osopardo1 October 22, 2024 16:20
@JosepSampe JosepSampe self-assigned this Oct 22, 2024
@Jiaweihu08 Jiaweihu08 requested review from Jiaweihu08 and removed request for Jiaweihu08 October 23, 2024 10:47
@JosepSampe JosepSampe requested a review from osopardo1 October 23, 2024 12:06
@osopardo1 osopardo1 merged commit 0b2324e into Qbeast-io:refactor Oct 24, 2024
1 check passed
JosepSampe added a commit to JosepSampe/qbeast-spark that referenced this pull request Oct 24, 2024
* Set dataChange to false on Table.optimize()
@JosepSampe JosepSampe deleted the fix-data-change branch October 24, 2024 15:13
@JosepSampe JosepSampe restored the fix-data-change branch October 24, 2024 15:20
@JosepSampe JosepSampe deleted the fix-data-change branch October 24, 2024 15:47
osopardo1 pushed a commit that referenced this pull request Oct 28, 2024
* Issue #417: Abstract the Qbeast Snapshot Module (#411)

* Issue #418: Abstract PreCommitHook and StagingDataManager (#421)

* Issue #418: Abstract RollupDataWriter and QbeastStats  (#423)

* Issue #419: Code Reorganization (#425)

* Issue #420: Create Separate Modules (#427)

* Issue #398: Fix small overhead added during the refactoring (#436)

* Issue #441: Fix data change on optimize (#442)
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