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

[fix] OffloadPoliciesImplBuilder missing method and not implements OffloadPolicies.Builder #11453

Merged

Conversation

lifepuzzlefun
Copy link
Contributor

org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder implement OffloadPolicies.Builder
and missing the method setS3ManagedLedgerOffloadRoleSessionName

… implement OffloadPolicies.Builder

and missing the method setS3ManagedLedgerOffloadRoleSessionName
@lifepuzzlefun lifepuzzlefun changed the title org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder implement OffloadPolicies.Builder org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implements OffloadPolicies.Builder Jul 25, 2021
@lifepuzzlefun lifepuzzlefun changed the title org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implements OffloadPolicies.Builder [fix] org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implements OffloadPolicies.Builder Jul 26, 2021
@lifepuzzlefun lifepuzzlefun changed the title [fix] org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implements OffloadPolicies.Builder [fix] OffloadPoliciesImplBuilder missing method and not implements OffloadPolicies.Builder Jul 26, 2021
@@ -472,7 +472,7 @@ private static Object getCompatibleValue(Properties properties, Field field) {
return value((String) object, field);
}

public static class OffloadPoliciesImplBuilder {
public static class OffloadPoliciesImplBuilder implements OffloadPolicies.Builder{
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public static class OffloadPoliciesImplBuilder implements OffloadPolicies.Builder{
public static class OffloadPoliciesImplBuilder implements OffloadPolicies.Builder {

@sijie sijie added this to the 2.9.0 milestone Jul 28, 2021
@codelipenghui codelipenghui merged commit b046233 into apache:master Aug 2, 2021
codelipenghui pushed a commit that referenced this pull request Aug 4, 2021
…floadPolicies.Builder (#11453)

* org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implement OffloadPolicies.Builder
and missing the method setS3ManagedLedgerOffloadRoleSessionName

(cherry picked from commit b046233)
@codelipenghui codelipenghui added the cherry-picked/branch-2.8 Archived: 2.8 is end of life label Aug 4, 2021
bharanic-dev pushed a commit to bharanic-dev/pulsar that referenced this pull request Mar 18, 2022
…floadPolicies.Builder (apache#11453)

* org.apache.pulsar.common.policies.data.OffloadPoliciesImplBuilder not implement OffloadPolicies.Builder
and missing the method setS3ManagedLedgerOffloadRoleSessionName
Sign up for free to 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.

None yet

3 participants