Skip to content

Memory leak in SendPacket.vi #4

Closed
@kjkirkegaard

Description

@kjkirkegaard

Hi

In the search for a explanation for why it will take in average 1.4ms to publish to a topic I think I found a memory leak in the SendPackage.vi.
We only use QoS = 0 in this application because we need speed, and we can survive loosing packages. With my knowledge when sending a package with QoS = 0 the MQTT Broker will not replay with an ACK and here the problem starts, because every time we send a package the PackageID is stored in the PackageIDs DVR.
image
but is will not be taken off the PackageIDs array because no ACK is returned to the MQTT Client and therefore the array will keep growing.

One solution to this could be to use the Wait for ack? to not save the PacketID when we don't need to wait for a ACK.
But I'm not 100% sure that the PackageID is only used for verifying ACK if not then this is not the correct solution.
image
image

But I did not find a solution to why it takes in average 1.4ms to publish to a topic.

Regards
Kaare Juul Kirkegaard

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions