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

Change log level to debug #1033

Merged
merged 2 commits into from
Jul 25, 2023
Merged

Change log level to debug #1033

merged 2 commits into from
Jul 25, 2023

Conversation

mattisonchao
Copy link
Member

Motivation

Change the log level to debug to avoid tons of logs.

Modifications

  • Change the log level to debug to avoid tons of logs.

Verifying this change

  • Make sure that the change passes the CI checks.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #1033 (c07a7bb) into master (1eccdae) will increase coverage by 0.11%.
The diff coverage is 77.77%.

❗ Current head c07a7bb differs from pull request most recent head 6b340f8. Consider uploading reports for the commit 6b340f8 to get more accurate results

@@             Coverage Diff              @@
##             master    #1033      +/-   ##
============================================
+ Coverage     77.28%   77.39%   +0.11%     
- Complexity      988     1001      +13     
============================================
  Files           111      111              
  Lines          4336     4362      +26     
  Branches        336      338       +2     
============================================
+ Hits           3351     3376      +25     
- Misses          802      805       +3     
+ Partials        183      181       -2     
Files Changed Coverage Δ
...e/pulsar/handlers/mqtt/MQTTChannelInitializer.java 64.58% <ø> (ø)
...ndlers/mqtt/proxy/MQTTProxyChannelInitializer.java 61.53% <ø> (ø)
...ive/pulsar/handlers/mqtt/support/MQTTConsumer.java 97.61% <ø> (ø)
...lsar/handlers/mqtt/support/Qos0PublishHandler.java 62.50% <0.00%> (-8.93%) ⬇️
...upport/systemtopic/MQTTEventSystemTopicClient.java 57.69% <ø> (-5.77%) ⬇️
...ystemtopic/SystemTopicBasedSystemEventService.java 79.03% <70.00%> (-0.63%) ⬇️
...lsar/handlers/mqtt/support/Qos1PublishHandler.java 48.00% <75.00%> (+0.05%) ⬆️
...andlers/mqtt/proxy/PulsarServiceLookupHandler.java 64.76% <80.00%> (+0.68%) ⬆️
...dlers/mqtt/support/event/AutoSubscribeHandler.java 81.13% <85.71%> (+0.28%) ⬆️
...lers/mqtt/support/event/PulsarEventCenterImpl.java 87.50% <86.66%> (-4.50%) ⬇️
... and 2 more

... and 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Technoboy- Technoboy- merged commit 726f526 into master Jul 25, 2023
42 of 44 checks passed
@Technoboy- Technoboy- added this to the 3.1.0 milestone Jul 25, 2023
@delete-merged-branch delete-merged-branch bot deleted the log/fix branch July 25, 2023 12:21
Technoboy- pushed a commit that referenced this pull request Jul 25, 2023
Technoboy- pushed a commit that referenced this pull request Jul 25, 2023
Technoboy- pushed a commit that referenced this pull request Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants