[jira] [Commented] (BEAM-13136) Clean leftovers of old ElasticSearchIO versions / test mechanism

["Beam JIRA Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/BEAM-13136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17512755#comment-17512755 ] 

Beam JIRA Bot commented on BEAM-13136:
--------------------------------------

This issue is P2 but has been unassigned without any comment for 60 days so it has been labeled "stale-P2". If this issue is still affecting you, we care! Please comment and remove the label. Otherwise, in 14 days the issue will be moved to P3.

Please see https://beam.apache.org/contribute/jira-priorities/ for a detailed explanation of what these priorities mean.


> 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
>            Priority: P2
>              Labels: stale-P2
>
> 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.1#820001)