REFACTOR-#6044: remove code duplication for get_objects_from_partitions

[anmyachev]

What do these changes do?

• first commit message and PR title follow format outlined here

  NOTE: If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

• passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
• passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
• signed commit with git commit -s
• Resolves CLN: remove code duplication for get_objects_from_partitions #6044
• tests added and passing
• module layout described at docs/development/architecture.rst is up-to-date

[YarShev]

Suggested change

	if hasattr(cls, "_execution_wrapper"):
	if hasattr(cls._partition_class, "execution_wrapper"):

Would this work? I am just thinking it would good for one class to keep the execution wrapper.

[anmyachev]

I don't really like that kind of thing. In the code below, we will materialize multiple partitions, and it will look a little strange that to materialize multiple partitions, we use the attribute of the partition, and not the manager, which is designed to work with multiple partitions.

On the other hand, all wrappers are implemented as mixins, so we can switch to inheritance.

[YarShev]

On the other hand, all wrappers are implemented as mixins, so we can switch to inheritance.

We should probably give it a try.

[anmyachev]

#6052

[YarShev]

Should we wait for #6052 in order to avoid double refactoring commits?

[anmyachev]

I'd like to merge it without waiting. Making these changes is faster than having to wait for the CI for this PR to complete again.