cometa-haley opened a new pull request, #15416:
URL: https://github.com/apache/pulsar/pull/15416
### Motivation
"measurement" is the actual mandatory field and the "device" error message
is confusing in my opinion. Thus I propose to replace it so it is
self-explainatory.
### Modifications
Replaced the word "device" (which is not a mandatory field) with
"measurement" which is the correct one (check line 67)
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
*(Please pick either of the following options)*
This change is a trivial rework
### Does this pull request potentially affect one of the following parts:
*If `yes` was chosen, please highlight the changes*
- Dependencies (does it add or upgrade a dependency): no
- The public API: no
- The schema: no
- The default values of configurations: no
- The wire protocol: no
- The rest endpoints: no
- The admin cli options: no
- Anything that affects deployment: no
(I am not really pro in Pulsar nor java, but my understanding is that this
will only affect people trying to configure topics correctly)
### Documentation
Need to update docs?
- [ ] `doc-required`
(Your PR needs to update docs and you will update later)
- [x] `no-need-doc`
At the moment there is no documentation as how the messages have to be built
for the connector to be able to process them. Might want to do this
documentation in the future, but first I need to figure out how & where to do
that:)
- [ ] `doc`
(Your PR contains doc changes)
- [ ] `doc-added`
(Docs have been already added)
--
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]
