honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Send the update-mapping events before actually indexing into the shard, because the latter may generate exceptions (like when the shard is not yet ready to accept new docs). 

Also improved the SimpleDeleteMappingTest, as it make take a while until the update-mapping event is processed.

Closes #3782
This commit is contained in:
Boaz Leskes 2013-09-26 14:31:07 +02:00
parent c63869b0be
commit b66f3c6834
2 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/main/java/org/elasticsearch/action/index/TransportIndexAction.java
View File

@ -202,6 +202,9 @@ public class TransportIndexAction extends TransportShardReplicationOperationActi
.version(request.version())
.versionType(request.versionType())
.origin(Engine.Operation.Origin.PRIMARY);
if (index.parsedDoc().mappingsModified()) {
updateMappingOnMaster(request);
}
indexShard.index(index);
version = index.version();
op = index;
@ -211,6 +214,9 @@ public class TransportIndexAction extends TransportShardReplicationOperationActi
.version(request.version())
.versionType(request.versionType())
.origin(Engine.Operation.Origin.PRIMARY);
if (create.parsedDoc().mappingsModified()) {
updateMappingOnMaster(request);
}
indexShard.create(create);
version = create.version();
op = create;
@ -223,9 +229,7 @@ public class TransportIndexAction extends TransportShardReplicationOperationActi
// ignore
}
}
if (op.parsedDoc().mappingsModified()) {
updateMappingOnMaster(request);
}
// update the version on the request, so it will be used for the replicas
request.version(version);

10
src/test/java/org/elasticsearch/indices/mapping/SimpleDeleteMappingTests.java
View File

@ -51,12 +51,10 @@ public class SimpleDeleteMappingTests extends AbstractIntegrationTest {
}
ClusterState clusterState = client().admin().cluster().prepareState().execute().actionGet().getState();
if (!clusterState.metaData().index("test").mappings().containsKey("type1")) {
logger.info("mapping of type [{}] is not in the clusterstate version: [{}] localNode: [{}] nodes: [{}]", "type1",
clusterState.version(), clusterState.nodes().localNode(), clusterState.nodes().dataNodes().values());
logger.info("Current mappings in clusterstate: [{}]", clusterState.metaData().index("test").mappings().keySet());
}
assertThat(clusterState.metaData().index("test").mappings().containsKey("type1"), equalTo(true));
for (int i = 0; i < 10 && !clusterState.metaData().index("test").mappings().containsKey("type1"); i++, Thread.sleep(100)) ;
assertThat(clusterState.metaData().index("test").mappings(), hasKey("type1"));
GetMappingsResponse mappingsResponse = client().admin().indices().prepareGetMappings("test").setTypes("type1").execute().actionGet();
assertThat(mappingsResponse.getMappings().get("test").get("type1"), notNullValue());