From 9e2143f3437629cfb8a3482f3664c04f61019b58 Mon Sep 17 00:00:00 2001
From: kimchy <kimchy@gmail.com>
Date: Mon, 7 Mar 2011 19:17:39 +0200
Subject: [PATCH] add a warning for types not to include dots (can't really
 throw a failure because of it since it will break backward)

---
 .../java/org/elasticsearch/index/mapper/MapperService.java     | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/MapperService.java b/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/MapperService.java
index 04bcfc10c14..07176d5fff1 100644
--- a/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/MapperService.java
+++ b/modules/elasticsearch/src/main/java/org/elasticsearch/index/mapper/MapperService.java
@@ -153,6 +153,9 @@ public class MapperService extends AbstractIndexComponent implements Iterable<Do
             if (mapper.type().contains("#")) {
                 throw new InvalidTypeNameException("mapping type name [" + mapper.type() + "] should not include '#' in it");
             }
+            if (mapper.type().contains(".")) {
+                logger.warn("Type [{}] contains a '.', it is recommended not to include it within a type name", mapper.type());
+            }
             remove(mapper.type()); // first remove it (in case its an update, we need to remove the aggregated mappers)
             mappers = newMapBuilder(mappers).put(mapper.type(), mapper).immutableMap();
             mapper.addFieldMapperListener(fieldMapperListener, true);