clintropolis opened a new pull request, #14955:
URL: https://github.com/apache/druid/pull/14955
### Description
Follow up to #14456, which in retrospect was a bit overly aggressive in
removing the ability to serialize v4 of the nested column format in favor of
always using the latest and greatest. This PR adds back the serializers, as
well as introduces a system config to allow for more flexibility in what
versions can be upgraded from.
changes:
* add back nested column v4 serializers
* 'json' schema by default still uses the newer 'nested common format' used
by 'auto', but now has an optional 'formatVersion' property which can be
specified to override format versions on native ingest jobs
* add system config to specify default column format stuff,
'druid.indexing.formats', and property
'druid.indexing.formats.nestedColumnFormatVersion' to specify system level
preferred nested column format for friendly rolling upgrades from versions
which do not support the newer 'nested common format' used by 'auto'
#### Release note
Add system level runtime.properties option to specify default column format
stuff, with prefix 'druid.indexing.formats', and property
'druid.indexing.formats.nestedColumnFormatVersion' to specify preferred nested
column format for friendly rolling upgrades from Druid 25 to Druid 28.
<hr>
This PR has:
- [ ] been self-reviewed.
- [ ] using the [concurrency
checklist](https://github.com/apache/druid/blob/master/dev/code-review/concurrency.md)
(Remove this item if the PR doesn't have any relation to concurrency.)
- [ ] added documentation for new or modified features or behaviors.
- [ ] a release note entry in the PR description.
- [ ] added Javadocs for most classes and all non-trivial methods. Linked
related entities via Javadoc links.
- [ ] added or updated version, license, or notice information in
[licenses.yaml](https://github.com/apache/druid/blob/master/dev/license.md)
- [ ] added comments explaining the "why" and the intent of the code
wherever would not be obvious for an unfamiliar reader.
- [ ] added unit tests or modified existing tests to cover new code paths,
ensuring the threshold for [code
coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md)
is met.
- [ ] added integration tests.
- [ ] been tested in a test Druid cluster.
--
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]
