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

[improve] [pip] PIP-394 Add two interfaces CursorMetadataSerializerProvider and CursorMetadataDeSerializerProvider to support newer or customized cursor metadata serializations #23608

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

poorbarcode
Copy link
Contributor

@poorbarcode poorbarcode commented Nov 18, 2024

Motivation

Modifications

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: x

…ovider and CursorMetadataDeSerializerProvider to support newer of customized cursor metadata serializations
@poorbarcode poorbarcode added this to the 4.1.0 milestone Nov 18, 2024
@poorbarcode poorbarcode self-assigned this Nov 18, 2024
@github-actions github-actions bot added PIP doc-not-needed Your PR changes do not impact docs labels Nov 18, 2024
@poorbarcode poorbarcode changed the title [improve] [pip] PIP-394 PIP-394 Add two interfaces CursorMetadataSerializerProvider and CursorMetadataDeSerializerProvider to support newer of customized cursor metadata serializations [improve] [pip] PIP-394 Add two interfaces CursorMetadataSerializerProvider and CursorMetadataDeSerializerProvider to support newer of customized cursor metadata serializations Nov 18, 2024
@poorbarcode poorbarcode changed the title [improve] [pip] PIP-394 Add two interfaces CursorMetadataSerializerProvider and CursorMetadataDeSerializerProvider to support newer of customized cursor metadata serializations [improve] [pip] PIP-394 Add two interfaces CursorMetadataSerializerProvider and CursorMetadataDeSerializerProvider to support newer or customized cursor metadata serializations Nov 18, 2024
pip/pip-394.md Outdated Show resolved Hide resolved
pip/pip-394.md Outdated Show resolved Hide resolved
pip/pip-394.md Outdated Show resolved Hide resolved
@poorbarcode poorbarcode requested a review from lhotari November 18, 2024 12:43
@shibd shibd requested review from Technoboy- and coderzc November 27, 2024 03:56
- We call the serialization that implemented before `4.0.0` `V1`, and call after the https://github.com/apache/pulsar/pull/9292 `v2`.
- Add all version of serialization into `branch-3.0`.
- Set the default value of `3.0.x` is `V1`, which is the same as the current status.
- Set the default value of `4.0.x` is `V1`, which is the same as the current status.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

4.0.0 default serialization is V2?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs PIP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants