Skip to content

fix: always return correct json schema to users#5109

Merged
Xuanwo merged 10 commits intomainfrom
jsonb-debug
Nov 4, 2025
Merged

fix: always return correct json schema to users#5109
Xuanwo merged 10 commits intomainfrom
jsonb-debug

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Oct 30, 2025

Our users hit a bug where appending data with a casted schema that includes JSON results in "invalid jsonb". I believe #5096 is the correct fix, but it introduces a whole new concept and needs more time to review. So I created this PR as a hotfix.

This PR updates the schema on the Python side to ensure users see the correct schema.


This PR was primarily authored with Codex using GPT-5-Codex and then hand-reviewed by me. I AM responsible for every change made in this PR. I aimed to keep it aligned with our goals, though I may have missed minor issues. Please flag anything that feels off, I'll fix it quickly.

Signed-off-by: Xuanwo <github@xuanwo.io>
@github-actions github-actions Bot added bug Something isn't working python labels Oct 30, 2025
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Copy link
Copy Markdown
Member

@westonpace westonpace left a comment

Choose a reason for hiding this comment

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

One question but the fix is good to have

Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo merged commit 80fa4c2 into main Nov 4, 2025
18 checks passed
@Xuanwo Xuanwo deleted the jsonb-debug branch November 4, 2025 12:37
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 2026
Our users hit a bug where appending data with a casted schema that
includes JSON results in "invalid jsonb". I believe
lance-format#5096 is the correct fix, but it
introduces a whole new concept and needs more time to review. So I
created this PR as a hotfix.

This PR updates the schema on the Python side to ensure users see the
correct schema.


---

**This PR was primarily authored with Codex using GPT-5-Codex and then
hand-reviewed by me. I AM responsible for every change made in this PR.
I aimed to keep it aligned with our goals, though I may have missed
minor issues. Please flag anything that feels off, I'll fix it
quickly.**

---------

Signed-off-by: Xuanwo <github@xuanwo.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants