Skip to content

Conversation

@Jinny-Wang
Copy link
Contributor

@Jinny-Wang Jinny-Wang commented Oct 23, 2025

…og() to public

Description

  • change loadHistoryUUID to protected,
  • change Origin#isFromTranslog to public
    so that subclasses of Engine implemented in plugin repository can access them

Related Issues

Resolves #19753

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Jinny-Wang Jinny-Wang force-pushed the publicize-methods-in-engine branch from 41b3342 to 3cc022a Compare October 23, 2025 22:13
@msfroh
Copy link
Contributor

msfroh commented Oct 23, 2025

@mch2 -- Any strong feelings on these APIs? You're the one who introduced loadHistoryUUID to share it between InternalEngine and NRTReplicationEngine. Any objections to making it protected so non-core Engine implementations can use it?

@github-actions
Copy link
Contributor

❌ Gradle check result for 3cc022a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@mch2
Copy link
Member

mch2 commented Oct 24, 2025

@mch2 -- Any strong feelings on these APIs? You're the one who introduced loadHistoryUUID to share it between InternalEngine and NRTReplicationEngine. Any objections to making it protected so non-core Engine implementations can use it?

Nope no issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement or improvement to existing feature or request _No response_

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] make Engine loadHistoryUUID protected and Origin#isFromTranslog public

5 participants