[BUG] TypeError on calling parallel_bulk (v2.4.0)

[evios]

What is the bug?

TypeError("_process_bulk_chunk() got multiple values for argument 'raise_on_exception'")

How can one reproduce the bug?

call

parallel_bulk(
            self._client,
            actions=self._session_generator(),
            chunk_size=100,
            raise_on_error=False,
            raise_on_exception=False,
            max_chunk_bytes=20 * 1024 * 1024,
            request_timeout=60,
        )

What is your host/environment?

Debian Bullseye

Do you have any additional context?

It was first found in version 2.4.0.
ignore_status transferred as 3rd positional argument in _process_bulk_chunk:

opensearch-py/opensearchpy/helpers/actions.py

Lines 486 to 488 in 567ede3

	lambda bulk_chunk: list(
	_process_bulk_chunk(
	client, bulk_chunk[1], bulk_chunk[0], ignore_status, *args, **kwargs

but raise_on_exception expected as 3rd positional:

opensearch-py/opensearchpy/helpers/actions.py

Lines 239 to 248 in 567ede3

	def _process_bulk_chunk(
	client: Any,
	bulk_actions: Any,
	bulk_data: Any,
	raise_on_exception: bool = True,
	raise_on_error: bool = True,
	ignore_status: Any = (),
	*args: Any,
	**kwargs: Any
	) -> Any:

Previous versions did not have it. Version 2.3.2 snippet:

opensearch-py/opensearchpy/helpers/actions.py

Lines 470 to 478 in 2f57661

	lambda bulk_chunk: list(
	_process_bulk_chunk(
	client,
	bulk_chunk[1],
	bulk_chunk[0],
	ignore_status=ignore_status,
	*args,
	**kwargs
	)

[dblock]

Likely a bug that slipped through test coverage :( Care to write a test that reproduces this? Maybe a fix?

[dblock]

Fixed in #601, appreciate any additional eyes @evios.

[evios]

@dblock thanks for the instant fixing. Sure, will keep an eye on the updated version :)

[dblock]

@evios If you want to try it from HEAD that'd be great.

[dblock]

We've released 2.4.2 with a fix.

[evios]

Thank you, it works smoothly with 2.4.2