for my particular use case it is fine to flip the default around. My concern was just that it is slightly less backwards compatible since if someone *had* defined a `name` field (even if it wasn't used), the proposed in this PR way would retain prior behavior. The only way to get the behavior in this PR is to add the `name` at the top level *AND* remove the `name` from the delegate.
I'm ok with making it more clear by using the `name` at the top level first, as long as folks are ok with the slightly unexpected behavior change that probably-will-not-be-in-the-wild [ Full content available at: https://github.com/apache/incubator-druid/pull/6219 ] This message was relayed via gitbox.apache.org for [email protected]
