Hi Tom,

Thanks for your input!

1. I'll add more details for the RequestConvertToJson and
XYZJsonDataConverter classes. Hopefully it will be more clear, but just to
answer your question, RequestConvertToJson does not return a
XYZJsonDataConverter, but rather it returns a JsonNode which will be
serialized. The JsonDataConverter is the new auto-generated schema for each
request/response type that contains the method to return the JsonNode to be
serialized.

2. There is no defined order of the properties, rather it's in the order
that it is set in. So if you first set key B, then key A, the properties
would appear with key B first. JsonNodes when serialized does not sort the
keys.

3. Yes, serialization is done via Jackson databind.

Thanks again,
Anastasia

On Fri, Sep 25, 2020 at 1:15 AM Tom Bentley <tbent...@redhat.com> wrote:

> Hi Anastasia,
>
> Thanks for the KIP, I can certainly see the benefit of this. I have a few
> questions:
>
> 1. I think it would be helpful to readers to explicitly illustrate the
> RequestConvertToJson and XYZJsonDataConverter classes (e.g. with method
> signatures for one or two methods), because currently it's not clear (to me
> at least) exactly what's being proposed. Does the RequestConvertToJson
> return a XYZJsonDataConverter?
>
> 2. Does the serialization have a defined order of properties (alphabetic
> perhaps)? My concern here is that properties appearing in order according
> to how they are iterated in a hash map might harm human readability of the
> logs.
>
> 3. Would the serialization be done via the Jackson databinding?
>
> Many thanks,
>
> Tom
>
> On Thu, Sep 24, 2020 at 11:49 PM Anastasia Vela <av...@confluent.io>
> wrote:
>
> > Hi all,
> >
> > I'd like to discuss KIP-673:
> >
> >
> https://cwiki.apache.org/confluence/display/KAFKA/KIP-673%3A+Emit+JSONs+with+new+auto-generated+schema
> >
> > This is a proposal to change the format of request and response traces to
> > JSON, which would be easier to load and parse, because the current format
> > is only JSON-like and not easily parsable.
> >
> > Let me know what you think,
> > Anastasia
> >
>

Reply via email to