walterddr opened a new pull request #7459:
URL: https://github.com/apache/pinot/pull/7459
## Description
Looking at the usage of StreamConfig and BatchConfig. it seems like they
should be used as
- an internal representation after parsing the TableConfig class
- performs validation to contents of the configuration instead of storing as
plain Strings.
Therefore, changing some of the API usages of BatchConfig.
## Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order:
Controller, Broker, Server, Minion)
NO
Does this PR fix a zero-downtime upgrade introduced earlier?
NO
Does this PR otherwise need attention when creating release notes? Things to
consider:
- New configuration options - NO
- Deprecation of configurations - NO
- Signature changes to public methods/interfaces - YES
- FileIngestionHelper no longer accepts BatchConfig as 3rd argument.
instead a Map is required.
- New plugins added or old plugins removed - NO
## Documentation
N/A
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
