Skip to content

[fix] Some typos in Cmd classes - #19190

Merged
tisonkun merged 1 commit into
apache:masterfrom
Nicklee007:fix-some-typos
Jan 12, 2023
Merged

[fix] Some typos in Cmd classes#19190
tisonkun merged 1 commit into
apache:masterfrom
Nicklee007:fix-some-typos

Conversation

@Nicklee007

Copy link
Copy Markdown
Contributor

Motivation

Fix some typos

Documentation

Check the box below or label this PR directly.

Need to update docs?

  • doc-not-needed
    (Please explain why)

@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Jan 11, 2023

@tisonkun tisonkun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@tisonkun tisonkun changed the title [typos]fix some typos in cmd [fix] Some typos in Cmd classes Jan 11, 2023
@tisonkun

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #19190 (e21ea45) into master (4b0dc9a) will decrease coverage by 11.63%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #19190       +/-   ##
=============================================
- Coverage     48.96%   37.33%   -11.64%     
+ Complexity     7300     1994     -5306     
=============================================
  Files           424      209      -215     
  Lines         45473    14450    -31023     
  Branches       4672     1577     -3095     
=============================================
- Hits          22268     5395    -16873     
+ Misses        20698     8475    -12223     
+ Partials       2507      580     -1927     
Flag Coverage Δ
unittests 37.33% <ø> (-11.64%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...apache/pulsar/broker/admin/v1/SchemasResource.java
...ersistentStickyKeyDispatcherMultipleConsumers.java
...ervice/persistent/MessageRedeliveryController.java
...ice/persistent/PersistentMessageExpiryMonitor.java
.../apache/pulsar/broker/admin/v2/ResourceQuotas.java
...vice/schema/AvroSchemaBasedCompatibilityCheck.java
.../org/apache/pulsar/broker/admin/v2/Namespaces.java
...ulsar/broker/stats/prometheus/metrics/Summary.java
...g/apache/pulsar/compaction/CompactedTopicImpl.java
...rg/apache/pulsar/broker/delayed/bucket/Bucket.java
... and 623 more

@Technoboy- Technoboy- added this to the 2.12.0 milestone Jan 12, 2023
@Nicklee007

Copy link
Copy Markdown
Contributor Author

/pulsarbot run-failure-checks

@tisonkun

Copy link
Copy Markdown
Member

Merging...

@tisonkun
tisonkun merged commit 89b1ff1 into apache:master Jan 12, 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 ready-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants