Skip to content

[improve][pip] PIP-286: Make the TopicCompactionService to support find entry based on publishTime or index#20867

Merged
codelipenghui merged 6 commits into
apache:masterfrom
coderzc:improve/pip-286
Sep 26, 2023
Merged

[improve][pip] PIP-286: Make the TopicCompactionService to support find entry based on publishTime or index#20867
codelipenghui merged 6 commits into
apache:masterfrom
coderzc:improve/pip-286

Conversation

@coderzc

@coderzc coderzc commented Jul 25, 2023

Copy link
Copy Markdown
Member

Documentation

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

Matching PR in forked repository

PR in forked repository:

@coderzc coderzc changed the title PIP-286: Add findNewestPosition method in the TopicCompactionService API [improve][pip] PIP-286: Add findNewestPosition method in the TopicCompactionService API Jul 25, 2023
@asafm

asafm commented Jul 26, 2023

Copy link
Copy Markdown
Contributor

If your number is off. Newest one I see before you is 288

@coderzc

coderzc commented Jul 26, 2023

Copy link
Copy Markdown
Member Author

If your number is off. Newest one I see before you is 288

But I didn't find PIP-286 in the previous PR

@asafm

asafm commented Jul 26, 2023

Copy link
Copy Markdown
Contributor

I understand, but better to have holes than go back and forth.

@Technoboy- Technoboy- added this to the 3.2.0 milestone Jul 31, 2023
@coderzc coderzc changed the title [improve][pip] PIP-286: Add findNewestPosition method in the TopicCompactionService API [improve][pip] PIP-286: Add findFirstEntryMetadata method in the TopicCompactionService API Aug 19, 2023
@coderzc coderzc force-pushed the improve/pip-286 branch 4 times, most recently from 06e7968 to aafceef Compare August 19, 2023 08:15
@coderzc coderzc changed the title [improve][pip] PIP-286: Add findFirstEntryMetadata method in the TopicCompactionService API [improve][pip] PIP-286: Make the TopicCompactionService to support find entry based on publishTime or index Sep 11, 2023

@codelipenghui codelipenghui left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please check the comment and I think you can start the VOTE for now.

Comment thread pip/pip-286.md
@coderzc

coderzc commented Sep 26, 2023

Copy link
Copy Markdown
Member Author

The vote has already closed, please see: https://lists.apache.org/thread/q27zg49mpr8otwh29s3sncdcx8ly7ws6

@codelipenghui codelipenghui merged commit 48d5b9d into apache:master Sep 26, 2023
liangyuanpeng pushed a commit to liangyuanpeng/pulsar that referenced this pull request Oct 11, 2023
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 type/PIP

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants