Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Transform] There is a lot common code between single-node-tests and multi-node-tests #107146

Open
przemekwitek opened this issue Apr 5, 2024 · 1 comment
Open

[Transform] There is a lot common code between single-node-tests and multi-node-tests #107146

przemekwitek opened this issue Apr 5, 2024 · 1 comment
Assignees
@przemekwitek
Labels
>enhancement :ml/Transform Transform Team:ML Meta label for the ML team

Comments

@przemekwitek
Copy link
Contributor

Description

We have 2 (among others) suites of transform integration tests: single-node-tests and multi-node-tests. Each of these suites has a base class: TransformRestTestCase. There share a lot of common (or similar) code that could be deduplicated.
@przemekwitek przemekwitek self-assigned this Apr 5, 2024
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Apr 5, 2024
@przemekwitek przemekwitek mentioned this issue Apr 5, 2024
Merged
@demjened demjened added the :ml/Transform Transform label Apr 5, 2024
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Apr 5, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@przemekwitek przemekwitek

Labels
>enhancement :ml/Transform Transform Team:ML Meta label for the ML team
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@demjened @przemekwitek @elasticsearchmachine