Routing mapping description rewritten #55

Open
wants to merge 4 commits into from

1 participant

@derrickburns

Please review to make sure I got this right. I did not understand the original description.

@kimchy kimchy commented on an outdated diff Jul 17, 2011
guide/reference/mapping/routing-field.textile
@@ -5,26 +5,26 @@ cat: guide
sidebar: reference_mapping
---
-p. The routing field allows to control the @_routing@ aspect when indexing data and explicit routing control is required.
+p. When an index is sharded, a shard must be selected to index an incoming document. The shard chosen is a function of the @routing value@. The @_routing@ field mapping specifies how the routing value is determined.
@kimchy
elastic member
kimchy added a note Jul 17, 2011

By default, the routing value is based on the document id, so its not really that the routing value is required. Maybe it makes sense to add it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@kimchy kimchy commented on an outdated diff Jul 17, 2011
guide/reference/mapping/routing-field.textile
h1. store / index
-p. The first thing the @_routing@ mapping does is to store the routing value provided (@store@ set to @yes@) and index it (@index@ set to @not_analyzed@). The reason why the routing is stored by default is so reindexing data will be possible if the routing value is completely external and not part of the docs.
+p. By default, the routing value is stored (@store@ set to @yes@) as the value of the @_routing@ field of the document and indexed (@index@ set to @not_analyzed@). Storing the routing value as a field in the document enables easy re-indexing, should that be necessary.
@kimchy
elastic member
kimchy added a note Jul 17, 2011

I would add that by default, if the routing value is used, then its stored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@derrickburns

Ok, I tried again. Let me know if this is right.

@derrickburns

Do you want this? If not, please close the issue. Thanks.

@derrickburns derrickburns reopened this Aug 8, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment