search

elastic / elasticsearch

Free and Open Source, Distributed, RESTful Search Engine
https://www.elastic.co/products/elasticsearch
Other
1.08k stars 24.84k forks source link

[ES|QL]: Implement NamedWritable and remove PlanNamedTypes #107788

Open not-napoleon opened 6 months ago

not-napoleon commented 6 months ago

While developing ES|QL initially, we decided to not use the Writable/NamedWritable infrastructure in Elasticsearch. This was in part due to the shared QL infrastructure code not implementing it, and not wanting to spend the dev time to do that. As we make progress on https://github.com/elastic/elasticsearch/issues/106679, those original reasons become less relevant. Migrating to the standard serialization code will reduce our surface area for maintenance and allow us to leverage the existing testing around that infrastructure.

elasticsearchmachine commented 6 months ago

Pinging @elastic/es-analytical-engine (Team:Analytics)