DarioBalinzo / kafka-connect-elasticsearch-source

Kafka Connect Elasticsearch Source
Apache License 2.0
65 stars 38 forks source link

support deep unions in lists #97

Closed Danny02 closed 1 year ago

Danny02 commented 1 year ago

I noticed that you merge the schemas of maps in lists.

So that for example the document

{
  "list": [
    {"a": "some value"},
    {"b": "other value"},
  ]
}

creates an inner schema where a and b are optional.

I noticed with some of my data that this is not done recursivly, for example this document does not work currently

{
  "list": [
    {"inner": {"a": "some value"}},
    {"inner": {"b": "other value"}},
  ]
}

This PR "fixes" this.