support for routing list in bulk op

Author: l-trotta

java-client/src/main/java/co/elastic/clients/elasticsearch/_helpers/bulk/IngesterOperation.java

@@ -30,6 +30,7 @@
 import co.elastic.clients.util.BinaryData;
 
 import javax.annotation.Nullable;
+import java.util.List;
 
 /**
  * A bulk operation whose size has been calculated and content turned to a binary blob (to compute its size).
@@ -212,6 +213,23 @@ private static int size(String name, @Nullable JsonEnum value) {
         }
     }
 
+    private static int size(String name, @Nullable List<String> value) {
+        if (value != null) {
+            if (value.isEmpty()) {
+                return name.length() + 6; // 6 added chars for empty array "name":[],
+            }
+            int listSize = 0;
+            for (String item : value) {
+                listSize += item.length();
+                listSize += 2; // +2 quotes each item
+            }
+            listSize += value.size()-1 ; // +1 comma between each item
+            return name.length() + listSize + 6; // "name":["item1","item2","item3"],
+        } else {
+            return 0;
+        }
+    }
+
     private static int basePropertiesSize(BulkOperationBase op) {
         return
             size("id", op.id()) +

java-client/src/test/java/co/elastic/clients/elasticsearch/_helpers/bulk/BulkIngesterTest.java

@@ -418,7 +418,7 @@ public void beforeBulk(long executionId, BulkRequest request, List<Void> context
             .listener(listener)
             .globalSettings(s -> s
                 .index("foo")
-                .routing("bar")
+                .routing("bar","aaa")
             )
         );
 
@@ -431,7 +431,8 @@ public void beforeBulk(long executionId, BulkRequest request, List<Void> context
         assertEquals(1, ingester.requestCount());
 
         assertEquals("foo", storedRequest.get().index());
-        assertEquals("bar", storedRequest.get().routing());
+        assertEquals("bar", storedRequest.get().routing().get(0));
+        assertEquals("aaa", storedRequest.get().routing().get(1));
     }
 
     @Test