From dcd2bbd9a96a30b8759f47c5e62e092b8aa86e04 Mon Sep 17 00:00:00 2001
From: kimchy <kimchy@gmail.com>
Date: Sat, 4 Jun 2011 01:26:41 +0300
Subject: [PATCH] ',' can be used in type name, closes #996.

---
 .../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 da67b54b660..bc9f5dd51b2 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
@@ -171,6 +171,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(",")) {
+                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());
             }