From 8342e0efb1e38c06fa975c7442ad897b764c3bd2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Honza=20Kr=C3=A1l?= Date: Fri, 15 Jan 2016 14:12:03 +0100 Subject: [PATCH] map already returns an iterator, no need to wrap it in iter Thanks to andreip for pointing it out! --- elasticsearch/helpers/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/elasticsearch/helpers/__init__.py b/elasticsearch/helpers/__init__.py index 745865f9..2f98ba7f 100644 --- a/elasticsearch/helpers/__init__.py +++ b/elasticsearch/helpers/__init__.py @@ -95,7 +95,7 @@ def _process_bulk_chunk(client, bulk_actions, raise_on_exception=True, raise_on_ # deserialize the data back, thisis expensive but only run on # errors if raise_on_exception is false, so shouldn't be a real # issue - bulk_data = iter(map(client.transport.serializer.loads, bulk_actions)) + bulk_data = map(client.transport.serializer.loads, bulk_actions) while True: try: # collect all the information about failed actions