Support for QoS1 and QoS2 publishing/subscribing about arduino-mqtt HOT 9 CLOSED

@q2dg yes, even AVR boards like the Uno should be fine. See comments in #53 (comment) for more metrics.

from arduino-mqtt.

Thanks for raising this.

The underlying library (Paho) uses constants to activate QOS1 an QOS2 and as the Arduino IDE offers no library configuration via constants, enabling these features would result in more program space usage for all users.

Therefore, this feature is delayed until the Arduino IDE offers a suitable solution.

Ongoing discussions:

• arduino/arduino-builder#15
• arduino/Arduino#5084

from arduino-mqtt.

@q2dg what boards would you like to use the QoS1 and QoS2 with?

from arduino-mqtt.

Since the increase in programming space usage is minimal, I think we should enable support for all QoS levels in v2.

The usage table again for future reference:

QoS0: 25768 bytes (9%)
QoS1: 26240 bytes (10%)
QoS2: 26552 bytes (10%)

from arduino-mqtt.

@sandeepmistry Hello! Well, I'm not a skilled technician but I think SAMD's boards (Zero and MKRxxx) -at least- could do the job, don't you think?

from arduino-mqtt.

I am trying to use this library with ESP8266 and QoS1. I have not written any app code yet, just in selecting sutable library yet, but I suppose that some modification would be useful in src/lib/MQTTClient.h:

#ifdef ESP8266
extern "C" void yield( void );
#endif

// only used in single-threaded mode where one command at a time is in process
template<class Network, class Timer, int a, int b>
int MQTT::Client<Network, Timer, a, b>::waitfor(int packet_type, Timer& timer)
{
    int rc = FAILURE;

    do
    {
        if (timer.expired())
            break; // we timed out
#ifdef ESP8266
        yield();
#endif
    }
    while ((rc = cycle(timer)) != packet_type);

    return rc;
}

Please, fix me if I am wrong!

from arduino-mqtt.

@alim-s-ajiev The library has been tested and successfully used with the ESP8266. No modification should be necessary to use it with the current feature set.

from arduino-mqtt.

Yes, you are right! It turns out that yield() is called inside cycle method for ESP8266 WiFi.

Thank you very much!

from arduino-mqtt.

This is beeing addressed in #59.

from arduino-mqtt.