From 9267038886625e385d49db69edfcd5fdbf9f1488 Mon Sep 17 00:00:00 2001
From: Jason Tedor <jason@tedor.me>
Date: Thu, 10 Aug 2017 11:26:58 +0900
Subject: [PATCH] Elasticsearch 5: Close content builder
This commit wraps the usage of a content builder in a try-with-resources
block to ensure that it is correctly closed.
---
.../com/yahoo/ycsb/db/elasticsearch5/ElasticsearchClient.java | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/elasticsearch5/src/main/java/com/yahoo/ycsb/db/elasticsearch5/ElasticsearchClient.java b/elasticsearch5/src/main/java/com/yahoo/ycsb/db/elasticsearch5/ElasticsearchClient.java
index d6a77bd3..aecee472 100644
--- a/elasticsearch5/src/main/java/com/yahoo/ycsb/db/elasticsearch5/ElasticsearchClient.java
+++ b/elasticsearch5/src/main/java/com/yahoo/ycsb/db/elasticsearch5/ElasticsearchClient.java
@@ -150,8 +150,7 @@ public class ElasticsearchClient extends DB {
@Override
public Status insert(String table, String key, Map<String, ByteIterator> values) {
- try {
- final XContentBuilder doc = jsonBuilder();
+ try (XContentBuilder doc = jsonBuilder()) {
doc.startObject();
for (final Entry<String, String> entry : StringByteIterator.getStringMap(values).entrySet()) {
--
GitLab