tkaymak commented on PR #38493:
URL: https://github.com/apache/beam/pull/38493#issuecomment-4673395729

   Thanks @gemini-code-assist — all three are fair, applied in `c7d09d04a96`:
   
   1. **`%` → `%s`** in the `checkState` message in 
`MqttReadSchemaTransformProvider` — genuine bug, the key set wouldn't have 
rendered.
   2. **`foo` → `connectionConfiguration`** on the write builder setter — 
agreed, leftover from the revived #32385 diff.
   3. **`System.out.println` → `LOG.info`** with parameterized formatting in 
the test.
   
   (Folded into the existing two commits to keep the history tidy rather than 
adding a fixup commit.)
   


-- 
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]

Reply via email to