Skip to content

Jolt processors: Migrate properties as well - #2

Merged
lfrancke merged 24 commits into
mainfrom
feat/more-migrations
Oct 13, 2025
Merged

Jolt processors: Migrate properties as well#2
lfrancke merged 24 commits into
mainfrom
feat/more-migrations

Conversation

@lfrancke

@lfrancke lfrancke commented Oct 11, 2025

Copy link
Copy Markdown
Member

This was missing in #1

  • Renames jolt_transform.rs to jolt_transform_json.rs to match the processor name (also rename the struct), this makes the diff look larger. If you look at the jolt_transform_record file you can see the "real" diff
  • Some properties have been renamed (I assume unintentionally but I don't know). For example: jolt-spec to Jolt Specification - this PR adds support for migrating these properties

@lfrancke lfrancke moved this to Development: In Progress in Stackable Engineering Oct 11, 2025
@lfrancke lfrancke self-assigned this Oct 11, 2025
Base automatically changed from feat/initial-commit to main October 13, 2025 08:59
@lfrancke
lfrancke marked this pull request as ready for review October 13, 2025 09:11
@lfrancke lfrancke moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Oct 13, 2025
@lfrancke

Copy link
Copy Markdown
Member Author

I'm going to merge this now. I can incorporate any feedback later.

@lfrancke
lfrancke merged commit 2ad1d61 into main Oct 13, 2025
5 of 6 checks passed
@lfrancke
lfrancke deleted the feat/more-migrations branch October 13, 2025 10:14
@Techassi Techassi moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Oct 16, 2025
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants