Skip to content

better handling pivot export failures#9531

Merged
pjain1 merged 2 commits into
mainfrom
better_pivot_error
Jun 10, 2026
Merged

better handling pivot export failures#9531
pjain1 merged 2 commits into
mainfrom
better_pivot_error

Conversation

@pjain1

@pjain1 pjain1 commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

To produce actionable error when pivot fails

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@pjain1 pjain1 requested a review from begelundmuller June 8, 2026 09:17
@pjain1 pjain1 requested a review from nishantmonu51 June 8, 2026 11:32
begelundmuller
begelundmuller approved these changes Jun 9, 2026
View reviewed changes

@begelundmuller begelundmuller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@pjain1 pjain1 merged commit 7263ccf into main Jun 10, 2026
11 checks passed
@pjain1 pjain1 deleted the better_pivot_error branch June 10, 2026 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@begelundmuller begelundmuller begelundmuller approved these changes
@nishantmonu51 nishantmonu51 Awaiting requested review from nishantmonu51

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pjain1 @begelundmuller