diff --git a/CHANGELOG.md b/CHANGELOG.md index 33702aa7..93658586 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) ### Deprecated ### Removed ### Fixed +- Fix `TypeError` on `parallel_bulk` ([#601](https://github.com/opensearch-project/opensearch-py/pull/601)) ### Security ## [2.4.1] diff --git a/guides/bulk.md b/guides/bulk.md index ef6b8c5d..60ae55ae 100644 --- a/guides/bulk.md +++ b/guides/bulk.md @@ -46,6 +46,8 @@ data = [ response = client.bulk(data) if response["errors"]: print(f"There were errors!") + for item in response["items"]: + print(f"{item['index']['status']}: {item['index']['error']['type']}") else: print(f"Bulk-inserted {len(rc['items'])} items.") ``` @@ -69,3 +71,29 @@ response = helpers.bulk(client, docs, max_retries=3) print(response) ``` +Bulk helpers support `parallel_bulk` which has options to turn off exceptions, chunk size, etc. + +```python +succeeded = [] +failed = [] +for success, item in helpers.parallel_bulk(client, + actions=data, + chunk_size=10, + raise_on_error=False, + raise_on_exception=False, + max_chunk_bytes=20 * 1024 * 1024, + request_timeout=60): + + if success: + succeeded.append(item) + else: + failed.append(item) + +if len(failed) > 0: + print(f"There were {len(failed)} errors:") + for item in failed: + print(f"{item['index']['error']}: {item['index']['exception']}") + +if len(succeeded) > 0: + print(f"Bulk-inserted {len(succeeded)} items.") +``` diff --git a/opensearchpy/helpers/actions.py b/opensearchpy/helpers/actions.py index 7f8ced35..960d5a1c 100644 --- a/opensearchpy/helpers/actions.py +++ b/opensearchpy/helpers/actions.py @@ -442,6 +442,8 @@ def parallel_bulk( max_chunk_bytes: int = 100 * 1024 * 1024, queue_size: int = 4, expand_action_callback: Any = expand_action, + raise_on_exception: bool = True, + raise_on_error: bool = True, ignore_status: Any = (), *args: Any, **kwargs: Any @@ -485,7 +487,14 @@ def _setup_queues(self) -> None: for result in pool.imap( lambda bulk_chunk: list( _process_bulk_chunk( - client, bulk_chunk[1], bulk_chunk[0], ignore_status, *args, **kwargs + client, + bulk_chunk[1], + bulk_chunk[0], + raise_on_exception, + raise_on_error, + ignore_status, + *args, + **kwargs ) ), _chunk_actions( diff --git a/samples/bulk/bulk-helpers.py b/samples/bulk/bulk-helpers.py index 3dc165c8..240009df 100755 --- a/samples/bulk/bulk-helpers.py +++ b/samples/bulk/bulk-helpers.py @@ -49,8 +49,34 @@ for i in range(100): data.append({"_index": index_name, "_id": i, "value": i}) +# serialized bulk raising an exception on error rc = helpers.bulk(client, data) print(f"Bulk-inserted {rc[0]} items.") +# parallel bulk with explicit error checking +succeeded = [] +failed = [] +for success, item in helpers.parallel_bulk( + client, + actions=data, + chunk_size=10, + raise_on_error=False, + raise_on_exception=False, + max_chunk_bytes=20 * 1024 * 1024, + request_timeout=60, +): + if success: + succeeded.append(item) + else: + failed.append(item) + +if len(failed) > 0: + print(f"There were {len(failed)} errors:") + for item in failed: + print(item["index"]["error"]) + +if len(succeeded) > 0: + print(f"Bulk-inserted {len(succeeded)} items.") + # delete index client.indices.delete(index=index_name) diff --git a/test_opensearchpy/test_helpers/test_actions.py b/test_opensearchpy/test_helpers/test_actions.py index 739e8647..e44dbc98 100644 --- a/test_opensearchpy/test_helpers/test_actions.py +++ b/test_opensearchpy/test_helpers/test_actions.py @@ -67,11 +67,62 @@ def test_all_chunks_sent(self, _process_bulk_chunk: Any) -> None: self.assertEqual(50, mock_process_bulk_chunk.call_count) # type: ignore + @mock.patch("opensearchpy.OpenSearch.bulk") + def test_with_all_options(self, _bulk: Any) -> None: + actions = ({"x": i} for i in range(100)) + list( + helpers.parallel_bulk( + OpenSearch(), + actions=actions, + chunk_size=2, + raise_on_error=False, + raise_on_exception=False, + max_chunk_bytes=20 * 1024 * 1024, + request_timeout=160, + ignore_status=(123), + ) + ) + + self.assertEqual(50, _bulk.call_count) + _bulk.assert_called_with( + '{"index":{}}\n{"x":98}\n{"index":{}}\n{"x":99}\n', request_timeout=160 + ) + + @mock.patch("opensearchpy.helpers.actions._process_bulk_chunk") + def test_process_bulk_chunk_with_all_options( + self, _process_bulk_chunk: Any + ) -> None: + actions = ({"x": i} for i in range(100)) + client = OpenSearch() + list( + helpers.parallel_bulk( + client, + actions=actions, + chunk_size=2, + raise_on_error=True, + raise_on_exception=True, + max_chunk_bytes=20 * 1024 * 1024, + request_timeout=160, + ignore_status=(123), + ) + ) + + self.assertEqual(50, _process_bulk_chunk.call_count) + _process_bulk_chunk.assert_called_with( + client, + ['{"index":{}}', '{"x":98}', '{"index":{}}', '{"x":99}'], + [({"index": {}}, {"x": 98}), ({"index": {}}, {"x": 99})], + True, + True, + 123, + request_timeout=160, + ) + @pytest.mark.skip # type: ignore @mock.patch( "opensearchpy.helpers.actions._process_bulk_chunk", # make sure we spend some time in the thread - side_effect=lambda *a: [ + side_effect=lambda *args, **kwargs: [ (True, time.sleep(0.001) or threading.current_thread().ident) # type: ignore ], )