[
https://issues.apache.org/jira/browse/BEAM-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17530104#comment-17530104
]
Evan Galpin commented on BEAM-13136:
------------------------------------
Fixed in
https://github.com/apache/beam/pull/16953/commits/b5352707a5e6ab994fc38c2e081dccb8504c6ecf
as part of https://github.com/apache/beam/pull/16953
> Clean leftovers of old ElasticSearchIO versions / test mechanism
> ----------------------------------------------------------------
>
> Key: BEAM-13136
> URL: https://issues.apache.org/jira/browse/BEAM-13136
> Project: Beam
> Issue Type: Improvement
> Components: io-java-elasticsearch
> Reporter: Etienne Chauchot
> Assignee: Evan Galpin
> Priority: P2
> Labels: stale-P2, stale-assigned
>
> While auditing about flakiness issues, I saw lefovers on the test code and
> production code of the IO. This list is not comprehensive but spots some
> places:
> * In ESTestCommon#testSplit there is a reference to ES2: ES2 cannot split
> the shards (missing API) so the split are fixed to the number of shards (5 by
> default in the test). As ES2 support was removed, this special case in the
> test can be removed
> * org.elasticsearch.bootstrap.JarHell was a hacky fix to disable jarHell
> test in ES test framework. This framework is no more used (testContainers) so
> we could remove the hack in all places
> * if (backendVersion == 2) in prod code + remove _shardPreference_ from
> _BoundedElasticsearchSource_
> * @ThreadLeakScope(ThreadLeakScope.Scope.NONE) come from es test framework
> that is no longer used
> * remove module elasticsearch-tests-2
> * please audit for other places that could need cleaning.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
