Copilot commented on code in PR #17090:
URL: https://github.com/apache/iotdb/pull/17090#discussion_r2731604649
##########
integration-test/src/test/java/org/apache/iotdb/relational/it/mqtt/IoTDBMQTTServiceIT.java:
##########
@@ -69,6 +69,7 @@ public void setUp() throws Exception {
mqtt.setPassword(PASSWORD);
mqtt.setConnectAttemptsMax(3);
mqtt.setReconnectDelay(10);
+ mqtt.setClientId("clientId01");
Review Comment:
This PR claims to "Add mqtt IT for json mode" but the test is configured to
use the "line" payload formatter (FORMATTER = "line" on line 56), not JSON. The
line formatter processes line protocol format data, while the json formatter
would process JSON format data. Either the PR description is incorrect, or the
test should be using FORMATTER = "json" to test JSON mode as described.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
