Skip to content

Revert "Revert "Parse v2 record. (#35408)" (#35469)" - #35497

Merged
Abacn merged 1 commit into
apache:masterfrom
changliiu:re-add-v2-parsing
Jul 1, 2025
Merged

Revert "Revert "Parse v2 record. (#35408)" (#35469)"#35497
Abacn merged 1 commit into
apache:masterfrom
changliiu:re-add-v2-parsing

Conversation

@changliiu

@changliiuchangliiu commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

No implementation change.

Only changes:

  1. remove pined proto version
  2. update the ChangeStreamrecordMapper comments.

Reverts #35469
#35480 should solve the dependency issue in the original reverted PR #35408

@changliiu
changliiuforce-pushed the re-add-v2-parsing branch 2 times, most recently from 9262476 to 75ba9cbCompareJuly 1, 2025 17:28
@changliiu
changliiu marked this pull request as ready for review July 1, 2025 17:36
@github-actions

Copy link
Copy Markdown
Contributor

Assigning reviewers:

R: @ahmedabu98 for label java.
R: @nielm for label spanner.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@Abacn
Abacn merged commit 9487042 into apache:masterJul 1, 2025
@changliiu
changliiu deleted the re-add-v2-parsing branch July 1, 2025 20:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@changliiu@dedocibula@Abacn