summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorRyan Ernst <ryan@iernst.net>2016-04-18 14:27:09 -0700
committerRyan Ernst <ryan@iernst.net>2016-04-18 14:27:09 -0700
commit8ce71f8f25fbf014fb595df4e50f4cd008d0dfbf (patch)
tree5eb2757b9f56285084444a7bfdab5d67e79ced40 /core
parent4e89b1706bdf7295a886ab1e1bb5f64034832f40 (diff)
parent88dd6ba430e729d473432076a6eba1f6cb85c136 (diff)
Merge pull request #17774 from rjernst/camelcase1
Rest: Remove 'case' parameter from rest apis
Diffstat (limited to 'core')
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponse.java2
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoResponse.java14
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodeStats.java10
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsResponse.java4
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/list/ListTasksResponse.java8
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/repositories/verify/VerifyRepositoryResponse.java4
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/status/SnapshotIndexStatus.java2
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/GetFieldMappingsResponse.java4
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/indices/segments/IndicesSegmentResponse.java6
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/indices/stats/IndicesStatsResponse.java2
-rw-r--r--core/src/main/java/org/elasticsearch/action/admin/indices/upgrade/get/UpgradeStatusResponse.java2
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/ClusterState.java8
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/metadata/AliasMetaData.java2
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/metadata/IndexMetaData.java4
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/metadata/IndexTemplateMetaData.java4
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetaData.java2
-rw-r--r--core/src/main/java/org/elasticsearch/cluster/node/DiscoveryNode.java2
-rw-r--r--core/src/main/java/org/elasticsearch/common/settings/Settings.java2
-rw-r--r--core/src/main/java/org/elasticsearch/common/unit/Fuzziness.java2
-rw-r--r--core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilder.java98
-rw-r--r--core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilderString.java19
-rw-r--r--core/src/main/java/org/elasticsearch/index/fielddata/FieldDataStats.java4
-rw-r--r--core/src/main/java/org/elasticsearch/index/search/stats/SearchStats.java2
-rw-r--r--core/src/main/java/org/elasticsearch/index/shard/IndexingStats.java2
-rw-r--r--core/src/main/java/org/elasticsearch/index/snapshots/blobstore/BlobStoreIndexShardSnapshots.java2
-rw-r--r--core/src/main/java/org/elasticsearch/monitor/fs/FsInfo.java6
-rw-r--r--core/src/main/java/org/elasticsearch/monitor/jvm/JvmStats.java6
-rw-r--r--core/src/main/java/org/elasticsearch/rest/AbstractRestChannel.java9
-rw-r--r--core/src/main/java/org/elasticsearch/rest/action/admin/indices/alias/get/RestGetAliasesAction.java2
-rw-r--r--core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetMappingAction.java2
-rw-r--r--core/src/main/java/org/elasticsearch/rest/action/admin/indices/settings/RestGetSettingsAction.java2
-rw-r--r--core/src/main/java/org/elasticsearch/rest/action/admin/indices/upgrade/RestUpgradeAction.java2
-rw-r--r--core/src/main/java/org/elasticsearch/rest/action/admin/indices/validate/query/RestValidateQueryAction.java2
-rw-r--r--core/src/main/java/org/elasticsearch/search/suggest/completion/CompletionStats.java6
-rw-r--r--core/src/main/java/org/elasticsearch/threadpool/ThreadPool.java2
-rw-r--r--core/src/main/java/org/elasticsearch/threadpool/ThreadPoolStats.java2
-rw-r--r--core/src/test/java/org/elasticsearch/common/xcontent/builder/XContentBuilderTests.java12
37 files changed, 70 insertions, 194 deletions
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponse.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponse.java
index 946897a2c9..7c0bd35028 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/health/ClusterHealthResponse.java
@@ -305,7 +305,7 @@ public class ClusterHealthResponse extends ActionResponse implements StatusToXCo
if (outputIndices) {
builder.startObject(Fields.INDICES);
for (ClusterIndexHealth indexHealth : clusterStateHealth.getIndices().values()) {
- builder.startObject(indexHealth.getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexHealth.getIndex());
indexHealth.toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoResponse.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoResponse.java
index 79eab0f61a..fdb15db8ff 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoResponse.java
@@ -64,23 +64,23 @@ public class NodesInfoResponse extends BaseNodesResponse<NodeInfo> implements To
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
- builder.field("cluster_name", getClusterName().value(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field("cluster_name", getClusterName().value());
builder.startObject("nodes");
for (NodeInfo nodeInfo : this) {
- builder.startObject(nodeInfo.getNode().getId(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(nodeInfo.getNode().getId());
- builder.field("name", nodeInfo.getNode().getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field("name", nodeInfo.getNode().getName());
builder.field("transport_address", nodeInfo.getNode().getAddress().toString());
- builder.field("host", nodeInfo.getNode().getHostName(), XContentBuilder.FieldCaseConversion.NONE);
- builder.field("ip", nodeInfo.getNode().getHostAddress(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field("host", nodeInfo.getNode().getHostName());
+ builder.field("ip", nodeInfo.getNode().getHostAddress());
builder.field("version", nodeInfo.getVersion());
builder.field("build_hash", nodeInfo.getBuild().shortHash());
if (nodeInfo.getServiceAttributes() != null) {
for (Map.Entry<String, String> nodeAttribute : nodeInfo.getServiceAttributes().entrySet()) {
- builder.field(nodeAttribute.getKey(), nodeAttribute.getValue(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(nodeAttribute.getKey(), nodeAttribute.getValue());
}
}
@@ -93,7 +93,7 @@ public class NodesInfoResponse extends BaseNodesResponse<NodeInfo> implements To
if (!nodeInfo.getNode().getAttributes().isEmpty()) {
builder.startObject("attributes");
for (Map.Entry<String, String> entry : nodeInfo.getNode().getAttributes().entrySet()) {
- builder.field(entry.getKey(), entry.getValue(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(entry.getKey(), entry.getValue());
}
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodeStats.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodeStats.java
index ce16a848fc..22e12d2baa 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodeStats.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodeStats.java
@@ -299,10 +299,10 @@ public class NodeStats extends BaseNodeResponse implements ToXContent {
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
if (!params.param("node_info_format", "default").equals("none")) {
- builder.field("name", getNode().getName(), XContentBuilder.FieldCaseConversion.NONE);
- builder.field("transport_address", getNode().getAddress().toString(), XContentBuilder.FieldCaseConversion.NONE);
- builder.field("host", getNode().getHostName(), XContentBuilder.FieldCaseConversion.NONE);
- builder.field("ip", getNode().getAddress(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field("name", getNode().getName());
+ builder.field("transport_address", getNode().getAddress().toString());
+ builder.field("host", getNode().getHostName());
+ builder.field("ip", getNode().getAddress());
builder.startArray("roles");
for (DiscoveryNode.Role role : getNode().getRoles()) {
@@ -313,7 +313,7 @@ public class NodeStats extends BaseNodeResponse implements ToXContent {
if (!getNode().getAttributes().isEmpty()) {
builder.startObject("attributes");
for (Map.Entry<String, String> attrEntry : getNode().getAttributes().entrySet()) {
- builder.field(attrEntry.getKey(), attrEntry.getValue(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(attrEntry.getKey(), attrEntry.getValue());
}
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsResponse.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsResponse.java
index ffd28f9025..af28c1fb5d 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/stats/NodesStatsResponse.java
@@ -65,7 +65,7 @@ public class NodesStatsResponse extends BaseNodesResponse<NodeStats> implements
builder.startObject("nodes");
for (NodeStats nodeStats : this) {
- builder.startObject(nodeStats.getNode().getId(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(nodeStats.getNode().getId());
builder.field("timestamp", nodeStats.getTimestamp());
nodeStats.toXContent(builder, params);
@@ -88,4 +88,4 @@ public class NodesStatsResponse extends BaseNodesResponse<NodeStats> implements
return "{ \"error\" : \"" + e.getMessage() + "\"}";
}
}
-} \ No newline at end of file
+}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/list/ListTasksResponse.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/list/ListTasksResponse.java
index 7b308affcc..f8ecd7a506 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/list/ListTasksResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/list/ListTasksResponse.java
@@ -163,7 +163,7 @@ public class ListTasksResponse extends BaseTasksResponse implements ToXContent {
builder.startObject("nodes");
for (Map.Entry<DiscoveryNode, List<TaskInfo>> entry : getPerNodeTasks().entrySet()) {
DiscoveryNode node = entry.getKey();
- builder.startObject(node.getId(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(node.getId());
builder.field("name", node.getName());
builder.field("transport_address", node.getAddress().toString());
builder.field("host", node.getHostName());
@@ -178,13 +178,13 @@ public class ListTasksResponse extends BaseTasksResponse implements ToXContent {
if (!node.getAttributes().isEmpty()) {
builder.startObject("attributes");
for (Map.Entry<String, String> attrEntry : node.getAttributes().entrySet()) {
- builder.field(attrEntry.getKey(), attrEntry.getValue(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(attrEntry.getKey(), attrEntry.getValue());
}
builder.endObject();
}
builder.startObject("tasks");
for(TaskInfo task : entry.getValue()) {
- builder.startObject(task.getTaskId().toString(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(task.getTaskId().toString());
task.toXContent(builder, params);
builder.endObject();
}
@@ -194,7 +194,7 @@ public class ListTasksResponse extends BaseTasksResponse implements ToXContent {
} else if ("parents".equals(groupBy)) {
builder.startObject("tasks");
for (TaskGroup group : getTaskGroups()) {
- builder.startObject(group.getTaskInfo().getTaskId().toString(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(group.getTaskInfo().getTaskId().toString());
group.toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/repositories/verify/VerifyRepositoryResponse.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/repositories/verify/VerifyRepositoryResponse.java
index ddc2f7035d..11020beac4 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/repositories/verify/VerifyRepositoryResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/repositories/verify/VerifyRepositoryResponse.java
@@ -86,8 +86,8 @@ public class VerifyRepositoryResponse extends ActionResponse implements ToXConte
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
builder.startObject(Fields.NODES);
for (DiscoveryNode node : nodes) {
- builder.startObject(node.getId(), XContentBuilder.FieldCaseConversion.NONE);
- builder.field(Fields.NAME, node.getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(node.getId());
+ builder.field(Fields.NAME, node.getName());
builder.endObject();
}
builder.endObject();
diff --git a/core/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/status/SnapshotIndexStatus.java b/core/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/status/SnapshotIndexStatus.java
index 5999fc3eb4..87d153a83f 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/status/SnapshotIndexStatus.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/status/SnapshotIndexStatus.java
@@ -96,7 +96,7 @@ public class SnapshotIndexStatus implements Iterable<SnapshotIndexShardStatus>,
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
- builder.startObject(getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(getIndex());
shardsStats.toXContent(builder, params);
stats.toXContent(builder, params);
builder.startObject(Fields.SHARDS);
diff --git a/core/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/GetFieldMappingsResponse.java b/core/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/GetFieldMappingsResponse.java
index 12ef72bfc8..e0cedcf841 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/GetFieldMappingsResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/GetFieldMappingsResponse.java
@@ -74,10 +74,10 @@ public class GetFieldMappingsResponse extends ActionResponse implements ToXConte
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
for (Map.Entry<String, Map<String, Map<String, FieldMappingMetaData>>> indexEntry : mappings.entrySet()) {
- builder.startObject(indexEntry.getKey(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexEntry.getKey());
builder.startObject("mappings");
for (Map.Entry<String, Map<String, FieldMappingMetaData>> typeEntry : indexEntry.getValue().entrySet()) {
- builder.startObject(typeEntry.getKey(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(typeEntry.getKey());
for (Map.Entry<String, FieldMappingMetaData> fieldEntry : typeEntry.getValue().entrySet()) {
builder.startObject(fieldEntry.getKey());
fieldEntry.getValue().toXContent(builder, params);
diff --git a/core/src/main/java/org/elasticsearch/action/admin/indices/segments/IndicesSegmentResponse.java b/core/src/main/java/org/elasticsearch/action/admin/indices/segments/IndicesSegmentResponse.java
index 6bcf7d71c3..05729f4f0f 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/indices/segments/IndicesSegmentResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/indices/segments/IndicesSegmentResponse.java
@@ -102,7 +102,7 @@ public class IndicesSegmentResponse extends BroadcastResponse implements ToXCont
builder.startObject(Fields.INDICES);
for (IndexSegments indexSegments : getIndices().values()) {
- builder.startObject(indexSegments.getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexSegments.getIndex());
builder.startObject(Fields.SHARDS);
for (IndexShardSegments indexSegment : indexSegments) {
@@ -164,7 +164,7 @@ public class IndicesSegmentResponse extends BroadcastResponse implements ToXCont
builder.endObject();
return builder;
}
-
+
static void toXContent(XContentBuilder builder, Accountable tree) throws IOException {
builder.startObject();
builder.field(Fields.DESCRIPTION, tree.toString());
@@ -208,4 +208,4 @@ public class IndicesSegmentResponse extends BroadcastResponse implements ToXCont
static final XContentBuilderString DESCRIPTION = new XContentBuilderString("description");
static final XContentBuilderString CHILDREN = new XContentBuilderString("children");
}
-} \ No newline at end of file
+}
diff --git a/core/src/main/java/org/elasticsearch/action/admin/indices/stats/IndicesStatsResponse.java b/core/src/main/java/org/elasticsearch/action/admin/indices/stats/IndicesStatsResponse.java
index 11f91397c5..a99a8883dd 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/indices/stats/IndicesStatsResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/indices/stats/IndicesStatsResponse.java
@@ -176,7 +176,7 @@ public class IndicesStatsResponse extends BroadcastResponse implements ToXConten
if ("indices".equalsIgnoreCase(level) || "shards".equalsIgnoreCase(level)) {
builder.startObject(Fields.INDICES);
for (IndexStats indexStats : getIndices().values()) {
- builder.startObject(indexStats.getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexStats.getIndex());
builder.startObject("primaries");
indexStats.getPrimaries().toXContent(builder, params);
diff --git a/core/src/main/java/org/elasticsearch/action/admin/indices/upgrade/get/UpgradeStatusResponse.java b/core/src/main/java/org/elasticsearch/action/admin/indices/upgrade/get/UpgradeStatusResponse.java
index 6c3a64e04f..b20f855450 100644
--- a/core/src/main/java/org/elasticsearch/action/admin/indices/upgrade/get/UpgradeStatusResponse.java
+++ b/core/src/main/java/org/elasticsearch/action/admin/indices/upgrade/get/UpgradeStatusResponse.java
@@ -126,7 +126,7 @@ public class UpgradeStatusResponse extends BroadcastResponse implements ToXConte
if (outputIndices) {
builder.startObject(Fields.INDICES);
for (IndexUpgradeStatus indexUpgradeStatus : getIndices().values()) {
- builder.startObject(indexUpgradeStatus.getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexUpgradeStatus.getIndex());
builder.byteSizeField(Fields.SIZE_IN_BYTES, Fields.SIZE, indexUpgradeStatus.getTotalBytes());
builder.byteSizeField(Fields.SIZE_TO_UPGRADE_IN_BYTES, Fields.SIZE_TO_UPGRADE, indexUpgradeStatus.getToUpgradeBytes());
diff --git a/core/src/main/java/org/elasticsearch/cluster/ClusterState.java b/core/src/main/java/org/elasticsearch/cluster/ClusterState.java
index ad79387ef3..7fa26df15b 100644
--- a/core/src/main/java/org/elasticsearch/cluster/ClusterState.java
+++ b/core/src/main/java/org/elasticsearch/cluster/ClusterState.java
@@ -428,7 +428,7 @@ public class ClusterState implements ToXContent, Diffable<ClusterState> {
builder.startObject("templates");
for (ObjectCursor<IndexTemplateMetaData> cursor : metaData().templates().values()) {
IndexTemplateMetaData templateMetaData = cursor.value;
- builder.startObject(templateMetaData.name(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(templateMetaData.name());
builder.field("template", templateMetaData.template());
builder.field("order", templateMetaData.order());
@@ -461,7 +461,7 @@ public class ClusterState implements ToXContent, Diffable<ClusterState> {
builder.startObject("indices");
for (IndexMetaData indexMetaData : metaData()) {
- builder.startObject(indexMetaData.getIndex().getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexMetaData.getIndex().getName());
builder.field("state", indexMetaData.getState().toString().toLowerCase(Locale.ENGLISH));
@@ -527,7 +527,7 @@ public class ClusterState implements ToXContent, Diffable<ClusterState> {
builder.startObject("routing_table");
builder.startObject("indices");
for (IndexRoutingTable indexRoutingTable : routingTable()) {
- builder.startObject(indexRoutingTable.getIndex().getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexRoutingTable.getIndex().getName());
builder.startObject("shards");
for (IndexShardRoutingTable indexShardRoutingTable : indexRoutingTable) {
builder.startArray(Integer.toString(indexShardRoutingTable.shardId().id()));
@@ -554,7 +554,7 @@ public class ClusterState implements ToXContent, Diffable<ClusterState> {
builder.startObject("nodes");
for (RoutingNode routingNode : getRoutingNodes()) {
- builder.startArray(routingNode.nodeId() == null ? "null" : routingNode.nodeId(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startArray(routingNode.nodeId() == null ? "null" : routingNode.nodeId());
for (ShardRouting shardRouting : routingNode) {
shardRouting.toXContent(builder, params);
}
diff --git a/core/src/main/java/org/elasticsearch/cluster/metadata/AliasMetaData.java b/core/src/main/java/org/elasticsearch/cluster/metadata/AliasMetaData.java
index d6d13b55bd..7e4d191748 100644
--- a/core/src/main/java/org/elasticsearch/cluster/metadata/AliasMetaData.java
+++ b/core/src/main/java/org/elasticsearch/cluster/metadata/AliasMetaData.java
@@ -281,7 +281,7 @@ public class AliasMetaData extends AbstractDiffable<AliasMetaData> {
}
public static void toXContent(AliasMetaData aliasMetaData, XContentBuilder builder, ToXContent.Params params) throws IOException {
- builder.startObject(aliasMetaData.alias(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(aliasMetaData.alias());
boolean binary = params.paramAsBoolean("binary", false);
diff --git a/core/src/main/java/org/elasticsearch/cluster/metadata/IndexMetaData.java b/core/src/main/java/org/elasticsearch/cluster/metadata/IndexMetaData.java
index 7d99f8fd74..c2ae93f12d 100644
--- a/core/src/main/java/org/elasticsearch/cluster/metadata/IndexMetaData.java
+++ b/core/src/main/java/org/elasticsearch/cluster/metadata/IndexMetaData.java
@@ -909,7 +909,7 @@ public class IndexMetaData implements Diffable<IndexMetaData>, FromXContentBuild
}
public static void toXContent(IndexMetaData indexMetaData, XContentBuilder builder, ToXContent.Params params) throws IOException {
- builder.startObject(indexMetaData.getIndex().getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexMetaData.getIndex().getName());
builder.field(KEY_VERSION, indexMetaData.getVersion());
builder.field(KEY_STATE, indexMetaData.getState().toString().toLowerCase(Locale.ENGLISH));
@@ -937,7 +937,7 @@ public class IndexMetaData implements Diffable<IndexMetaData>, FromXContentBuild
builder.endArray();
for (ObjectObjectCursor<String, Custom> cursor : indexMetaData.getCustoms()) {
- builder.startObject(cursor.key, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(cursor.key);
cursor.value.toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/cluster/metadata/IndexTemplateMetaData.java b/core/src/main/java/org/elasticsearch/cluster/metadata/IndexTemplateMetaData.java
index dce78a3dda..13d7e152ba 100644
--- a/core/src/main/java/org/elasticsearch/cluster/metadata/IndexTemplateMetaData.java
+++ b/core/src/main/java/org/elasticsearch/cluster/metadata/IndexTemplateMetaData.java
@@ -317,7 +317,7 @@ public class IndexTemplateMetaData extends AbstractDiffable<IndexTemplateMetaDat
@SuppressWarnings("unchecked")
public static void toXContent(IndexTemplateMetaData indexTemplateMetaData, XContentBuilder builder, ToXContent.Params params) throws IOException {
- builder.startObject(indexTemplateMetaData.name(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexTemplateMetaData.name());
builder.field("order", indexTemplateMetaData.order());
builder.field("template", indexTemplateMetaData.template());
@@ -355,7 +355,7 @@ public class IndexTemplateMetaData extends AbstractDiffable<IndexTemplateMetaDat
}
for (ObjectObjectCursor<String, IndexMetaData.Custom> cursor : indexTemplateMetaData.customs()) {
- builder.startObject(cursor.key, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(cursor.key);
cursor.value.toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetaData.java b/core/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetaData.java
index aa8f204953..2dc842ceaa 100644
--- a/core/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetaData.java
+++ b/core/src/main/java/org/elasticsearch/cluster/metadata/RepositoriesMetaData.java
@@ -197,7 +197,7 @@ public class RepositoriesMetaData extends AbstractDiffable<Custom> implements Me
* @param params serialization parameters
*/
public static void toXContent(RepositoryMetaData repository, XContentBuilder builder, ToXContent.Params params) throws IOException {
- builder.startObject(repository.name(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(repository.name());
builder.field("type", repository.type());
builder.startObject("settings");
repository.settings().toXContent(builder, params);
diff --git a/core/src/main/java/org/elasticsearch/cluster/node/DiscoveryNode.java b/core/src/main/java/org/elasticsearch/cluster/node/DiscoveryNode.java
index ea1973c405..0de737792f 100644
--- a/core/src/main/java/org/elasticsearch/cluster/node/DiscoveryNode.java
+++ b/core/src/main/java/org/elasticsearch/cluster/node/DiscoveryNode.java
@@ -335,7 +335,7 @@ public class DiscoveryNode implements Writeable<DiscoveryNode>, ToXContent {
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
- builder.startObject(getId(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(getId());
builder.field("name", getName());
builder.field("transport_address", getAddress().toString());
diff --git a/core/src/main/java/org/elasticsearch/common/settings/Settings.java b/core/src/main/java/org/elasticsearch/common/settings/Settings.java
index 5608eca394..ac1a3662c8 100644
--- a/core/src/main/java/org/elasticsearch/common/settings/Settings.java
+++ b/core/src/main/java/org/elasticsearch/common/settings/Settings.java
@@ -751,7 +751,7 @@ public final class Settings implements ToXContent {
}
} else {
for (Map.Entry<String, String> entry : settings.getAsMap().entrySet()) {
- builder.field(entry.getKey(), entry.getValue(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(entry.getKey(), entry.getValue());
}
}
return builder;
diff --git a/core/src/main/java/org/elasticsearch/common/unit/Fuzziness.java b/core/src/main/java/org/elasticsearch/common/unit/Fuzziness.java
index 9068c5ffa5..139dfd7549 100644
--- a/core/src/main/java/org/elasticsearch/common/unit/Fuzziness.java
+++ b/core/src/main/java/org/elasticsearch/common/unit/Fuzziness.java
@@ -43,7 +43,7 @@ public final class Fuzziness implements ToXContent, Writeable<Fuzziness> {
public static final Fuzziness ONE = new Fuzziness(1);
public static final Fuzziness TWO = new Fuzziness(2);
public static final Fuzziness AUTO = new Fuzziness("AUTO");
- public static final ParseField FIELD = new ParseField(X_FIELD_NAME.camelCase().getValue());
+ public static final ParseField FIELD = new ParseField(X_FIELD_NAME.value());
private final String fuzziness;
diff --git a/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilder.java b/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilder.java
index 2229d45840..83122646df 100644
--- a/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilder.java
+++ b/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilder.java
@@ -51,33 +51,12 @@ import java.util.Map;
import java.util.concurrent.TimeUnit;
/**
- *
+ * A utility to build XContent (ie json).
*/
public final class XContentBuilder implements BytesStream, Releasable {
- public enum FieldCaseConversion {
- /**
- * No conversion will occur.
- */
- NONE,
- /**
- * Camel Case will be converted to Underscore casing.
- */
- UNDERSCORE,
- /**
- * Underscore will be converted to Camel case.
- */
- CAMELCASE
- }
-
public final static DateTimeFormatter defaultDatePrinter = ISODateTimeFormat.dateTime().withZone(DateTimeZone.UTC);
- protected static FieldCaseConversion globalFieldCaseConversion = FieldCaseConversion.NONE;
-
- public static void globalFieldCaseConversion(FieldCaseConversion globalFieldCaseConversion) {
- XContentBuilder.globalFieldCaseConversion = globalFieldCaseConversion;
- }
-
public static XContentBuilder builder(XContent xContent) throws IOException {
return new XContentBuilder(xContent, new BytesStreamOutput());
}
@@ -94,10 +73,6 @@ public final class XContentBuilder implements BytesStream, Releasable {
private final OutputStream bos;
- private FieldCaseConversion fieldCaseConversion = globalFieldCaseConversion;
-
- private StringBuilder cachedStringBuilder;
-
private boolean humanReadable = false;
/**
@@ -130,11 +105,6 @@ public final class XContentBuilder implements BytesStream, Releasable {
this.generator = xContent.createGenerator(bos, filters, inclusive);
}
- public XContentBuilder fieldCaseConversion(FieldCaseConversion fieldCaseConversion) {
- this.fieldCaseConversion = fieldCaseConversion;
- return this;
- }
-
public XContentType contentType() {
return generator.contentType();
}
@@ -176,24 +146,12 @@ public final class XContentBuilder implements BytesStream, Releasable {
return this;
}
- public XContentBuilder startObject(String name, FieldCaseConversion conversion) throws IOException {
- field(name, conversion);
- startObject();
- return this;
- }
-
public XContentBuilder startObject(XContentBuilderString name) throws IOException {
field(name);
startObject();
return this;
}
- public XContentBuilder startObject(XContentBuilderString name, FieldCaseConversion conversion) throws IOException {
- field(name, conversion);
- startObject();
- return this;
- }
-
public XContentBuilder startObject() throws IOException {
generator.writeStartObject();
return this;
@@ -240,12 +198,6 @@ public final class XContentBuilder implements BytesStream, Releasable {
return this;
}
- public XContentBuilder startArray(String name, FieldCaseConversion conversion) throws IOException {
- field(name, conversion);
- startArray();
- return this;
- }
-
public XContentBuilder startArray(String name) throws IOException {
field(name);
startArray();
@@ -269,39 +221,13 @@ public final class XContentBuilder implements BytesStream, Releasable {
}
public XContentBuilder field(XContentBuilderString name) throws IOException {
- return field(name, fieldCaseConversion);
- }
-
- public XContentBuilder field(XContentBuilderString name, FieldCaseConversion conversion) throws IOException {
- if (conversion == FieldCaseConversion.UNDERSCORE) {
- generator.writeFieldName(name.underscore());
- } else if (conversion == FieldCaseConversion.CAMELCASE) {
- generator.writeFieldName(name.camelCase());
- } else {
- generator.writeFieldName(name.underscore());
- }
- return this;
+ return field(name.value());
}
public XContentBuilder field(String name) throws IOException {
- return field(name, fieldCaseConversion);
- }
-
- public XContentBuilder field(String name, FieldCaseConversion conversion) throws IOException {
if (name == null) {
throw new IllegalArgumentException("field name cannot be null");
}
- if (conversion == FieldCaseConversion.UNDERSCORE) {
- if (cachedStringBuilder == null) {
- cachedStringBuilder = new StringBuilder();
- }
- name = Strings.toUnderscoreCase(name, cachedStringBuilder);
- } else if (conversion == FieldCaseConversion.CAMELCASE) {
- if (cachedStringBuilder == null) {
- cachedStringBuilder = new StringBuilder();
- }
- name = Strings.toCamelCase(name, cachedStringBuilder);
- }
generator.writeFieldName(name);
return this;
}
@@ -336,16 +262,6 @@ public final class XContentBuilder implements BytesStream, Releasable {
return this;
}
- public XContentBuilder field(String name, String value, FieldCaseConversion conversion) throws IOException {
- field(name, conversion);
- if (value == null) {
- generator.writeNull();
- } else {
- generator.writeString(value);
- }
- return this;
- }
-
public XContentBuilder field(XContentBuilderString name, String value) throws IOException {
field(name);
if (value == null) {
@@ -356,16 +272,6 @@ public final class XContentBuilder implements BytesStream, Releasable {
return this;
}
- public XContentBuilder field(XContentBuilderString name, String value, FieldCaseConversion conversion) throws IOException {
- field(name, conversion);
- if (value == null) {
- generator.writeNull();
- } else {
- generator.writeString(value);
- }
- return this;
- }
-
public XContentBuilder field(String name, Integer value) throws IOException {
field(name);
if (value == null) {
diff --git a/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilderString.java b/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilderString.java
index ea1f6f1dd8..b056131f58 100644
--- a/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilderString.java
+++ b/core/src/main/java/org/elasticsearch/common/xcontent/XContentBuilderString.java
@@ -19,27 +19,18 @@
package org.elasticsearch.common.xcontent;
-import org.elasticsearch.common.Strings;
-
/**
- *
+ * TODO: remove this, it is just a wrapper that is no longer needed
*/
public class XContentBuilderString {
- private final XContentString underscore;
-
- private final XContentString camelCase;
+ private final String value;
public XContentBuilderString(String value) {
- underscore = new XContentString(Strings.toUnderscoreCase(value));
- camelCase = new XContentString(Strings.toCamelCase(value));
- }
-
- public XContentString underscore() {
- return underscore;
+ this.value = value;
}
- public XContentString camelCase() {
- return camelCase;
+ public String value() {
+ return value;
}
}
diff --git a/core/src/main/java/org/elasticsearch/index/fielddata/FieldDataStats.java b/core/src/main/java/org/elasticsearch/index/fielddata/FieldDataStats.java
index ec55ae9d59..3f3f1b550c 100644
--- a/core/src/main/java/org/elasticsearch/index/fielddata/FieldDataStats.java
+++ b/core/src/main/java/org/elasticsearch/index/fielddata/FieldDataStats.java
@@ -54,7 +54,7 @@ public class FieldDataStats implements Streamable, ToXContent {
this.memorySize += stats.memorySize;
this.evictions += stats.evictions;
if (stats.fields != null) {
- if (fields == null) {
+ if (fields == null) {
fields = stats.fields.clone();
} else {
assert !stats.fields.containsKey(null);
@@ -138,7 +138,7 @@ public class FieldDataStats implements Streamable, ToXContent {
final long[] values = fields.values;
for (int i = 0; i < keys.length; i++) {
if (keys[i] != null) {
- builder.startObject((String) keys[i], XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject((String) keys[i]);
builder.byteSizeField(Fields.MEMORY_SIZE_IN_BYTES, Fields.MEMORY_SIZE, values[i]);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/index/search/stats/SearchStats.java b/core/src/main/java/org/elasticsearch/index/search/stats/SearchStats.java
index bcabd7c5e1..83252103c4 100644
--- a/core/src/main/java/org/elasticsearch/index/search/stats/SearchStats.java
+++ b/core/src/main/java/org/elasticsearch/index/search/stats/SearchStats.java
@@ -309,7 +309,7 @@ public class SearchStats implements Streamable, ToXContent {
if (groupStats != null && !groupStats.isEmpty()) {
builder.startObject(Fields.GROUPS);
for (Map.Entry<String, Stats> entry : groupStats.entrySet()) {
- builder.startObject(entry.getKey(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(entry.getKey());
entry.getValue().toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/index/shard/IndexingStats.java b/core/src/main/java/org/elasticsearch/index/shard/IndexingStats.java
index 27cda2ca1c..beafbefddc 100644
--- a/core/src/main/java/org/elasticsearch/index/shard/IndexingStats.java
+++ b/core/src/main/java/org/elasticsearch/index/shard/IndexingStats.java
@@ -258,7 +258,7 @@ public class IndexingStats implements Streamable, ToXContent {
if (typeStats != null && !typeStats.isEmpty()) {
builder.startObject(Fields.TYPES);
for (Map.Entry<String, Stats> entry : typeStats.entrySet()) {
- builder.startObject(entry.getKey(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(entry.getKey());
entry.getValue().toXContent(builder, params);
builder.endObject();
}
diff --git a/core/src/main/java/org/elasticsearch/index/snapshots/blobstore/BlobStoreIndexShardSnapshots.java b/core/src/main/java/org/elasticsearch/index/snapshots/blobstore/BlobStoreIndexShardSnapshots.java
index 1e0d55f2f6..7b7a8dd982 100644
--- a/core/src/main/java/org/elasticsearch/index/snapshots/blobstore/BlobStoreIndexShardSnapshots.java
+++ b/core/src/main/java/org/elasticsearch/index/snapshots/blobstore/BlobStoreIndexShardSnapshots.java
@@ -221,7 +221,7 @@ public class BlobStoreIndexShardSnapshots implements Iterable<SnapshotFiles>, To
// Then we list all snapshots with list of all blobs that are used by the snapshot
builder.startObject(Fields.SNAPSHOTS);
for (SnapshotFiles snapshot : shardSnapshots) {
- builder.startObject(snapshot.snapshot(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(snapshot.snapshot());
builder.startArray(Fields.FILES);
for (FileInfo fileInfo : snapshot.indexFiles()) {
builder.value(fileInfo.name());
diff --git a/core/src/main/java/org/elasticsearch/monitor/fs/FsInfo.java b/core/src/main/java/org/elasticsearch/monitor/fs/FsInfo.java
index b97457af70..55ce159f39 100644
--- a/core/src/main/java/org/elasticsearch/monitor/fs/FsInfo.java
+++ b/core/src/main/java/org/elasticsearch/monitor/fs/FsInfo.java
@@ -166,13 +166,13 @@ public class FsInfo implements Iterable<FsInfo.Path>, Streamable, ToXContent {
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
builder.startObject();
if (path != null) {
- builder.field(Fields.PATH, path, XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(Fields.PATH, path);
}
if (mount != null) {
- builder.field(Fields.MOUNT, mount, XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(Fields.MOUNT, mount);
}
if (type != null) {
- builder.field(Fields.TYPE, type, XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(Fields.TYPE, type);
}
if (total != -1) {
diff --git a/core/src/main/java/org/elasticsearch/monitor/jvm/JvmStats.java b/core/src/main/java/org/elasticsearch/monitor/jvm/JvmStats.java
index 276ef537bc..c78ed60806 100644
--- a/core/src/main/java/org/elasticsearch/monitor/jvm/JvmStats.java
+++ b/core/src/main/java/org/elasticsearch/monitor/jvm/JvmStats.java
@@ -195,7 +195,7 @@ public class JvmStats implements Streamable, ToXContent {
builder.startObject(Fields.POOLS);
for (MemoryPool pool : mem) {
- builder.startObject(pool.getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(pool.getName());
builder.byteSizeField(Fields.USED_IN_BYTES, Fields.USED, pool.used);
builder.byteSizeField(Fields.MAX_IN_BYTES, Fields.MAX, pool.max);
@@ -219,7 +219,7 @@ public class JvmStats implements Streamable, ToXContent {
builder.startObject(Fields.COLLECTORS);
for (GarbageCollector collector : gc) {
- builder.startObject(collector.getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(collector.getName());
builder.field(Fields.COLLECTION_COUNT, collector.getCollectionCount());
builder.timeValueField(Fields.COLLECTION_TIME_IN_MILLIS, Fields.COLLECTION_TIME, collector.collectionTime);
builder.endObject();
@@ -232,7 +232,7 @@ public class JvmStats implements Streamable, ToXContent {
if (bufferPools != null) {
builder.startObject(Fields.BUFFER_POOLS);
for (BufferPool bufferPool : bufferPools) {
- builder.startObject(bufferPool.getName(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(bufferPool.getName());
builder.field(Fields.COUNT, bufferPool.getCount());
builder.byteSizeField(Fields.USED_IN_BYTES, Fields.USED, bufferPool.used);
builder.byteSizeField(Fields.TOTAL_CAPACITY_IN_BYTES, Fields.TOTAL_CAPACITY, bufferPool.totalCapacity);
diff --git a/core/src/main/java/org/elasticsearch/rest/AbstractRestChannel.java b/core/src/main/java/org/elasticsearch/rest/AbstractRestChannel.java
index 89434b1ce4..aa00943ed0 100644
--- a/core/src/main/java/org/elasticsearch/rest/AbstractRestChannel.java
+++ b/core/src/main/java/org/elasticsearch/rest/AbstractRestChannel.java
@@ -71,15 +71,6 @@ public abstract class AbstractRestChannel implements RestChannel {
}
builder.humanReadable(request.paramAsBoolean("human", builder.humanReadable()));
-
- String casing = request.param("case");
- if (casing != null && "camelCase".equals(casing)) {
- builder.fieldCaseConversion(XContentBuilder.FieldCaseConversion.CAMELCASE);
- } else {
- // we expect all REST interfaces to write results in underscore casing, so
- // no need for double casing
- builder.fieldCaseConversion(XContentBuilder.FieldCaseConversion.NONE);
- }
return builder;
}
diff --git a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/alias/get/RestGetAliasesAction.java b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/alias/get/RestGetAliasesAction.java
index da439c63d5..d18b013efb 100644
--- a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/alias/get/RestGetAliasesAction.java
+++ b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/alias/get/RestGetAliasesAction.java
@@ -83,7 +83,7 @@ public class RestGetAliasesAction extends BaseRestHandler {
builder.startObject();
for (ObjectObjectCursor<String, List<AliasMetaData>> entry : response.getAliases()) {
- builder.startObject(entry.key, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(entry.key);
builder.startObject(Fields.ALIASES);
for (AliasMetaData alias : entry.value) {
AliasMetaData.Builder.toXContent(alias, builder, ToXContent.EMPTY_PARAMS);
diff --git a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetMappingAction.java b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetMappingAction.java
index 12c29bb781..29012ba86f 100644
--- a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetMappingAction.java
+++ b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/mapping/get/RestGetMappingAction.java
@@ -88,7 +88,7 @@ public class RestGetMappingAction extends BaseRestHandler {
if (indexEntry.value.isEmpty()) {
continue;
}
- builder.startObject(indexEntry.key, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(indexEntry.key);
builder.startObject(Fields.MAPPINGS);
for (ObjectObjectCursor<String, MappingMetaData> typeEntry : indexEntry.value) {
builder.field(typeEntry.key);
diff --git a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/settings/RestGetSettingsAction.java b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/settings/RestGetSettingsAction.java
index 300423a6ed..8c24e2c68e 100644
--- a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/settings/RestGetSettingsAction.java
+++ b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/settings/RestGetSettingsAction.java
@@ -77,7 +77,7 @@ public class RestGetSettingsAction extends BaseRestHandler {
if (cursor.value.getAsMap().isEmpty()) {
continue;
}
- builder.startObject(cursor.key, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(cursor.key);
builder.startObject("settings");
cursor.value.toXContent(builder, request);
builder.endObject();
diff --git a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/upgrade/RestUpgradeAction.java b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/upgrade/RestUpgradeAction.java
index 60a781f90b..7c4c0f51b8 100644
--- a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/upgrade/RestUpgradeAction.java
+++ b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/upgrade/RestUpgradeAction.java
@@ -89,7 +89,7 @@ public class RestUpgradeAction extends BaseRestHandler {
buildBroadcastShardsHeader(builder, request, response);
builder.startObject("upgraded_indices");
for (Map.Entry<String, Tuple<Version, String>> entry : response.versions().entrySet()) {
- builder.startObject(entry.getKey(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(entry.getKey());
builder.field("upgrade_version", entry.getValue().v1());
builder.field("oldest_lucene_segment_version", entry.getValue().v2());
builder.endObject();
diff --git a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/validate/query/RestValidateQueryAction.java b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/validate/query/RestValidateQueryAction.java
index 86d6e9d608..c0a209375b 100644
--- a/core/src/main/java/org/elasticsearch/rest/action/admin/indices/validate/query/RestValidateQueryAction.java
+++ b/core/src/main/java/org/elasticsearch/rest/action/admin/indices/validate/query/RestValidateQueryAction.java
@@ -102,7 +102,7 @@ public class RestValidateQueryAction extends BaseRestHandler {
for (QueryExplanation explanation : response.getQueryExplanation()) {
builder.startObject();
if (explanation.getIndex() != null) {
- builder.field(INDEX_FIELD, explanation.getIndex(), XContentBuilder.FieldCaseConversion.NONE);
+ builder.field(INDEX_FIELD, explanation.getIndex());
}
builder.field(VALID_FIELD, explanation.isValid());
if (explanation.getError() != null) {
diff --git a/core/src/main/java/org/elasticsearch/search/suggest/completion/CompletionStats.java b/core/src/main/java/org/elasticsearch/search/suggest/completion/CompletionStats.java
index 5ec517dd8d..7bb96aeedc 100644
--- a/core/src/main/java/org/elasticsearch/search/suggest/completion/CompletionStats.java
+++ b/core/src/main/java/org/elasticsearch/search/suggest/completion/CompletionStats.java
@@ -80,7 +80,7 @@ public class CompletionStats implements Streamable, ToXContent {
} else {
out.writeBoolean(true);
out.writeVInt(fields.size());
-
+
assert !fields.containsKey(null);
final Object[] keys = fields.keys;
final long[] values = fields.values;
@@ -105,7 +105,7 @@ public class CompletionStats implements Streamable, ToXContent {
final long[] values = fields.values;
for (int i = 0; i < keys.length; i++) {
if (keys[i] != null) {
- builder.startObject((String) keys[i], XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject((String) keys[i]);
builder.byteSizeField(Fields.SIZE_IN_BYTES, Fields.SIZE, values[i]);
builder.endObject();
}
@@ -137,7 +137,7 @@ public class CompletionStats implements Streamable, ToXContent {
sizeInBytes += completion.getSizeInBytes();
if (completion.fields != null) {
- if (fields == null) {
+ if (fields == null) {
fields = completion.fields.clone();
} else {
assert !completion.fields.containsKey(null);
diff --git a/core/src/main/java/org/elasticsearch/threadpool/ThreadPool.java b/core/src/main/java/org/elasticsearch/threadpool/ThreadPool.java
index 3088107339..f612a770b8 100644
--- a/core/src/main/java/org/elasticsearch/threadpool/ThreadPool.java
+++ b/core/src/main/java/org/elasticsearch/threadpool/ThreadPool.java
@@ -889,7 +889,7 @@ public class ThreadPool extends AbstractComponent implements Closeable {
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
- builder.startObject(name, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(name);
builder.field(Fields.TYPE, type.getType());
if (min != -1) {
builder.field(Fields.MIN, min);
diff --git a/core/src/main/java/org/elasticsearch/threadpool/ThreadPoolStats.java b/core/src/main/java/org/elasticsearch/threadpool/ThreadPoolStats.java
index af53482c62..f16d6a886d 100644
--- a/core/src/main/java/org/elasticsearch/threadpool/ThreadPoolStats.java
+++ b/core/src/main/java/org/elasticsearch/threadpool/ThreadPoolStats.java
@@ -112,7 +112,7 @@ public class ThreadPoolStats implements Streamable, ToXContent, Iterable<ThreadP
@Override
public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
- builder.startObject(name, XContentBuilder.FieldCaseConversion.NONE);
+ builder.startObject(name);
if (threads != -1) {
builder.field(Fields.THREADS, threads);
}
diff --git a/core/src/test/java/org/elasticsearch/common/xcontent/builder/XContentBuilderTests.java b/core/src/test/java/org/elasticsearch/common/xcontent/builder/XContentBuilderTests.java
index 36b16d6a17..321ebc40e2 100644
--- a/core/src/test/java/org/elasticsearch/common/xcontent/builder/XContentBuilderTests.java
+++ b/core/src/test/java/org/elasticsearch/common/xcontent/builder/XContentBuilderTests.java
@@ -47,8 +47,6 @@ import java.util.Locale;
import java.util.Map;
import java.util.TimeZone;
-import static org.elasticsearch.common.xcontent.XContentBuilder.FieldCaseConversion.CAMELCASE;
-import static org.elasticsearch.common.xcontent.XContentBuilder.FieldCaseConversion.UNDERSCORE;
import static org.hamcrest.Matchers.equalTo;
public class XContentBuilderTests extends ESTestCase {
@@ -169,16 +167,6 @@ public class XContentBuilderTests extends ESTestCase {
assertThat(sData, equalTo("{\"name\":\"something\", source : { test : \"value\" },\"name2\":\"something2\"}"));
}
- public void testFieldCaseConversion() throws Exception {
- XContentBuilder builder = XContentFactory.contentBuilder(XContentType.JSON).fieldCaseConversion(CAMELCASE);
- builder.startObject().field("test_name", "value").endObject();
- assertThat(builder.string(), equalTo("{\"testName\":\"value\"}"));
-
- builder = XContentFactory.contentBuilder(XContentType.JSON).fieldCaseConversion(UNDERSCORE);
- builder.startObject().field("testName", "value").endObject();
- assertThat(builder.string(), equalTo("{\"test_name\":\"value\"}"));
- }
-
public void testByteConversion() throws Exception {
XContentBuilder builder = XContentFactory.contentBuilder(XContentType.JSON);
builder.startObject().field("test_name", (Byte)(byte)120).endObject();