Skip to content

[Managed-ledger]optimized constant naming convention#14004

Merged
hangc0276 merged 2 commits into
apache:masterfrom
liudezhi2098:optimized_constant_naming_convention
Jan 29, 2022
Merged

[Managed-ledger]optimized constant naming convention#14004
hangc0276 merged 2 commits into
apache:masterfrom
liudezhi2098:optimized_constant_naming_convention

Conversation

@liudezhi2098

Copy link
Copy Markdown
Contributor

Motivation

public static NullLedgerOffloader instance_ = new NullLedgerOffloader();

modify to CONSTANT_CASE style.

Modifications

  • Use async instead of sync method.

Documentation

Need to update docs?

  • no-need-doc

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jan 28, 2022
@liudezhi2098

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

1 similar comment
@liudezhi2098

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@codelipenghui codelipenghui added this to the 2.10.0 milestone Jan 28, 2022
@liudezhi2098

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@liudezhi2098

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@hangc0276
hangc0276 merged commit 00e7cc6 into apache:master Jan 29, 2022
BewareMyPower added a commit to streamnative/kop that referenced this pull request Feb 14, 2022
Bump Pulsar version to 2.10.0.0-rc2 and fix the incompatibility issues introduced from apache/pulsar#14004 and apache/pulsar#12401.
BewareMyPower added a commit to streamnative/kop that referenced this pull request Feb 14, 2022
Bump Pulsar version to 2.10.0.0-rc2 and fix the incompatibility issues introduced from apache/pulsar#14004 and apache/pulsar#12401.

(cherry picked from commit 1e6826b)
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants