Skip to content
This repository was archived by the owner on Feb 16, 2023. It is now read-only.

Commit 33b365c

Browse files
etsybaevJordan Scott
authored and
Jordan Scott
committed
[9964] destination-mqtt: fixed build tests failures (airbytehq#13099)
* [9964] destination-mqtt: fixed build tests failures
1 parent c4030fe commit 33b365c

File tree

3 files changed

+11
-2
lines changed

3 files changed

+11
-2
lines changed

airbyte-integrations/connectors/destination-mqtt/src/main/java/io/airbyte/integrations/destination/mqtt/MqttDestination.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
package io.airbyte.integrations.destination.mqtt;
66

77
import com.fasterxml.jackson.databind.JsonNode;
8+
import com.google.common.base.Charsets;
89
import com.google.common.collect.ImmutableMap;
910
import io.airbyte.commons.json.Jsons;
1011
import io.airbyte.integrations.BaseConnector;
@@ -49,7 +50,7 @@ public AirbyteConnectionStatus check(final JsonNode config) {
4950
COLUMN_NAME_EMITTED_AT, System.currentTimeMillis(),
5051
COLUMN_NAME_DATA, Jsons.jsonNode(ImmutableMap.of("test-key", "test-value"))));
5152

52-
final MqttMessage message = new MqttMessage(payload.toString().getBytes());
53+
final MqttMessage message = new MqttMessage(payload.toString().getBytes(Charsets.UTF_8));
5354
message.setQos(mqttConfig.getQos());
5455
message.setRetained(mqttConfig.isRetainedMessage());
5556

airbyte-integrations/connectors/destination-mqtt/src/main/java/io/airbyte/integrations/destination/mqtt/MqttRecordConsumer.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
package io.airbyte.integrations.destination.mqtt;
66

77
import com.fasterxml.jackson.databind.JsonNode;
8+
import com.google.common.base.Charsets;
89
import com.google.common.collect.ImmutableMap;
910
import io.airbyte.commons.json.Jsons;
1011
import io.airbyte.commons.lang.Exceptions;
@@ -86,7 +87,7 @@ protected void acceptTracked(final AirbyteMessage airbyteMessage) {
8687
MqttDestination.COLUMN_NAME_EMITTED_AT, recordMessage.getEmittedAt(),
8788
MqttDestination.COLUMN_NAME_DATA, recordMessage.getData()));
8889

89-
final MqttMessage message = new MqttMessage(payload.toString().getBytes());
90+
final MqttMessage message = new MqttMessage(payload.toString().getBytes(Charsets.UTF_8));
9091
message.setRetained(config.isRetainedMessage());
9192
message.setQos(config.getQos());
9293

docs/integrations/destinations/mqtt.md

+7
Original file line numberDiff line numberDiff line change
@@ -76,3 +76,10 @@ You should now have all the requirements needed to configure MQTT as a destinati
7676
More info about this can be found in the [OASIS MQTT standard site](http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/mqtt-v3.1.1.html).
7777

7878
_NOTE_: MQTT version 5 is not supported yet.
79+
80+
81+
## Changelog
82+
83+
| Version | Date | Pull Request | Subject |
84+
| :--- | :--- | :--- | :--- |
85+
| 0.1.2 | 2022-05-24 | [13099](https://github.com/airbytehq/airbyte/pull/13099) | Fixed build's tests |

0 commit comments

Comments
 (0)