Skip to content

[fix] Fix typos in class Message and Producer #25265

Merged
codelipenghui merged 1 commit intoapache:masterfrom
Hyeon-moGu:fix/typo-javadoc-and-log-messages
Mar 5, 2026
Merged

[fix] Fix typos in class Message and Producer #25265
codelipenghui merged 1 commit intoapache:masterfrom
Hyeon-moGu:fix/typo-javadoc-and-log-messages

Conversation

@Hyeon-moGu
Copy link
Contributor

@Hyeon-moGu Hyeon-moGu commented Feb 26, 2026

Motivation

Fix typos in Message and Producer classes to improve code readability and documentation clarity.

Modifications

Fixed a typo in the Message class ( optiona -> optional )
Fixed a typo in the Producer class ( autorizing -> authorizing )

Documentation

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

@github-actions
Copy link

@Hyeon-moGu Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

@github-actions github-actions bot added doc-not-needed Your PR changes do not impact docs and removed doc-label-missing labels Feb 26, 2026
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.58%. Comparing base (82846e5) to head (00a588c).
⚠️ Report is 4 commits behind head on master.

Files with missing lines Patch % Lines
...ava/org/apache/pulsar/broker/service/Producer.java 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #25265      +/-   ##
============================================
- Coverage     72.73%   72.58%   -0.16%     
- Complexity    34047    34393     +346     
============================================
  Files          1959     1959              
  Lines        155418   155418              
  Branches      17731    17731              
============================================
- Hits         113050   112806     -244     
- Misses        33378    33603     +225     
- Partials       8990     9009      +19     
Flag Coverage Δ
inttests 25.73% <0.00%> (-0.43%) ⬇️
systests 22.33% <0.00%> (-0.14%) ⬇️
unittests 73.57% <0.00%> (-0.13%) ⬇️

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

Files with missing lines Coverage Δ
...ain/java/org/apache/pulsar/client/api/Message.java 0.00% <ø> (ø)
...ava/org/apache/pulsar/broker/service/Producer.java 81.00% <0.00%> (ø)

... and 109 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codelipenghui codelipenghui merged commit 629ddeb into apache:master Mar 5, 2026
55 of 62 checks passed
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.

3 participants