Skip to content

feat(connectors): add BSON payload support to connectors runtime#3299

Draft
Standing-Man wants to merge 2 commits into
apache:masterfrom
Standing-Man:bson-payload
Draft

feat(connectors): add BSON payload support to connectors runtime#3299
Standing-Man wants to merge 2 commits into
apache:masterfrom
Standing-Man:bson-payload

Conversation

@Standing-Man
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #1847.

Rationale

What changed?

Local Execution

  • Passed / not passed
  • Pre-commit hooks ran / not ran

AI Usage

Signed-off-by: StandingMan <jmtangcs@gmail.com>
@github-actions
Copy link
Copy Markdown

Thanks for the pull request. It is now waiting for review, labeled S-waiting-on-review.

You can update that label as the review goes back and forth, with slash commands - each on its own line, in a regular PR comment (not an inline review reply):

  • /ready - mark it S-waiting-on-review again, after addressing feedback
  • /author - mark it S-waiting-on-author (maintainers, or anyone who has had a PR merged before)
  • /request-review @user ... - request reviewers (@user or @org/team)

Commands take up to ~90s to apply. If no reaction (👍 or 😕) appears on your comment, the apply step likely failed - check the repo's Actions tab for the PR Triage Apply run. Commands posted inside a review body (rather than a normal comment) cannot be reacted to, so they stay log-only.

See CONTRIBUTING.md for details.

@github-actions github-actions Bot added the S-waiting-on-review PR is waiting on a reviewer label May 22, 2026
@Standing-Man Standing-Man marked this pull request as draft May 22, 2026 12:15
@github-actions github-actions Bot removed the S-waiting-on-review PR is waiting on a reviewer label May 22, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

❌ Patch coverage is 36.72840% with 205 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.85%. Comparing base (2337883) to head (a051713).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
core/connectors/sdk/src/encoders/avro.rs 42.80% 151 Missing and 8 partials ⚠️
core/connectors/sdk/src/encoders/proto.rs 0.00% 16 Missing ⚠️
core/connectors/sdk/src/lib.rs 0.00% 10 Missing ⚠️
...ore/connectors/sdk/src/transforms/proto_convert.rs 0.00% 6 Missing ⚠️
core/connectors/sdk/src/decoders/bson.rs 0.00% 5 Missing ⚠️
core/connectors/sdk/src/encoders/bson.rs 0.00% 5 Missing ⚠️
core/connectors/sdk/src/encoders/flatbuffer.rs 0.00% 3 Missing ⚠️
core/connectors/sinks/http_sink/src/lib.rs 0.00% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (36.72%) is below the target coverage (50.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #3299       +/-   ##
=============================================
- Coverage     74.18%   60.85%   -13.34%     
  Complexity      943      943               
=============================================
  Files          1200     1206        +6     
  Lines        109645    98397    -11248     
  Branches      86534    74986    -11548     
=============================================
- Hits          81345    59877    -21468     
- Misses        25553    35762    +10209     
- Partials       2747     2758       +11     
Components Coverage Δ
Rust Core 57.85% <36.72%> (-17.59%) ⬇️
Java SDK 58.44% <ø> (ø)
C# SDK 70.22% <ø> (+0.78%) ⬆️
Python SDK 81.43% <ø> (ø)
Node SDK 91.44% <ø> (ø)
Go SDK 39.91% <ø> (ø)
Files with missing lines Coverage Δ
core/connectors/sinks/http_sink/src/lib.rs 85.63% <0.00%> (-0.16%) ⬇️
core/connectors/sdk/src/encoders/flatbuffer.rs 79.67% <0.00%> (-0.99%) ⬇️
core/connectors/sdk/src/decoders/bson.rs 0.00% <0.00%> (ø)
core/connectors/sdk/src/encoders/bson.rs 0.00% <0.00%> (ø)
...ore/connectors/sdk/src/transforms/proto_convert.rs 71.76% <0.00%> (-0.58%) ⬇️
core/connectors/sdk/src/lib.rs 50.50% <0.00%> (-5.68%) ⬇️
core/connectors/sdk/src/encoders/proto.rs 69.03% <0.00%> (-1.52%) ⬇️
core/connectors/sdk/src/encoders/avro.rs 60.55% <42.80%> (-12.07%) ⬇️

... and 289 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: StandingMan <jmtangcs@gmail.com>
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.

Add support for BSON payload to connectors runtime

1 participant