Remove deprecated String concat in Painless

elastic / elasticsearch

Free and Open Source, Distributed, RESTful Search Engine

https://www.elastic.co/products/elasticsearch

Other

69.74k stars 24.68k forks source link

Remove deprecated String concat in Painless #52601

Open jdconrad opened 4 years ago

jdconrad commented 4 years ago

Painless currently supports two versions of String concatenation using the + operator in Painless. In Java 8 and below, a traditional StringBuilder is used. In Java 9 and above a new technique is used involving invokedynamic instructions. We should remove the older version as we will no longer support Java 8 in our next major release.

elasticmachine commented 4 years ago

Pinging @elastic/es-core-infra (:Core/Infra/Scripting)

danielcr10 commented 3 years ago

Hi @jdconrad , I'm new to the project and would like to try to contribute. Can I try this?

icemakegirl commented 2 years ago

Is somebody working here? I'd like undertain better