Skip to content

[feat] Add util to reconstruct Arrow data#12749

Merged
sfc-gh-bnisco merged 1 commit intodevelopfrom
bnisco/ccv2-stack-25
Oct 24, 2025
Merged

[feat] Add util to reconstruct Arrow data#12749
sfc-gh-bnisco merged 1 commit intodevelopfrom
bnisco/ccv2-stack-25

Conversation

@sfc-gh-bnisco
Copy link
Copy Markdown
Collaborator

@sfc-gh-bnisco sfc-gh-bnisco commented Oct 10, 2025 •
edited
Loading

Describe your changes

Added utility function reconstructMixedData to handle Arrow data references in CCv2. This function processes data objects containing Arrow references and replaces them with actual Arrow Tables by:

  1. Handling root-level Arrow references
  2. Processing first-level properties of objects for Arrow references
  3. Preserving other data types (primitives, arrays)

Also added comprehensive unit tests to verify the function's behavior with different input scenarios.

Testing Plan

  • Unit Tests: Added extensive test suite in reconstructMixedData.test.ts covering:
    • Handling of primitive data types
    • Processing of root-level Arrow references
    • Error handling for missing or invalid Arrow data
    • Proper replacement of first-level properties
    • Preservation of nested objects and arrays

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

This was referenced Oct 10, 2025
Merged
Merged
Merged
Merged
Merged
Merged
@snyk-io
Copy link
Copy Markdown
Contributor

snyk-io bot commented Oct 10, 2025 •
edited
Loading

✅ Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
✅ Licenses 0 0 0 0 0 issues
✅ Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

This was referenced Oct 10, 2025
Merged
Merged
Merged
Merged
Merged
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 10, 2025 •
edited
Loading

✅ PR preview is ready!

Name Link
📦 Wheel file https://core-previews.s3-us-west-2.amazonaws.com/pr-12749/streamlit-1.50.0-py3-none-any.whl
📦 @streamlit/component-v2-lib Download from artifacts
🕹️ Preview app pr-12749.streamlit.app (☁️ Deploy here if not accessible)

This was referenced Oct 10, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@sfc-gh-bnisco Graphite App
Copy link
Copy Markdown
Collaborator Author

sfc-gh-bnisco commented Oct 10, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Oct 10, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@sfc-gh-bnisco sfc-gh-bnisco force-pushed the bnisco/ccv2-stack-25 branch 2 times, most recently from 0af7790 to e4e3246 Compare October 24, 2025 18:14
@sfc-gh-bnisco sfc-gh-bnisco force-pushed the bnisco/ccv2-stack-24 branch 2 times, most recently from c943800 to 1709f1e Compare October 24, 2025 19:56
@sfc-gh-bnisco Graphite App
Copy link
Copy Markdown
Collaborator Author

sfc-gh-bnisco commented Oct 24, 2025 •
edited
Loading

Merge activity

  • Oct 24, 8:22 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Oct 24, 8:24 PM UTC: Graphite rebased this pull request as part of a merge.
  • Oct 24, 8:47 PM UTC: @sfc-gh-bnisco merged this pull request with Graphite.

@sfc-gh-bnisco sfc-gh-bnisco changed the base branch from bnisco/ccv2-stack-24 to graphite-base/12749 October 24, 2025 20:22
@sfc-gh-bnisco sfc-gh-bnisco changed the base branch from graphite-base/12749 to develop October 24, 2025 20:23
@sfc-gh-bnisco sfc-gh-bnisco merged commit 65b50ae into develop Oct 24, 2025
37 checks passed
@sfc-gh-bnisco sfc-gh-bnisco deleted the bnisco/ccv2-stack-25 branch October 24, 2025 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@graphite-app graphite-app[bot] graphite-app[bot] left review comments

Copilot code review Copilot Copilot left review comments

@lukasmasuch lukasmasuch lukasmasuch approved these changes

Assignees

No one assigned

Labels

change:feature PR contains new feature or enhancement implementation impact:internal PR changes only affect internal code

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sfc-gh-bnisco @lukasmasuch