Skip to content

Conversation

@Technoboy-
Copy link
Contributor

Master Issue: #369

Motivation

Support publish with Content type, Correlation data, Response topic properties.

Documentation

  • no-need-doc

@Technoboy- Technoboy- requested review from a team and codelipenghui as code owners June 13, 2022 13:29
@Technoboy- Technoboy- self-assigned this Jun 13, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jun 14, 2022

Codecov Report

Merging #663 (2b88c8d) into master (0d0af93) will increase coverage by 0.00%.
The diff coverage is 58.11%.

@@            Coverage Diff            @@
##             master     #663   +/-   ##
=========================================
  Coverage     77.97%   77.97%           
- Complexity      910      923   +13     
=========================================
  Files           108      108           
  Lines          3923     3978   +55     
  Branches        276      295   +19     
=========================================
+ Hits           3059     3102   +43     
- Misses          702      708    +6     
- Partials        162      168    +6     
Impacted Files Coverage Δ
...s/mqtt/proxy/MQTTProxyProtocolMethodProcessor.java 77.89% <0.00%> (-0.86%) ⬇️
...lsar/handlers/mqtt/support/Qos1PublishHandler.java 64.81% <0.00%> (ø)
...qtt/support/MQTTBrokerProtocolMethodProcessor.java 78.84% <41.17%> (-0.99%) ⬇️
...ar/handlers/mqtt/utils/PulsarMessageConverter.java 87.73% <90.00%> (+2.02%) ⬆️
...lsar/handlers/mqtt/messages/MqttPropertyUtils.java 89.47% <100.00%> (-0.78%) ⬇️
.../pulsar/handlers/mqtt/messages/ack/MqttPubAck.java 87.80% <100.00%> (+0.30%) ⬆️
...handlers/mqtt/restrictions/ClientRestrictions.java 68.00% <100.00%> (+1.33%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 818f4b0...2b88c8d. Read the comment docs.

@Technoboy- Technoboy- merged commit 3659389 into streamnative:master Jun 15, 2022
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.

3 participants