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

Add test for dropping a fixed attribute and adding it back as var-sized #2083

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

kounelisagis
Copy link
Member

@kounelisagis kounelisagis commented Oct 9, 2024

Following TileDB-Inc/TileDB#5321 and TileDB-Inc/TileDB#5362, let's add a TileDB-Py test.


[sc-57039]

@kounelisagis kounelisagis changed the title [2.26.3] Add test for dropping a fixed attribute and adding it back as var-sized [2.27.0] Add test for dropping a fixed attribute and adding it back as var-sized Oct 11, 2024
@ihnorton ihnorton requested review from bekadavis9 and removed request for bekadavis9 October 15, 2024 01:58
@kounelisagis kounelisagis force-pushed the agis/schema-evolution-fixed-to-var branch 2 times, most recently from 5293c1f to 0b1c770 Compare October 16, 2024 10:04
@kounelisagis kounelisagis changed the title [2.27.0] Add test for dropping a fixed attribute and adding it back as var-sized Add test for dropping a fixed attribute and adding it back as var-sized Oct 16, 2024
@kounelisagis kounelisagis force-pushed the agis/schema-evolution-fixed-to-var branch from c62b9b8 to cae6ff3 Compare November 12, 2024 12:33
bekadavis9 added a commit to TileDB-Inc/TileDB that referenced this pull request Nov 12, 2024
Schema evolution bug fix: Reads no longer fail after dropping a fixed
attribute and adding it back as var-sized.

Followup to #5321 to address a previously-missed case caught by
[TileDB-Inc/TileDB-Py/#2083](TileDB-Inc/TileDB-Py#2083)

[sc-55085]

---
TYPE: BUG
DESC: Schema evolution bug fix: Reads no longer fail after dropping a
fixed attribute and adding it back as var-sized, part 2.
Copy link
Member

@ihnorton ihnorton left a comment

Choose a reason for hiding this comment

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

LGTM after addressing the fill value check request from @bekadavis9

@kounelisagis kounelisagis marked this pull request as ready for review November 15, 2024 15:36
@kounelisagis kounelisagis merged commit a2f8ccc into dev Nov 15, 2024
30 checks passed
@kounelisagis kounelisagis deleted the agis/schema-evolution-fixed-to-var branch November 15, 2024 15:37
kounelisagis added a commit that referenced this pull request Nov 29, 2024
kounelisagis added a commit that referenced this pull request Nov 29, 2024
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.

3 participants