Skip to content

Conversation

@Technoboy-
Copy link
Contributor

Master Issue: #369

Motivation

Support MQTT-5 Conn-ACK with subscriptionIdentifiersAvailable and maximumPacketSize

  • no-need-doc

@codecov-commenter
Copy link

Codecov Report

Merging #685 (ad31756) into master (617fba1) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #685      +/-   ##
============================================
- Coverage     78.20%   78.12%   -0.09%     
+ Complexity      956      953       -3     
============================================
  Files           108      108              
  Lines          4084     4096      +12     
  Branches        309      309              
============================================
+ Hits           3194     3200       +6     
- Misses          719      725       +6     
  Partials        171      171              
Impacted Files Coverage Δ
.../streamnative/pulsar/handlers/mqtt/Connection.java 88.53% <100.00%> (+0.07%) ⬆️
...sar/handlers/mqtt/messages/ack/MqttConnectAck.java 75.53% <100.00%> (+2.27%) ⬆️
...s/mqtt/proxy/MQTTProxyProtocolMethodProcessor.java 77.97% <100.00%> (+0.07%) ⬆️
...handlers/mqtt/restrictions/ServerRestrictions.java 60.00% <100.00%> (+10.00%) ⬆️
...qtt/support/MQTTBrokerProtocolMethodProcessor.java 78.96% <100.00%> (+0.06%) ⬆️
...pulsar/handlers/mqtt/MQTTCommonInboundHandler.java 77.94% <0.00%> (-5.89%) ⬇️
...e/pulsar/handlers/mqtt/utils/PulsarTopicUtils.java 76.52% <0.00%> (-1.74%) ⬇️

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 617fba1...ad31756. Read the comment docs.

@Technoboy- Technoboy- merged commit 3bfcc5c into streamnative:master Jun 25, 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