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

[CORE-2951] Integration test for Transforms Rust SDK SR module #18555

Merged
merged 7 commits into from
May 21, 2024

Conversation

oleiman
Copy link
Member

@oleiman oleiman commented May 17, 2024

Basic example of rust SR usage and hook into integration test github action.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v24.1.x
  • v23.3.x
  • v23.2.x

Release Notes

Bug Fixes

  • Fixed misnamed schema registry ABI stubs in the Data Transforms Rust SDK and improved tests.

@oleiman oleiman self-assigned this May 17, 2024
@github-actions github-actions bot added area/build area/wasm WASM Data Transforms labels May 17, 2024
@oleiman oleiman force-pushed the xform/core-2951/test-rust-sr branch 5 times, most recently from cc509ad to 82f684c Compare May 17, 2024 22:31
voutilad and others added 3 commits May 19, 2024 18:47
Fixes redpanda-data#18528

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
Try to extract a schema ID from a byte slice.

Returns:
- SchemaId on success
- SchemaRegistryError::BadHeader on failure
  - when either the 0 magic byte is missing or len(slice) < 5

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
@oleiman oleiman force-pushed the xform/core-2951/test-rust-sr branch from 82f684c to 6a88fe5 Compare May 20, 2024 01:49
For integration testing.

Duplicates go/transform/internal/testdata/schema_registry

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
Also adjusts golang example to remove namespace, which doesn't add much
in terms of verification and is tricky to work with in rust.

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
@oleiman oleiman force-pushed the xform/core-2951/test-rust-sr branch from 6a88fe5 to 83c099d Compare May 20, 2024 01:52
@oleiman oleiman marked this pull request as ready for review May 20, 2024 02:00
@oleiman oleiman requested a review from a team as a code owner May 20, 2024 02:00
@oleiman oleiman requested review from clee, a team, BenPope and pgellert and removed request for a team and BenPope May 20, 2024 02:00
@oleiman oleiman changed the title [CORE-2951] Xform/core 2951/test rust sr [CORE-2951] Integration test for Transforms Rust SDK SR module May 20, 2024
@oleiman oleiman merged commit e246304 into redpanda-data:dev May 21, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build area/wasm WASM Data Transforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants