DATAMONGO-1261 - Release version 1.7.2.RELEASE (Fowler SR2).

We now push the aliased field name down the aggregation pipeline for projections including operations. This allows to reference them in a later stage. Prior to this change the field reference was potentially resolved to the target field of the operation which did not result in an error but lead to false results.

Original pull request: #311.

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,9 +0,0 @@
-Thank you for proposing a pull request. This template will guide you through the essential steps necessary for a pull request.
-Make sure that:
-
-- [ ] You have read the [Spring Data contribution guidelines](https://github.com/spring-projects/spring-data-build/blob/master/CONTRIBUTING.adoc).
-- [ ] There is a ticket in the bug tracker for the project in our [JIRA](https://jira.spring.io/browse/DATAMONGO).
-- [ ] You use the code formatters provided [here](https://github.com/spring-projects/spring-data-build/tree/master/etc/ide) and have them applied to your changes. Don’t submit any formatting related changes.
-- [ ] You submit test cases (unit or integration tests) that back your changes.
-- [ ] You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).
-- [ ] You provide your full name and an email address registered with your GitHub account. If you’re a first-time submitter, make sure you have completed the [Contributor’s License Agreement form](https://support.springsource.com/spring_committer_signup).

.travis.yml

@@ -3,29 +3,13 @@ language: java
 jdk:
   - oraclejdk8
 
-before_script:
-  - mongod --version
+services:
+  - mongodb
 
 env:
   matrix:
     - PROFILE=ci
     - PROFILE=mongo-next
-    - PROFILE=mongo3
-    - PROFILE=mongo3-next
-    - PROFILE=mongo31
-    - PROFILE=mongo32
-    - PROFILE=mongo33
-    - PROFILE=mongo34-next
-
-# Current MongoDB version is 2.4.2 as of 2016-04, see https://github.com/travis-ci/travis-ci/issues/3694
-# apt-get starts a MongoDB instance so it's not started using before_script
-addons:
-  apt:
-    sources:
-    - mongodb-3.2-precise
-    packages:
-    - mongodb-org-server
-    - mongodb-org-shell
 
 sudo: false
 

CODE_OF_CONDUCT.adoc

@@ -1,27 +0,0 @@
-= Contributor Code of Conduct
-
-As contributors and maintainers of this project, and in the interest of fostering an open and welcoming community, we pledge to respect all people who contribute through reporting issues, posting feature requests, updating documentation, submitting pull requests or patches, and other activities.
-
-We are committed to making participation in this project a harassment-free experience for everyone, regardless of level of experience, gender, gender identity and expression, sexual orientation, disability, personal appearance, body size, race, ethnicity, age, religion, or nationality.
-
-Examples of unacceptable behavior by participants include:
-
-* The use of sexualized language or imagery
-* Personal attacks
-* Trolling or insulting/derogatory comments
-* Public or private harassment
-* Publishing other's private information, such as physical or electronic addresses,
-  without explicit permission
-* Other unethical or unprofessional conduct
-
-Project maintainers have the right and responsibility to remove, edit, or reject comments, commits, code, wiki edits, issues, and other contributions that are not aligned to this Code of Conduct, or to ban temporarily or permanently any contributor for other behaviors that they deem inappropriate, threatening, offensive, or harmful.
-
-By adopting this Code of Conduct, project maintainers commit themselves to fairly and consistently applying these principles to every aspect of managing this project. Project maintainers who do not follow or enforce the Code of Conduct may be permanently removed from the project team.
-
-This Code of Conduct applies both within project spaces and in public spaces when an individual is representing the project or its community.
-
-Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting a project maintainer at spring-code-of-conduct@pivotal.io.
-All complaints will be reviewed and investigated and will result in a response that is deemed necessary and appropriate to the circumstances.
-Maintainers are obligated to maintain confidentiality with regard to the reporter of an incident.
-
-This Code of Conduct is adapted from the http://contributor-covenant.org[Contributor Covenant], version 1.3.0, available at http://contributor-covenant.org/version/1/3/0/[contributor-covenant.org/version/1/3/0/].

CONTRIBUTING.MD

@@ -0,0 +1 @@
+You find the contribution guidelines for Spring Data projects [here](https://github.com/spring-projects/spring-data-build/blob/master/CONTRIBUTING.md).

CONTRIBUTING.adoc

@@ -1,3 +0,0 @@
-= Spring Data contribution guidelines
-
-You find the contribution guidelines for Spring Data projects https://github.com/spring-projects/spring-data-build/blob/master/CONTRIBUTING.adoc[here].

pom.xml

@@ -1,11 +1,11 @@
-<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
 
 	<modelVersion>4.0.0</modelVersion>
 
 	<groupId>org.springframework.data</groupId>
 	<artifactId>spring-data-mongodb-parent</artifactId>
-	<version>1.9.11.RELEASE</version>
+	<version>1.7.2.RELEASE</version>
 	<packaging>pom</packaging>
 
 	<name>Spring Data MongoDB</name>
@@ -15,7 +15,7 @@
 	<parent>
 		<groupId>org.springframework.data.build</groupId>
 		<artifactId>spring-data-parent</artifactId>
-		<version>1.8.11.RELEASE</version>
+		<version>1.6.2.RELEASE</version>
 	</parent>
 
 	<modules>
@@ -28,8 +28,8 @@
 	<properties>
 		<project.type>multi</project.type>
 		<dist.id>spring-data-mongodb</dist.id>
-		<springdata.commons>1.12.11.RELEASE</springdata.commons>
-		<mongo>2.14.3</mongo>
+		<springdata.commons>1.10.2.RELEASE</springdata.commons>
+		<mongo>2.13.0</mongo>
 		<mongo.osgi>2.13.0</mongo.osgi>
 	</properties>
 
@@ -107,7 +107,7 @@
 
 			<id>mongo-next</id>
 			<properties>
-				<mongo>2.15.0-SNAPSHOT</mongo>
+				<mongo>2.14.0-SNAPSHOT</mongo>
 			</properties>
 
 			<repositories>
@@ -123,7 +123,7 @@
 
 			<id>mongo3</id>
 			<properties>
-				<mongo>3.0.4</mongo>
+				<mongo>3.0.2</mongo>
 			</properties>
 
 		</profile>
@@ -132,7 +132,7 @@
 
 			<id>mongo3-next</id>
 			<properties>
-				<mongo>3.0.5-SNAPSHOT</mongo>
+				<mongo>3.0.0-SNAPSHOT</mongo>
 			</properties>
 
 			<repositories>
@@ -143,70 +143,6 @@
 			</repositories>
 
 		</profile>
-
-		<profile>
-
-			<id>mongo31</id>
-			<properties>
-				<mongo>3.1.1</mongo>
-			</properties>
-
-		</profile>
-
-		<profile>
-
-			<id>mongo32</id>
-			<properties>
-				<mongo>3.2.2</mongo>
-			</properties>
-
-		</profile>
-
-		<profile>
-
-			<id>mongo33</id>
-			<properties>
-				<mongo>3.3.0</mongo>
-			</properties>
-
-			<repositories>
-				<repository>
-					<id>mongo-snapshots</id>
-					<url>https://oss.sonatype.org/content/repositories/snapshots</url>
-				</repository>
-			</repositories>
-
-		</profile>
-
-		<profile>
-
-			<id>mongo34-next</id>
-			<properties>
-				<mongo>3.4.0-SNAPSHOT</mongo>
-			</properties>
-
-			<repositories>
-				<repository>
-					<id>mongo-snapshots</id>
-					<url>https://oss.sonatype.org/content/repositories/snapshots</url>
-				</repository>
-			</repositories>
-
-		</profile>
-
-		<profile>
-			<id>release</id>
-			<build>
-				<plugins>
-					<plugin>
-						<groupId>org.jfrog.buildinfo</groupId>
-						<artifactId>artifactory-maven-plugin</artifactId>
-						<inherited>false</inherited>
-					</plugin>
-				</plugins>
-			</build>
-		</profile>
-
 	</profiles>
 
 	<dependencies>

spring-data-mongodb-cross-store/pom.xml

@@ -6,7 +6,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-mongodb-parent</artifactId>
-		<version>1.9.11.RELEASE</version>
+		<version>1.7.2.RELEASE</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 
@@ -48,7 +48,7 @@
 		<dependency>
 			<groupId>org.springframework.data</groupId>
 			<artifactId>spring-data-mongodb</artifactId>
-			<version>1.9.11.RELEASE</version>
+			<version>1.7.2.RELEASE</version>
 		</dependency>
 
 		<dependency>

spring-data-mongodb-cross-store/src/main/java/org/springframework/data/mongodb/crossstore/MongoChangeSetPersister.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -37,8 +37,6 @@ import com.mongodb.MongoException;
 /**
  * @author Thomas Risberg
  * @author Oliver Gierke
- * @author Alex Vengrovsk
- * @author Mark Paluch
  */
 public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 
@@ -47,7 +45,7 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 	private static final String ENTITY_FIELD_NAME = "_entity_field_name";
 	private static final String ENTITY_FIELD_CLASS = "_entity_field_class";
 
-	private final Logger log = LoggerFactory.getLogger(getClass());
+	protected final Logger log = LoggerFactory.getLogger(getClass());
 
 	private MongoTemplate mongoTemplate;
 	private EntityManagerFactory entityManagerFactory;
@@ -78,25 +76,25 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 		dbk.put(ENTITY_ID, id);
 		dbk.put(ENTITY_CLASS, entityClass.getName());
 		if (log.isDebugEnabled()) {
-			log.debug("Loading MongoDB data for {}", dbk);
+			log.debug("Loading MongoDB data for " + dbk);
 		}
 		mongoTemplate.execute(collName, new CollectionCallback<Object>() {
 			public Object doInCollection(DBCollection collection) throws MongoException, DataAccessException {
 				for (DBObject dbo : collection.find(dbk)) {
 					String key = (String) dbo.get(ENTITY_FIELD_NAME);
 					if (log.isDebugEnabled()) {
-						log.debug("Processing key: {}", key);
+						log.debug("Processing key: " + key);
 					}
 					if (!changeSet.getValues().containsKey(key)) {
 						String className = (String) dbo.get(ENTITY_FIELD_CLASS);
 						if (className == null) {
-							throw new DataIntegrityViolationException(
-									"Unble to convert property " + key + ": Invalid metadata, " + ENTITY_FIELD_CLASS + " not available");
+							throw new DataIntegrityViolationException("Unble to convert property " + key + ": Invalid metadata, "
+									+ ENTITY_FIELD_CLASS + " not available");
 						}
 						Class<?> clazz = ClassUtils.resolveClassName(className, ClassUtils.getDefaultClassLoader());
 						Object value = mongoTemplate.getConverter().read(clazz, dbo);
 						if (log.isDebugEnabled()) {
-							log.debug("Adding to ChangeSet: {}", key);
+							log.debug("Adding to ChangeSet: " + key);
 						}
 						changeSet.set(key, value);
 					}
@@ -111,9 +109,9 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 	 * @see org.springframework.data.crossstore.ChangeSetPersister#getPersistentId(org.springframework.data.crossstore.ChangeSetBacked, org.springframework.data.crossstore.ChangeSet)
 	 */
 	public Object getPersistentId(ChangeSetBacked entity, ChangeSet cs) throws DataAccessException {
-		if (log.isDebugEnabled()) {
-			log.debug("getPersistentId called on {}", entity);
-		}
+
+		log.debug("getPersistentId called on " + entity);
+
 		if (entityManagerFactory == null) {
 			throw new DataAccessResourceFailureException("EntityManagerFactory cannot be null");
 		}
@@ -132,7 +130,7 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 		}
 
 		if (log.isDebugEnabled()) {
-			log.debug("Flush: changeset: {}", cs.getValues());
+			log.debug("Flush: changeset: " + cs.getValues());
 		}
 
 		String collName = getCollectionNameForEntity(entity.getClass());
@@ -154,7 +152,7 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 				});
 				if (value == null) {
 					if (log.isDebugEnabled()) {
-						log.debug("Flush: removing: {}", dbQuery);
+						log.debug("Flush: removing: " + dbQuery);
 					}
 					mongoTemplate.execute(collName, new CollectionCallback<Object>() {
 						public Object doInCollection(DBCollection collection) throws MongoException, DataAccessException {
@@ -166,7 +164,7 @@ public class MongoChangeSetPersister implements ChangeSetPersister<Object> {
 					final DBObject dbDoc = new BasicDBObject();
 					dbDoc.putAll(dbQuery);
 					if (log.isDebugEnabled()) {
-						log.debug("Flush: saving: {}", dbQuery);
+						log.debug("Flush: saving: " + dbQuery);
 					}
 					mongoTemplate.getConverter().write(value, dbDoc);
 					dbDoc.put(ENTITY_FIELD_CLASS, value.getClass().getName());

spring-data-mongodb-distribution/pom.xml

@@ -13,7 +13,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-mongodb-parent</artifactId>
-		<version>1.9.11.RELEASE</version>
+		<version>1.7.2.RELEASE</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 

spring-data-mongodb-log4j/pom.xml

@@ -1,11 +1,11 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
 	<modelVersion>4.0.0</modelVersion>
-	
+
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-mongodb-parent</artifactId>
-		<version>1.9.11.RELEASE</version>
+		<version>1.7.2.RELEASE</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 

spring-data-mongodb-log4j/src/main/java/org/springframework/data/mongodb/log4j/MongoLog4jAppender.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,7 +30,6 @@ import com.mongodb.BasicDBList;
 import com.mongodb.BasicDBObject;
 import com.mongodb.DB;
 import com.mongodb.Mongo;
-import com.mongodb.MongoClient;
 import com.mongodb.WriteConcern;
 
 /**
@@ -38,7 +37,6 @@ import com.mongodb.WriteConcern;
  * 
  * @author Jon Brisbin
  * @author Oliver Gierke
- * @auhtor Christoph Strobl
  */
 public class MongoLog4jAppender extends AppenderSkeleton {
 
@@ -60,8 +58,8 @@ public class MongoLog4jAppender extends AppenderSkeleton {
 	protected String collectionPattern = "%c";
 	protected PatternLayout collectionLayout = new PatternLayout(collectionPattern);
 	protected String applicationId = System.getProperty("APPLICATION_ID", null);
-	protected WriteConcern warnOrHigherWriteConcern = WriteConcern.ACKNOWLEDGED;
-	protected WriteConcern infoOrLowerWriteConcern = WriteConcern.UNACKNOWLEDGED;
+	protected WriteConcern warnOrHigherWriteConcern = WriteConcern.SAFE;
+	protected WriteConcern infoOrLowerWriteConcern = WriteConcern.NORMAL;
 	protected Mongo mongo;
 	protected DB db;
 
@@ -130,7 +128,7 @@ public class MongoLog4jAppender extends AppenderSkeleton {
 	}
 
 	protected void connectToMongo() throws UnknownHostException {
-		this.mongo = new MongoClient(host, port);
+		this.mongo = new Mongo(host, port);
 		this.db = mongo.getDB(database);
 	}
 
@@ -162,7 +160,7 @@ public class MongoLog4jAppender extends AppenderSkeleton {
 
 		// Copy properties into document
 		Map<Object, Object> props = event.getProperties();
-		if (null != props && !props.isEmpty()) {
+		if (null != props && props.size() > 0) {
 			BasicDBObject propsDbo = new BasicDBObject();
 			for (Map.Entry<Object, Object> entry : props.entrySet()) {
 				propsDbo.put(entry.getKey().toString(), entry.getValue().toString());

spring-data-mongodb-log4j/src/test/java/org/springframework/data/mongodb/log4j/MongoLog4jAppenderIntegrationTests.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,44 +22,37 @@ import java.util.Calendar;
 
 import org.apache.log4j.Logger;
 import org.apache.log4j.MDC;
-import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
 
-import com.mongodb.BasicDBObject;
 import com.mongodb.DB;
 import com.mongodb.DBCursor;
-import com.mongodb.MongoClient;
+import com.mongodb.Mongo;
 
 /**
  * Integration tests for {@link MongoLog4jAppender}.
  * 
  * @author Jon Brisbin
  * @author Oliver Gierke
- * @author Christoph Strobl
  */
 public class MongoLog4jAppenderIntegrationTests {
 
 	static final String NAME = MongoLog4jAppenderIntegrationTests.class.getName();
 
-	private static final Logger log = Logger.getLogger(NAME);
-	MongoClient mongo;
+	Logger log = Logger.getLogger(NAME);
+	Mongo mongo;
 	DB db;
 	String collection;
 
 	@Before
 	public void setUp() throws Exception {
 
-		mongo = new MongoClient("localhost", 27017);
+		mongo = new Mongo("localhost", 27017);
 		db = mongo.getDB("logs");
 
 		Calendar now = Calendar.getInstance();
 		collection = String.valueOf(now.get(Calendar.YEAR)) + String.format("%1$02d", now.get(Calendar.MONTH) + 1);
-	}
-
-	@After
-	public void tearDown() {
-		db.getCollection(collection).remove(new BasicDBObject());
+		db.getCollection(collection).drop();
 	}
 
 	@Test
@@ -71,6 +64,7 @@ public class MongoLog4jAppenderIntegrationTests {
 		log.error("ERROR message");
 
 		DBCursor msgs = db.getCollection(collection).find();
+
 		assertThat(msgs.count(), is(4));
 	}
 

spring-data-mongodb/Spring Data MongoDB.sonargraph

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
-<context version="7.2.2.230">
+<context version="7.1.10.209">
     <scope type="Project" name="spring-data-mongodb">
         <element type="TypeFilterReferenceOverridden" name="Filter">
             <element type="IncludeTypePattern" name="org.springframework.data.mongodb.**"/>
@@ -35,12 +35,6 @@
                     <dependency toName="Project|spring-data-mongodb::Layer|Repositories::Subsystem|API" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Repositories::Subsystem|Implementation" type="AllowedDependency"/>
                 </element>
-                <element type="Subsystem" name="CDI">
-                    <element type="TypeFilter" name="Assignment">
-                        <element type="IncludeTypePattern" name="**.cdi.**"/>
-                    </element>
-                    <stereotype name="Unrestricted"/>
-                </element>
                 <dependency toName="Project|spring-data-mongodb::Layer|Config" type="AllowedDependency"/>
                 <dependency toName="Project|spring-data-mongodb::Layer|Core" type="AllowedDependency"/>
             </element>
@@ -82,11 +76,6 @@
                     </element>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Geospatial" type="AllowedDependency"/>
                 </element>
-                <element type="Subsystem" name="Script">
-                    <element type="TypeFilter" name="Assignment">
-                        <element type="IncludeTypePattern" name="**.script.**"/>
-                    </element>
-                </element>
                 <element type="Subsystem" name="Conversion">
                     <element type="TypeFilter" name="Assignment">
                         <element type="IncludeTypePattern" name="**.core.convert.**"/>
@@ -94,7 +83,6 @@
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Geospatial" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Mapping" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Query" type="AllowedDependency"/>
-                    <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Script" type="AllowedDependency"/>
                 </element>
                 <element type="Subsystem" name="SpEL">
                     <element type="TypeFilter" name="Assignment">
@@ -117,11 +105,6 @@
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Mapping" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Query" type="AllowedDependency"/>
                 </element>
-                <element type="Subsystem" name="MapReduce">
-                    <element type="TypeFilter" name="Assignment">
-                        <element type="IncludeTypePattern" name="**.mapreduce.**"/>
-                    </element>
-                </element>
                 <element type="Subsystem" name="Core">
                     <element type="TypeFilter" name="Assignment">
                         <element type="WeakTypePattern" name="**.core.**"/>
@@ -130,10 +113,8 @@
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Conversion" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Geospatial" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Index" type="AllowedDependency"/>
-                    <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|MapReduce" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Mapping" type="AllowedDependency"/>
                     <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Query" type="AllowedDependency"/>
-                    <dependency toName="Project|spring-data-mongodb::Layer|Core::Subsystem|Script" type="AllowedDependency"/>
                 </element>
                 <element type="Subsystem" name="Util">
                     <element type="TypeFilter" name="Assignment">
@@ -188,32 +169,7 @@
             </element>
             <element type="Subsystem" name="Querydsl">
                 <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="com.querydsl.**"/>
-                </element>
-            </element>
-            <element type="Subsystem" name="Slf4j">
-                <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="org.slf4j.**"/>
-                </element>
-            </element>
-            <element type="Subsystem" name="Jackson">
-                <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="com.fasterxml.jackson.**"/>
-                </element>
-            </element>
-            <element type="Subsystem" name="DOM">
-                <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="org.w3c.dom.**"/>
-                </element>
-            </element>
-            <element type="Subsystem" name="AOP Alliance">
-                <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="org.aopalliance.**"/>
-                </element>
-            </element>
-            <element type="Subsystem" name="Guava">
-                <element type="TypeFilter" name="Assignment">
-                    <element type="IncludeTypePattern" name="com.google.common.**"/>
+                    <element type="IncludeTypePattern" name="com.mysema.query.**"/>
                 </element>
             </element>
         </architecture>

spring-data-mongodb/pom.xml

@@ -11,7 +11,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-mongodb-parent</artifactId>
-		<version>1.9.11.RELEASE</version>
+		<version>1.7.2.RELEASE</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 
@@ -59,14 +59,14 @@
 		</dependency>
 
 		<dependency>
-			<groupId>com.querydsl</groupId>
+			<groupId>com.mysema.querydsl</groupId>
 			<artifactId>querydsl-mongodb</artifactId>
 			<version>${querydsl}</version>
 			<optional>true</optional>
 		</dependency>
 
 		<dependency>
-			<groupId>com.querydsl</groupId>
+			<groupId>com.mysema.querydsl</groupId>
 			<artifactId>querydsl-apt</artifactId>
 			<version>${querydsl}</version>
 			<scope>provided</scope>
@@ -183,7 +183,7 @@
 				<version>${apt}</version>
 				<dependencies>
 					<dependency>
-						<groupId>com.querydsl</groupId>
+						<groupId>com.mysema.querydsl</groupId>
 						<artifactId>querydsl-apt</artifactId>
 						<version>${querydsl}</version>
 					</dependency>

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/BulkOperationException.java

@@ -1,61 +0,0 @@
-/*
- * Copyright 2015 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb;
-
-import java.util.List;
-
-import org.springframework.dao.DataAccessException;
-
-import com.mongodb.BulkWriteError;
-import com.mongodb.BulkWriteException;
-import com.mongodb.BulkWriteResult;
-
-/**
- * Is thrown when errors occur during bulk operations.
- * 
- * @author Tobias Trelle
- * @author Oliver Gierke
- * @since 1.9
- */
-public class BulkOperationException extends DataAccessException {
-
-	private static final long serialVersionUID = 73929601661154421L;
-
-	private final List<BulkWriteError> errors;
-	private final BulkWriteResult result;
-
-	/**
-	 * Creates a new {@link BulkOperationException} with the given message and source {@link BulkWriteException}.
-	 * 
-	 * @param message must not be {@literal null}.
-	 * @param source must not be {@literal null}.
-	 */
-	public BulkOperationException(String message, BulkWriteException source) {
-
-		super(message, source);
-
-		this.errors = source.getWriteErrors();
-		this.result = source.getWriteResult();
-	}
-
-	public List<BulkWriteError> getErrors() {
-		return errors;
-	}
-
-	public BulkWriteResult getResult() {
-		return result;
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/MappingMongoConverterParser.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -361,9 +361,7 @@ public class MappingMongoConverterParser implements BeanDefinitionParser {
 		 * @param filters
 		 */
 		public NegatingFilter(TypeFilter... filters) {
-
-			Assert.notNull(filters, "TypeFilters must not be null");
-
+			Assert.notNull(filters);
 			this.delegates = new HashSet<TypeFilter>(Arrays.asList(filters));
 		}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/MongoAuditingRegistrar.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,24 +15,24 @@
  */
 package org.springframework.data.mongodb.config;
 
+import static org.springframework.beans.factory.config.BeanDefinition.*;
+import static org.springframework.data.mongodb.config.BeanNames.*;
+
 import java.lang.annotation.Annotation;
 
-import org.springframework.beans.factory.FactoryBean;
 import org.springframework.beans.factory.config.BeanDefinition;
-import org.springframework.beans.factory.support.AbstractBeanDefinition;
 import org.springframework.beans.factory.support.BeanDefinitionBuilder;
 import org.springframework.beans.factory.support.BeanDefinitionRegistry;
+import org.springframework.beans.factory.support.RootBeanDefinition;
 import org.springframework.context.annotation.ImportBeanDefinitionRegistrar;
 import org.springframework.core.type.AnnotationMetadata;
 import org.springframework.data.auditing.IsNewAwareAuditingHandler;
 import org.springframework.data.auditing.config.AuditingBeanDefinitionRegistrarSupport;
 import org.springframework.data.auditing.config.AuditingConfiguration;
 import org.springframework.data.config.ParsingUtils;
-import org.springframework.data.mapping.context.MappingContext;
-import org.springframework.data.mongodb.core.convert.MappingMongoConverter;
-import org.springframework.data.mongodb.core.mapping.MongoPersistentEntity;
-import org.springframework.data.mongodb.core.mapping.MongoPersistentProperty;
+import org.springframework.data.mongodb.core.mapping.MongoMappingContext;
 import org.springframework.data.mongodb.core.mapping.event.AuditingEventListener;
+import org.springframework.data.support.IsNewStrategyFactory;
 import org.springframework.util.Assert;
 
 /**
@@ -71,6 +71,7 @@ class MongoAuditingRegistrar extends AuditingBeanDefinitionRegistrarSupport {
 		Assert.notNull(annotationMetadata, "AnnotationMetadata must not be null!");
 		Assert.notNull(registry, "BeanDefinitionRegistry must not be null!");
 
+		defaultDependenciesIfNecessary(registry, annotationMetadata);
 		super.registerBeanDefinitions(annotationMetadata, registry);
 	}
 
@@ -84,11 +85,7 @@ class MongoAuditingRegistrar extends AuditingBeanDefinitionRegistrarSupport {
 		Assert.notNull(configuration, "AuditingConfiguration must not be null!");
 
 		BeanDefinitionBuilder builder = BeanDefinitionBuilder.rootBeanDefinition(IsNewAwareAuditingHandler.class);
-
-		BeanDefinitionBuilder definition = BeanDefinitionBuilder.genericBeanDefinition(MongoMappingContextLookup.class);
-		definition.setAutowireMode(AbstractBeanDefinition.AUTOWIRE_CONSTRUCTOR);
-
-		builder.addConstructorArgValue(definition.getBeanDefinition());
+		builder.addConstructorArgReference(MAPPING_CONTEXT_BEAN_NAME);
 		return configureDefaultAuditHandlerAttributes(configuration, builder);
 	}
 
@@ -105,58 +102,29 @@ class MongoAuditingRegistrar extends AuditingBeanDefinitionRegistrarSupport {
 
 		BeanDefinitionBuilder listenerBeanDefinitionBuilder = BeanDefinitionBuilder
 				.rootBeanDefinition(AuditingEventListener.class);
-		listenerBeanDefinitionBuilder
-				.addConstructorArgValue(ParsingUtils.getObjectFactoryBeanDefinition(getAuditingHandlerBeanName(), registry));
+		listenerBeanDefinitionBuilder.addConstructorArgValue(ParsingUtils.getObjectFactoryBeanDefinition(
+				getAuditingHandlerBeanName(), registry));
 
 		registerInfrastructureBeanWithId(listenerBeanDefinitionBuilder.getBeanDefinition(),
 				AuditingEventListener.class.getName(), registry);
 	}
 
 	/**
-	 * Simple helper to be able to wire the {@link MappingContext} from a {@link MappingMongoConverter} bean available in
-	 * the application context.
-	 *
-	 * @author Oliver Gierke
+	 * Register default bean definitions for a {@link MongoMappingContext} and an {@link IsNewStrategyFactory} in case we
+	 * don't find beans with the assumed names in the registry.
+	 * 
+	 * @param registry the {@link BeanDefinitionRegistry} to use to register the components into.
+	 * @param source the source which the registered components shall be registered with
 	 */
-	static class MongoMappingContextLookup
-			implements FactoryBean<MappingContext<? extends MongoPersistentEntity<?>, MongoPersistentProperty>> {
+	private void defaultDependenciesIfNecessary(BeanDefinitionRegistry registry, Object source) {
 
-		private final MappingMongoConverter converter;
+		if (!registry.containsBeanDefinition(MAPPING_CONTEXT_BEAN_NAME)) {
 
-		/**
-		 * Creates a new {@link MongoMappingContextLookup} for the given {@link MappingMongoConverter}.
-		 * 
-		 * @param converter must not be {@literal null}.
-		 */
-		public MongoMappingContextLookup(MappingMongoConverter converter) {
-			this.converter = converter;
-		}
+			RootBeanDefinition definition = new RootBeanDefinition(MongoMappingContext.class);
+			definition.setRole(ROLE_INFRASTRUCTURE);
+			definition.setSource(source);
 
-		/* 
-		 * (non-Javadoc)
-		 * @see org.springframework.beans.factory.FactoryBean#getObject()
-		 */
-		@Override
-		public MappingContext<? extends MongoPersistentEntity<?>, MongoPersistentProperty> getObject() throws Exception {
-			return converter.getMappingContext();
-		}
-
-		/* 
-		 * (non-Javadoc)
-		 * @see org.springframework.beans.factory.FactoryBean#getObjectType()
-		 */
-		@Override
-		public Class<?> getObjectType() {
-			return MappingContext.class;
-		}
-
-		/* 
-		 * (non-Javadoc)
-		 * @see org.springframework.beans.factory.FactoryBean#isSingleton()
-		 */
-		@Override
-		public boolean isSingleton() {
-			return true;
+			registry.registerBeanDefinition(MAPPING_CONTEXT_BEAN_NAME, definition);
 		}
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/config/MongoDbFactoryParser.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2015 by the original author(s).
+ * Copyright 2011-2014 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,10 +18,6 @@ package org.springframework.data.mongodb.config;
 import static org.springframework.data.config.ParsingUtils.*;
 import static org.springframework.data.mongodb.config.MongoParsingUtils.*;
 
-import java.util.Collections;
-import java.util.HashSet;
-import java.util.Set;
-
 import org.springframework.beans.factory.BeanDefinitionStoreException;
 import org.springframework.beans.factory.config.BeanDefinition;
 import org.springframework.beans.factory.parsing.BeanComponentDefinition;
@@ -38,7 +34,6 @@ import org.springframework.util.StringUtils;
 import org.w3c.dom.Element;
 
 import com.mongodb.Mongo;
-import com.mongodb.MongoClientURI;
 import com.mongodb.MongoURI;
 
 /**
@@ -47,22 +42,9 @@ import com.mongodb.MongoURI;
  * @author Jon Brisbin
  * @author Oliver Gierke
  * @author Thomas Darimont
- * @author Christoph Strobl
- * @author Viktor Khoroshko
  */
 public class MongoDbFactoryParser extends AbstractBeanDefinitionParser {
 
-	private static final Set<String> MONGO_URI_ALLOWED_ADDITIONAL_ATTRIBUTES;
-
-	static {
-
-		Set<String> mongoUriAllowedAdditionalAttributes = new HashSet<String>();
-		mongoUriAllowedAdditionalAttributes.add("id");
-		mongoUriAllowedAdditionalAttributes.add("write-concern");
-
-		MONGO_URI_ALLOWED_ADDITIONAL_ATTRIBUTES = Collections.unmodifiableSet(mongoUriAllowedAdditionalAttributes);
-	}
-
 	/* 
 	 * (non-Javadoc)
 	 * @see org.springframework.beans.factory.xml.AbstractBeanDefinitionParser#resolveId(org.w3c.dom.Element, org.springframework.beans.factory.support.AbstractBeanDefinition, org.springframework.beans.factory.xml.ParserContext)
@@ -82,25 +64,29 @@ public class MongoDbFactoryParser extends AbstractBeanDefinitionParser {
 	@Override
 	protected AbstractBeanDefinition parseInternal(Element element, ParserContext parserContext) {
 
-		// Common setup
-		BeanDefinitionBuilder dbFactoryBuilder = BeanDefinitionBuilder.genericBeanDefinition(SimpleMongoDbFactory.class);
-		setPropertyValue(dbFactoryBuilder, element, "write-concern", "writeConcern");
-
-		BeanDefinition mongoUri = getMongoUri(element, parserContext);
-
-		if (mongoUri != null) {
-
-			dbFactoryBuilder.addConstructorArgValue(mongoUri);
-			return getSourceBeanDefinition(dbFactoryBuilder, parserContext, element);
-		}
+		Object source = parserContext.extractSource(element);
 
 		BeanComponentDefinitionBuilder helper = new BeanComponentDefinitionBuilder(element, parserContext);
 
+		String uri = element.getAttribute("uri");
 		String mongoRef = element.getAttribute("mongo-ref");
 		String dbname = element.getAttribute("dbname");
 
 		BeanDefinition userCredentials = getUserCredentialsBeanDefinition(element, parserContext);
 
+		// Common setup
+		BeanDefinitionBuilder dbFactoryBuilder = BeanDefinitionBuilder.genericBeanDefinition(SimpleMongoDbFactory.class);
+		setPropertyValue(dbFactoryBuilder, element, "write-concern", "writeConcern");
+
+		if (StringUtils.hasText(uri)) {
+			if (StringUtils.hasText(mongoRef) || StringUtils.hasText(dbname) || userCredentials != null) {
+				parserContext.getReaderContext().error("Configure either Mongo URI or details individually!", source);
+			}
+
+			dbFactoryBuilder.addConstructorArgValue(getMongoUri(uri));
+			return getSourceBeanDefinition(dbFactoryBuilder, parserContext, element);
+		}
+
 		// Defaulting
 		if (StringUtils.hasText(mongoRef)) {
 			dbFactoryBuilder.addConstructorArgReference(mongoRef);
@@ -161,42 +147,14 @@ public class MongoDbFactoryParser extends AbstractBeanDefinitionParser {
 	}
 
 	/**
-	 * Creates a {@link BeanDefinition} for a {@link MongoURI} or {@link MongoClientURI} depending on configured
-	 * attributes. <br />
-	 * Errors when configured element contains {@literal uri} or {@literal client-uri} along with other attributes except
-	 * {@literal write-concern} and/or {@literal id}.
+	 * Creates a {@link BeanDefinition} for a {@link MongoURI}.
 	 * 
-	 * @param element must not be {@literal null}.
-	 * @param parserContext
-	 * @return {@literal null} in case no client-/uri defined.
+	 * @param uri
+	 * @return
 	 */
-	private BeanDefinition getMongoUri(Element element, ParserContext parserContext) {
+	private BeanDefinition getMongoUri(String uri) {
 
-		boolean hasClientUri = element.hasAttribute("client-uri");
-
-		if (!hasClientUri && !element.hasAttribute("uri")) {
-			return null;
-		}
-
-		int allowedAttributesCount = 1;
-		for (String attribute : MONGO_URI_ALLOWED_ADDITIONAL_ATTRIBUTES) {
-
-			if (element.hasAttribute(attribute)) {
-				allowedAttributesCount++;
-			}
-		}
-
-		if (element.getAttributes().getLength() > allowedAttributesCount) {
-
-			parserContext.getReaderContext().error(
-					"Configure either " + (hasClientUri ? "Mongo Client URI" : "Mongo URI") + " or details individually!",
-					parserContext.extractSource(element));
-		}
-
-		Class<?> type = hasClientUri ? MongoClientURI.class : MongoURI.class;
-		String uri = hasClientUri ? element.getAttribute("client-uri") : element.getAttribute("uri");
-
-		BeanDefinitionBuilder builder = BeanDefinitionBuilder.genericBeanDefinition(type);
+		BeanDefinitionBuilder builder = BeanDefinitionBuilder.genericBeanDefinition(MongoURI.class);
 		builder.addConstructorArgValue(uri);
 
 		return builder.getBeanDefinition();

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/BulkOperations.java

@@ -1,145 +0,0 @@
-/*
- * Copyright 2015-2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core;
-
-import java.util.List;
-
-import org.springframework.data.mongodb.core.query.Query;
-import org.springframework.data.mongodb.core.query.Update;
-import org.springframework.data.util.Pair;
-
-import com.mongodb.BulkWriteResult;
-
-/**
- * Bulk operations for insert/update/remove actions on a collection. These bulks operation are available since MongoDB
- * 2.6 and make use of low level bulk commands on the protocol level. This interface defines a fluent API to add
- * multiple single operations or list of similar operations in sequence which can then eventually be executed by calling
- * {@link #execute()}.
- * 
- * @author Tobias Trelle
- * @author Oliver Gierke
- * @since 1.9
- */
-public interface BulkOperations {
-
-	/**
-	 * Mode for bulk operation.
-	 **/
-	public enum BulkMode {
-
-		/** Perform bulk operations in sequence. The first error will cancel processing. */
-		ORDERED,
-
-		/** Perform bulk operations in parallel. Processing will continue on errors. */
-		UNORDERED
-	};
-
-	/**
-	 * Add a single insert to the bulk operation.
-	 * 
-	 * @param documents the document to insert, must not be {@literal null}.
-	 * @return the current {@link BulkOperations} instance with the insert added, will never be {@literal null}.
-	 */
-	BulkOperations insert(Object documents);
-
-	/**
-	 * Add a list of inserts to the bulk operation.
-	 * 
-	 * @param documents List of documents to insert, must not be {@literal null}.
-	 * @return the current {@link BulkOperations} instance with the insert added, will never be {@literal null}.
-	 */
-	BulkOperations insert(List<? extends Object> documents);
-
-	/**
-	 * Add a single update to the bulk operation. For the update request, only the first matching document is updated.
-	 * 
-	 * @param query update criteria, must not be {@literal null}.
-	 * @param update {@link Update} operation to perform, must not be {@literal null}.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations updateOne(Query query, Update update);
-
-	/**
-	 * Add a list of updates to the bulk operation. For each update request, only the first matching document is updated.
-	 * 
-	 * @param updates Update operations to perform.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations updateOne(List<Pair<Query, Update>> updates);
-
-	/**
-	 * Add a single update to the bulk operation. For the update request, all matching documents are updated.
-	 * 
-	 * @param query Update criteria.
-	 * @param update Update operation to perform.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations updateMulti(Query query, Update update);
-
-	/**
-	 * Add a list of updates to the bulk operation. For each update request, all matching documents are updated.
-	 * 
-	 * @param updates Update operations to perform.
-	 * @return The bulk operation.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations updateMulti(List<Pair<Query, Update>> updates);
-
-	/**
-	 * Add a single upsert to the bulk operation. An upsert is an update if the set of matching documents is not empty,
-	 * else an insert.
-	 * 
-	 * @param query Update criteria.
-	 * @param update Update operation to perform.
-	 * @return The bulk operation.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations upsert(Query query, Update update);
-
-	/**
-	 * Add a list of upserts to the bulk operation. An upsert is an update if the set of matching documents is not empty,
-	 * else an insert.
-	 * 
-	 * @param updates Updates/insert operations to perform.
-	 * @return The bulk operation.
-	 * @return the current {@link BulkOperations} instance with the update added, will never be {@literal null}.
-	 */
-	BulkOperations upsert(List<Pair<Query, Update>> updates);
-
-	/**
-	 * Add a single remove operation to the bulk operation.
-	 * 
-	 * @param remove the {@link Query} to select the documents to be removed, must not be {@literal null}.
-	 * @return the current {@link BulkOperations} instance with the removal added, will never be {@literal null}.
-	 */
-	BulkOperations remove(Query remove);
-
-	/**
-	 * Add a list of remove operations to the bulk operation.
-	 * 
-	 * @param removes the remove operations to perform, must not be {@literal null}.
-	 * @return the current {@link BulkOperations} instance with the removal added, will never be {@literal null}.
-	 */
-	BulkOperations remove(List<Query> removes);
-
-	/**
-	 * Execute all bulk operations using the default write concern.
-	 * 
-	 * @return Result of the bulk operation providing counters for inserts/updates etc.
-	 * @throws {@link BulkOperationException} if an error occurred during bulk processing.
-	 */
-	BulkWriteResult execute();
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/DefaultBulkOperations.java

@@ -1,337 +0,0 @@
-/*
- * Copyright 2015-2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core;
-
-import java.util.Arrays;
-import java.util.List;
-
-import org.springframework.dao.DataAccessException;
-import org.springframework.dao.support.PersistenceExceptionTranslator;
-import org.springframework.data.mongodb.core.query.Query;
-import org.springframework.data.mongodb.core.query.Update;
-import org.springframework.data.util.Pair;
-import org.springframework.util.Assert;
-
-import com.mongodb.BasicDBObject;
-import com.mongodb.BulkWriteException;
-import com.mongodb.BulkWriteOperation;
-import com.mongodb.BulkWriteRequestBuilder;
-import com.mongodb.BulkWriteResult;
-import com.mongodb.DBCollection;
-import com.mongodb.DBObject;
-import com.mongodb.WriteConcern;
-
-/**
- * Default implementation for {@link BulkOperations}.
- * 
- * @author Tobias Trelle
- * @author Oliver Gierke
- * @author Christoph Strobl
- * @since 1.9
- */
-class DefaultBulkOperations implements BulkOperations {
-
-	private final MongoOperations mongoOperations;
-	private final BulkMode bulkMode;
-	private final String collectionName;
-	private final Class<?> entityType;
-
-	private PersistenceExceptionTranslator exceptionTranslator;
-	private WriteConcernResolver writeConcernResolver;
-	private WriteConcern defaultWriteConcern;
-
-	private BulkWriteOperation bulk;
-
-	/**
-	 * Creates a new {@link DefaultBulkOperations} for the given {@link MongoOperations}, {@link BulkMode}, collection
-	 * name and {@link WriteConcern}.
-	 * 
-	 * @param mongoOperations The underlying {@link MongoOperations}, must not be {@literal null}.
-	 * @param bulkMode must not be {@literal null}.
-	 * @param collectionName Name of the collection to work on, must not be {@literal null} or empty.
-	 * @param entityType the entity type, can be {@literal null}.
-	 */
-	DefaultBulkOperations(MongoOperations mongoOperations, BulkMode bulkMode, String collectionName,
-			Class<?> entityType) {
-
-		Assert.notNull(mongoOperations, "MongoOperations must not be null!");
-		Assert.notNull(bulkMode, "BulkMode must not be null!");
-		Assert.hasText(collectionName, "Collection name must not be null or empty!");
-
-		this.mongoOperations = mongoOperations;
-		this.bulkMode = bulkMode;
-		this.collectionName = collectionName;
-		this.entityType = entityType;
-
-		this.exceptionTranslator = new MongoExceptionTranslator();
-		this.writeConcernResolver = DefaultWriteConcernResolver.INSTANCE;
-
-		this.bulk = initBulkOperation();
-	}
-
-	/**
-	 * Configures the {@link PersistenceExceptionTranslator} to be used. Defaults to {@link MongoExceptionTranslator}.
-	 * 
-	 * @param exceptionTranslator can be {@literal null}.
-	 */
-	public void setExceptionTranslator(PersistenceExceptionTranslator exceptionTranslator) {
-		this.exceptionTranslator = exceptionTranslator == null ? new MongoExceptionTranslator() : exceptionTranslator;
-	}
-
-	/**
-	 * Configures the {@link WriteConcernResolver} to be used. Defaults to {@link DefaultWriteConcernResolver}.
-	 * 
-	 * @param writeConcernResolver can be {@literal null}.
-	 */
-	public void setWriteConcernResolver(WriteConcernResolver writeConcernResolver) {
-		this.writeConcernResolver = writeConcernResolver == null ? DefaultWriteConcernResolver.INSTANCE
-				: writeConcernResolver;
-	}
-
-	/**
-	 * Configures the default {@link WriteConcern} to be used. Defaults to {@literal null}.
-	 * 
-	 * @param defaultWriteConcern can be {@literal null}.
-	 */
-	public void setDefaultWriteConcern(WriteConcern defaultWriteConcern) {
-		this.defaultWriteConcern = defaultWriteConcern;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#insert(java.lang.Object)
-	 */
-	@Override
-	public BulkOperations insert(Object document) {
-
-		Assert.notNull(document, "Document must not be null!");
-
-		if (document instanceof DBObject) {
-
-			bulk.insert((DBObject) document);
-			return this;
-		}
-
-		DBObject sink = new BasicDBObject();
-		mongoOperations.getConverter().write(document, sink);
-		bulk.insert(sink);
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#insert(java.util.List)
-	 */
-	@Override
-	public BulkOperations insert(List<? extends Object> documents) {
-
-		Assert.notNull(documents, "Documents must not be null!");
-
-		for (Object document : documents) {
-			insert(document);
-		}
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#updateOne(org.springframework.data.mongodb.core.query.Query, org.springframework.data.mongodb.core.query.Update)
-	 */
-	@Override
-	@SuppressWarnings("unchecked")
-	public BulkOperations updateOne(Query query, Update update) {
-
-		Assert.notNull(query, "Query must not be null!");
-		Assert.notNull(update, "Update must not be null!");
-
-		return updateOne(Arrays.asList(Pair.of(query, update)));
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#updateOne(java.util.List)
-	 */
-	@Override
-	public BulkOperations updateOne(List<Pair<Query, Update>> updates) {
-
-		Assert.notNull(updates, "Updates must not be null!");
-
-		for (Pair<Query, Update> update : updates) {
-			update(update.getFirst(), update.getSecond(), false, false);
-		}
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#updateMulti(org.springframework.data.mongodb.core.query.Query, org.springframework.data.mongodb.core.query.Update)
-	 */
-	@Override
-	@SuppressWarnings("unchecked")
-	public BulkOperations updateMulti(Query query, Update update) {
-
-		Assert.notNull(query, "Query must not be null!");
-		Assert.notNull(update, "Update must not be null!");
-
-		return updateMulti(Arrays.asList(Pair.of(query, update)));
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#updateMulti(java.util.List)
-	 */
-	@Override
-	public BulkOperations updateMulti(List<Pair<Query, Update>> updates) {
-
-		Assert.notNull(updates, "Updates must not be null!");
-
-		for (Pair<Query, Update> update : updates) {
-			update(update.getFirst(), update.getSecond(), false, true);
-		}
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#upsert(org.springframework.data.mongodb.core.query.Query, org.springframework.data.mongodb.core.query.Update)
-	 */
-	@Override
-	public BulkOperations upsert(Query query, Update update) {
-		return update(query, update, true, true);
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#upsert(java.util.List)
-	 */
-	@Override
-	public BulkOperations upsert(List<Pair<Query, Update>> updates) {
-
-		for (Pair<Query, Update> update : updates) {
-			upsert(update.getFirst(), update.getSecond());
-		}
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#remove(org.springframework.data.mongodb.core.query.Query)
-	 */
-	@Override
-	public BulkOperations remove(Query query) {
-
-		Assert.notNull(query, "Query must not be null!");
-
-		bulk.find(query.getQueryObject()).remove();
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#remove(java.util.List)
-	 */
-	@Override
-	public BulkOperations remove(List<Query> removes) {
-
-		Assert.notNull(removes, "Removals must not be null!");
-
-		for (Query query : removes) {
-			remove(query);
-		}
-
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.BulkOperations#executeBulk()
-	 */
-	@Override
-	public BulkWriteResult execute() {
-
-		MongoAction action = new MongoAction(defaultWriteConcern, MongoActionOperation.BULK, collectionName, entityType,
-				null, null);
-		WriteConcern writeConcern = writeConcernResolver.resolve(action);
-
-		try {
-
-			return writeConcern == null ? bulk.execute() : bulk.execute(writeConcern);
-
-		} catch (BulkWriteException o_O) {
-
-			DataAccessException toThrow = exceptionTranslator.translateExceptionIfPossible(o_O);
-			throw toThrow == null ? o_O : toThrow;
-
-		} finally {
-			this.bulk = initBulkOperation();
-		}
-	}
-
-	/**
-	 * Performs update and upsert bulk operations.
-	 * 
-	 * @param query the {@link Query} to determine documents to update.
-	 * @param update the {@link Update} to perform, must not be {@literal null}.
-	 * @param upsert whether to upsert.
-	 * @param multi whether to issue a multi-update.
-	 * @return the {@link BulkOperations} with the update registered.
-	 */
-	private BulkOperations update(Query query, Update update, boolean upsert, boolean multi) {
-
-		Assert.notNull(query, "Query must not be null!");
-		Assert.notNull(update, "Update must not be null!");
-
-		BulkWriteRequestBuilder builder = bulk.find(query.getQueryObject());
-
-		if (upsert) {
-
-			if (multi) {
-				builder.upsert().update(update.getUpdateObject());
-			} else {
-				builder.upsert().updateOne(update.getUpdateObject());
-			}
-
-		} else {
-
-			if (multi) {
-				builder.update(update.getUpdateObject());
-			} else {
-				builder.updateOne(update.getUpdateObject());
-			}
-		}
-
-		return this;
-	}
-
-	private final BulkWriteOperation initBulkOperation() {
-
-		DBCollection collection = mongoOperations.getCollection(collectionName);
-
-		switch (bulkMode) {
-			case ORDERED:
-				return collection.initializeOrderedBulkOperation();
-			case UNORDERED:
-				return collection.initializeUnorderedBulkOperation();
-		}
-
-		throw new IllegalStateException("BulkMode was null!");
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/DefaultIndexOperations.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,15 +15,17 @@
  */
 package org.springframework.data.mongodb.core;
 
+import static org.springframework.data.domain.Sort.Direction.*;
+
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.List;
 
 import org.springframework.dao.DataAccessException;
-import org.springframework.data.mongodb.core.convert.QueryMapper;
 import org.springframework.data.mongodb.core.index.IndexDefinition;
+import org.springframework.data.mongodb.core.index.IndexField;
 import org.springframework.data.mongodb.core.index.IndexInfo;
-import org.springframework.data.mongodb.core.mapping.MongoPersistentEntity;
 import org.springframework.util.Assert;
 
 import com.mongodb.DBCollection;
@@ -40,12 +42,12 @@ import com.mongodb.MongoException;
  */
 public class DefaultIndexOperations implements IndexOperations {
 
-	private static final String PARTIAL_FILTER_EXPRESSION_KEY = "partialFilterExpression";
+	private static final Double ONE = Double.valueOf(1);
+	private static final Double MINUS_ONE = Double.valueOf(-1);
+	private static final Collection<String> TWO_D_IDENTIFIERS = Arrays.asList("2d", "2dsphere");
 
 	private final MongoOperations mongoOperations;
 	private final String collectionName;
-	private final QueryMapper mapper;
-	private final Class<?> type;
 
 	/**
 	 * Creates a new {@link DefaultIndexOperations}.
@@ -54,26 +56,12 @@ public class DefaultIndexOperations implements IndexOperations {
 	 * @param collectionName must not be {@literal null}.
 	 */
 	public DefaultIndexOperations(MongoOperations mongoOperations, String collectionName) {
-		this(mongoOperations, collectionName, null);
-	}
-
-	/**
-	 * Creates a new {@link DefaultIndexOperations}.
-	 *
-	 * @param mongoOperations must not be {@literal null}.
-	 * @param collectionName must not be {@literal null}.
-	 * @param type Type used for mapping potential partial index filter expression. Can be {@literal null}.
-	 * @since 1.10
-	 */
-	public DefaultIndexOperations(MongoOperations mongoOperations, String collectionName, Class<?> type) {
 
 		Assert.notNull(mongoOperations, "MongoOperations must not be null!");
 		Assert.notNull(collectionName, "Collection name can not be null!");
 
 		this.mongoOperations = mongoOperations;
 		this.collectionName = collectionName;
-		this.mapper = new QueryMapper(mongoOperations.getConverter());
-		this.type = type;
 	}
 
 	/*
@@ -81,20 +69,9 @@ public class DefaultIndexOperations implements IndexOperations {
 	 * @see org.springframework.data.mongodb.core.IndexOperations#ensureIndex(org.springframework.data.mongodb.core.index.IndexDefinition)
 	 */
 	public void ensureIndex(final IndexDefinition indexDefinition) {
-
 		mongoOperations.execute(collectionName, new CollectionCallback<Object>() {
 			public Object doInCollection(DBCollection collection) throws MongoException, DataAccessException {
 				DBObject indexOptions = indexDefinition.getIndexOptions();
-
-				if (indexOptions != null && indexOptions.containsField(PARTIAL_FILTER_EXPRESSION_KEY)) {
-
-					Assert.isInstanceOf(DBObject.class, indexOptions.get(PARTIAL_FILTER_EXPRESSION_KEY));
-
-					indexOptions.put(PARTIAL_FILTER_EXPRESSION_KEY,
-							mapper.getMappedObject((DBObject) indexOptions.get(PARTIAL_FILTER_EXPRESSION_KEY),
-									lookupPersistentEntity(type, collectionName)));
-				}
-
 				if (indexOptions != null) {
 					collection.createIndex(indexDefinition.getIndexKeys(), indexOptions);
 				} else {
@@ -102,24 +79,6 @@ public class DefaultIndexOperations implements IndexOperations {
 				}
 				return null;
 			}
-
-			private MongoPersistentEntity<?> lookupPersistentEntity(Class<?> entityType, String collection) {
-
-				if (entityType != null) {
-					return mongoOperations.getConverter().getMappingContext().getPersistentEntity(entityType);
-				}
-
-				Collection<? extends MongoPersistentEntity<?>> entities = mongoOperations.getConverter().getMappingContext()
-						.getPersistentEntities();
-
-				for (MongoPersistentEntity<?> entity : entities) {
-					if (entity.getCollection().equals(collection)) {
-						return entity;
-					}
-				}
-
-				return null;
-			}
 		});
 	}
 
@@ -167,9 +126,7 @@ public class DefaultIndexOperations implements IndexOperations {
 	public List<IndexInfo> getIndexInfo() {
 
 		return mongoOperations.execute(collectionName, new CollectionCallback<List<IndexInfo>>() {
-
 			public List<IndexInfo> doInCollection(DBCollection collection) throws MongoException, DataAccessException {
-
 				List<DBObject> dbObjectList = collection.getIndexInfo();
 				return getIndexData(dbObjectList);
 			}
@@ -179,7 +136,44 @@ public class DefaultIndexOperations implements IndexOperations {
 				List<IndexInfo> indexInfoList = new ArrayList<IndexInfo>();
 
 				for (DBObject ix : dbObjectList) {
-					indexInfoList.add(IndexInfo.indexInfoOf(ix));
+
+					DBObject keyDbObject = (DBObject) ix.get("key");
+					int numberOfElements = keyDbObject.keySet().size();
+
+					List<IndexField> indexFields = new ArrayList<IndexField>(numberOfElements);
+
+					for (String key : keyDbObject.keySet()) {
+
+						Object value = keyDbObject.get(key);
+
+						if (TWO_D_IDENTIFIERS.contains(value)) {
+							indexFields.add(IndexField.geo(key));
+						} else if ("text".equals(value)) {
+
+							DBObject weights = (DBObject) ix.get("weights");
+							for (String fieldName : weights.keySet()) {
+								indexFields.add(IndexField.text(fieldName, Float.valueOf(weights.get(fieldName).toString())));
+							}
+
+						} else {
+
+							Double keyValue = new Double(value.toString());
+
+							if (ONE.equals(keyValue)) {
+								indexFields.add(IndexField.create(key, ASC));
+							} else if (MINUS_ONE.equals(keyValue)) {
+								indexFields.add(IndexField.create(key, DESC));
+							}
+						}
+					}
+
+					String name = ix.get("name").toString();
+
+					boolean unique = ix.containsField("unique") ? (Boolean) ix.get("unique") : false;
+					boolean dropDuplicates = ix.containsField("dropDups") ? (Boolean) ix.get("dropDups") : false;
+					boolean sparse = ix.containsField("sparse") ? (Boolean) ix.get("sparse") : false;
+					String language = ix.containsField("default_language") ? (String) ix.get("default_language") : "";
+					indexInfoList.add(new IndexInfo(indexFields, name, unique, dropDuplicates, sparse, language));
 				}
 
 				return indexInfoList;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/DefaultScriptOperations.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2016 the original author or authors.
+ * Copyright 2014-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -98,7 +98,7 @@ class DefaultScriptOperations implements ScriptOperations {
 
 			@Override
 			public Object doInDB(DB db) throws MongoException, DataAccessException {
-				return db.eval(script.getCode(), convertScriptArgs(false, args));
+				return db.eval(script.getCode(), convertScriptArgs(args));
 			}
 		});
 	}
@@ -155,7 +155,7 @@ class DefaultScriptOperations implements ScriptOperations {
 		return scriptNames;
 	}
 
-	private Object[] convertScriptArgs(boolean quote, Object... args) {
+	private Object[] convertScriptArgs(Object... args) {
 
 		if (ObjectUtils.isEmpty(args)) {
 			return args;
@@ -164,15 +164,15 @@ class DefaultScriptOperations implements ScriptOperations {
 		List<Object> convertedValues = new ArrayList<Object>(args.length);
 
 		for (Object arg : args) {
-			convertedValues.add(arg instanceof String && quote ? String.format("'%s'", arg)
-					: this.mongoOperations.getConverter().convertToMongoType(arg));
+			convertedValues.add(arg instanceof String ? String.format("'%s'", arg) : this.mongoOperations.getConverter()
+					.convertToMongoType(arg));
 		}
 
 		return convertedValues.toArray();
 	}
 
 	private String convertAndJoinScriptArgs(Object... args) {
-		return ObjectUtils.isEmpty(args) ? "" : StringUtils.arrayToCommaDelimitedString(convertScriptArgs(true, args));
+		return ObjectUtils.isEmpty(args) ? "" : StringUtils.arrayToCommaDelimitedString(convertScriptArgs(args));
 	}
 
 	/**

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/DefaultWriteConcernResolver.java

@@ -1,32 +0,0 @@
-/*
- * Copyright 2015 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core;
-
-import com.mongodb.WriteConcern;
-
-/**
- * Default {@link WriteConcernResolver} resolving the {@link WriteConcern} from the given {@link MongoAction}.
- *
- * @author Oliver Gierke
- */
-enum DefaultWriteConcernResolver implements WriteConcernResolver {
-
-	INSTANCE;
-
-	public WriteConcern resolve(MongoAction action) {
-		return action.getDefaultWriteConcern();
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/GeoCommandStatistics.java

@@ -1,72 +0,0 @@
-/*
- * Copyright 2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core;
-
-import org.springframework.util.Assert;
-
-import com.mongodb.BasicDBObject;
-import com.mongodb.DBObject;
-
-/**
- * Value object to mitigate different representations of geo command execution results in MongoDB.
- * 
- * @author Oliver Gierke
- * @soundtrack Fruitcake - Jeff Coffin (The Inside of the Outside)
- */
-class GeoCommandStatistics {
-
-	private static final GeoCommandStatistics NONE = new GeoCommandStatistics(new BasicDBObject());
-
-	private final DBObject source;
-
-	/**
-	 * Creates a new {@link GeoCommandStatistics} instance with the given source document.
-	 * 
-	 * @param source must not be {@literal null}.
-	 */
-	private GeoCommandStatistics(DBObject source) {
-
-		Assert.notNull(source, "Source document must not be null!");
-		this.source = source;
-	}
-
-	/**
-	 * Creates a new {@link GeoCommandStatistics} from the given command result extracting the statistics.
-	 * 
-	 * @param commandResult must not be {@literal null}.
-	 * @return
-	 */
-	public static GeoCommandStatistics from(DBObject commandResult) {
-
-		Assert.notNull(commandResult, "Command result must not be null!");
-
-		Object stats = commandResult.get("stats");
-		return stats == null ? NONE : new GeoCommandStatistics((DBObject) stats);
-	}
-
-	/**
-	 * Returns the average distance reported by the command result. Mitigating a removal of the field in case the command
-	 * didn't return any result introduced in MongoDB 3.2 RC1.
-	 * 
-	 * @return
-	 * @see https://jira.mongodb.org/browse/SERVER-21024
-	 */
-	public double getAverageDistance() {
-
-		Object averageDistance = source.get("avgDistance");
-		return averageDistance == null ? Double.NaN : (Double) averageDistance;
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoActionOperation.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2015 the original author or authors.
+ * Copyright 2011-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -25,5 +25,5 @@ package org.springframework.data.mongodb.core;
  */
 public enum MongoActionOperation {
 
-	REMOVE, UPDATE, INSERT, INSERT_LIST, SAVE, BULK;
+	REMOVE, UPDATE, INSERT, INSERT_LIST, SAVE
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoAdmin.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,8 +27,7 @@ import com.mongodb.Mongo;
  * Mongo server administration exposed via JMX annotations
  * 
  * @author Mark Pollack
- * @author Thomas Darimont
- * @author Mark Paluch
+ * @author Thomas Darimont
  */
 @ManagedResource(description = "Mongo Admin Operations")
 public class MongoAdmin implements MongoAdminOperations {
@@ -36,11 +35,10 @@ public class MongoAdmin implements MongoAdminOperations {
 	private final Mongo mongo;
 	private String username;
 	private String password;
-	private String authenticationDatabaseName;
+	private String authenticationDatabaseName;
 
 	public MongoAdmin(Mongo mongo) {
-
-		Assert.notNull(mongo, "Mongo must not be null!");
+		Assert.notNull(mongo);
 		this.mongo = mongo;
 	}
 
@@ -86,16 +84,16 @@ public class MongoAdmin implements MongoAdminOperations {
 		this.password = password;
 	}
 
-	/**
-	 * Sets the authenticationDatabaseName to use to authenticate with the Mongo database.
-	 * 
-	 * @param authenticationDatabaseName The authenticationDatabaseName to use.
-	 */
-	public void setAuthenticationDatabaseName(String authenticationDatabaseName) {
-		this.authenticationDatabaseName = authenticationDatabaseName;
-	}
-
+	/**
+	 * Sets the authenticationDatabaseName to use to authenticate with the Mongo database.
+	 * 
+	 * @param authenticationDatabaseName The authenticationDatabaseName to use.
+	 */
+	public void setAuthenticationDatabaseName(String authenticationDatabaseName) {
+		this.authenticationDatabaseName = authenticationDatabaseName;
+	}
+
 	DB getDB(String databaseName) {
-		return MongoDbUtils.getDB(mongo, databaseName, new UserCredentials(username, password), authenticationDatabaseName);
+		return MongoDbUtils.getDB(mongo, databaseName, new UserCredentials(username, password), authenticationDatabaseName);
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoExceptionTranslator.java

@@ -25,14 +25,10 @@ import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.dao.DuplicateKeyException;
 import org.springframework.dao.InvalidDataAccessApiUsageException;
 import org.springframework.dao.InvalidDataAccessResourceUsageException;
-import org.springframework.dao.PermissionDeniedDataAccessException;
 import org.springframework.dao.support.PersistenceExceptionTranslator;
-import org.springframework.data.mongodb.BulkOperationException;
 import org.springframework.data.mongodb.UncategorizedMongoDbException;
-import org.springframework.data.mongodb.util.MongoDbErrorCodes;
 import org.springframework.util.ClassUtils;
 
-import com.mongodb.BulkWriteException;
 import com.mongodb.MongoException;
 
 /**
@@ -46,12 +42,12 @@ import com.mongodb.MongoException;
  */
 public class MongoExceptionTranslator implements PersistenceExceptionTranslator {
 
-	private static final Set<String> DULICATE_KEY_EXCEPTIONS = new HashSet<String>(
-			Arrays.asList("MongoException.DuplicateKey", "DuplicateKeyException"));
+	private static final Set<String> DULICATE_KEY_EXCEPTIONS = new HashSet<String>(Arrays.asList(
+			"MongoException.DuplicateKey", "DuplicateKeyException"));
 
-	private static final Set<String> RESOURCE_FAILURE_EXCEPTIONS = new HashSet<String>(
-			Arrays.asList("MongoException.Network", "MongoSocketException", "MongoException.CursorNotFound",
-					"MongoCursorNotFoundException", "MongoServerSelectionException", "MongoTimeoutException"));
+	private static final Set<String> RESOURCE_FAILURE_EXCEPTIONS = new HashSet<String>(Arrays.asList(
+			"MongoException.Network", "MongoSocketException", "MongoException.CursorNotFound",
+			"MongoCursorNotFoundException", "MongoServerSelectionException", "MongoTimeoutException"));
 
 	private static final Set<String> RESOURCE_USAGE_EXCEPTIONS = new HashSet<String>(
 			Arrays.asList("MongoInternalException"));
@@ -85,24 +81,17 @@ public class MongoExceptionTranslator implements PersistenceExceptionTranslator
 			return new DataIntegrityViolationException(ex.getMessage(), ex);
 		}
 
-		if (ex instanceof BulkWriteException) {
-			return new BulkOperationException(ex.getMessage(), (BulkWriteException) ex);
-		}
-
 		// All other MongoExceptions
 		if (ex instanceof MongoException) {
 
 			int code = ((MongoException) ex).getCode();
 
-			if (MongoDbErrorCodes.isDuplicateKeyCode(code)) {
+			if (code == 11000 || code == 11001) {
 				throw new DuplicateKeyException(ex.getMessage(), ex);
-			} else if (MongoDbErrorCodes.isDataAccessResourceFailureCode(code)) {
+			} else if (code == 12000 || code == 13440) {
 				throw new DataAccessResourceFailureException(ex.getMessage(), ex);
-			} else if (MongoDbErrorCodes.isInvalidDataAccessApiUsageCode(code) || code == 10003 || code == 12001
-					|| code == 12010 || code == 12011 || code == 12012) {
+			} else if (code == 10003 || code == 12001 || code == 12010 || code == 12011 || code == 12012) {
 				throw new InvalidDataAccessApiUsageException(ex.getMessage(), ex);
-			} else if (MongoDbErrorCodes.isPermissionDeniedCode(code)) {
-				throw new PermissionDeniedDataAccessException(ex.getMessage(), ex);
 			}
 			return new UncategorizedMongoDbException(ex.getMessage(), ex);
 		}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/MongoOperations.java

@@ -20,7 +20,6 @@ import java.util.List;
 import java.util.Set;
 
 import org.springframework.data.geo.GeoResults;
-import org.springframework.data.mongodb.core.BulkOperations.BulkMode;
 import org.springframework.data.mongodb.core.aggregation.Aggregation;
 import org.springframework.data.mongodb.core.aggregation.AggregationResults;
 import org.springframework.data.mongodb.core.aggregation.TypedAggregation;
@@ -191,7 +190,7 @@ public interface MongoOperations {
 	<T> DBCollection createCollection(Class<T> entityClass);
 
 	/**
-	 * Create a collection with a name based on the provided entity class using the options.
+	 * Create a collect with a name based on the provided entity class using the options.
 	 * 
 	 * @param entityClass class that determines the collection to create
 	 * @param collectionOptions options to use when creating the collection.
@@ -208,7 +207,7 @@ public interface MongoOperations {
 	DBCollection createCollection(String collectionName);
 
 	/**
-	 * Create a collection with the provided name and options.
+	 * Create a collect with the provided name and options.
 	 * 
 	 * @param collectionName name of the collection
 	 * @param collectionOptions options to use when creating the collection.
@@ -293,34 +292,6 @@ public interface MongoOperations {
 	 */
 	ScriptOperations scriptOps();
 
-	/**
-	 * Returns a new {@link BulkOperations} for the given collection.
-	 * 
-	 * @param mode the {@link BulkMode} to use for bulk operations, must not be {@literal null}.
-	 * @param collectionName the name of the collection to work on, must not be {@literal null} or empty.
-	 * @return {@link BulkOperations} on the named collection
-	 */
-	BulkOperations bulkOps(BulkMode mode, String collectionName);
-
-	/**
-	 * Returns a new {@link BulkOperations} for the given entity type.
-	 * 
-	 * @param mode the {@link BulkMode} to use for bulk operations, must not be {@literal null}.
-	 * @param entityType the name of the entity class, must not be {@literal null}.
-	 * @return {@link BulkOperations} on the named collection associated of the given entity class.
-	 */
-	BulkOperations bulkOps(BulkMode mode, Class<?> entityType);
-
-	/**
-	 * Returns a new {@link BulkOperations} for the given entity type and collection name.
-	 * 
-	 * @param mode the {@link BulkMode} to use for bulk operations, must not be {@literal null}.
-	 * @param entityClass the name of the entity class, must not be {@literal null}.
-	 * @param collectionName the name of the collection to work on, must not be {@literal null} or empty.
-	 * @return {@link BulkOperations} on the named collection associated with the given entity class.
-	 */
-	BulkOperations bulkOps(BulkMode mode, Class<?> entityType, String collectionName);
-
 	/**
 	 * Query for a list of objects of type T from the collection used by the entity class.
 	 * <p/>
@@ -629,8 +600,8 @@ public interface MongoOperations {
 	<T> T findById(Object id, Class<T> entityClass, String collectionName);
 
 	/**
-	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify
-	 * <a/> to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query}.
+	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify<a/>
+	 * to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query}.
 	 * 
 	 * @param query the {@link Query} class that specifies the {@link Criteria} used to find a record and also an optional
 	 *          fields specification.
@@ -641,8 +612,8 @@ public interface MongoOperations {
 	<T> T findAndModify(Query query, Update update, Class<T> entityClass);
 
 	/**
-	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify
-	 * <a/> to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query}.
+	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify<a/>
+	 * to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query}.
 	 * 
 	 * @param query the {@link Query} class that specifies the {@link Criteria} used to find a record and also an optional
 	 *          fields specification.
@@ -654,8 +625,8 @@ public interface MongoOperations {
 	<T> T findAndModify(Query query, Update update, Class<T> entityClass, String collectionName);
 
 	/**
-	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify
-	 * <a/> to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query} taking
+	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify<a/>
+	 * to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query} taking
 	 * {@link FindAndModifyOptions} into account.
 	 * 
 	 * @param query the {@link Query} class that specifies the {@link Criteria} used to find a record and also an optional
@@ -668,8 +639,8 @@ public interface MongoOperations {
 	<T> T findAndModify(Query query, Update update, FindAndModifyOptions options, Class<T> entityClass);
 
 	/**
-	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify
-	 * <a/> to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query} taking
+	 * Triggers <a href="http://docs.mongodb.org/manual/reference/method/db.collection.findAndModify/">findAndModify<a/>
+	 * to apply provided {@link Update} on documents matching {@link Criteria} of given {@link Query} taking
 	 * {@link FindAndModifyOptions} into account.
 	 * 
 	 * @param query the {@link Query} class that specifies the {@link Criteria} used to find a record and also an optional
@@ -757,9 +728,9 @@ public interface MongoOperations {
 	 * <p/>
 	 * If you object has an "Id' property, it will be set with the generated Id from MongoDB. If your Id property is a
 	 * String then MongoDB ObjectId will be used to populate that string. Otherwise, the conversion from ObjectId to your
-	 * property type will be handled by Spring's BeanWrapper class that leverages Type Conversion API. See
-	 * <a href="http://docs.spring.io/spring/docs/current/spring-framework-reference/html/validation.html#core-convert" >
-	 * Spring's Type Conversion"</a> for more details.
+	 * property type will be handled by Spring's BeanWrapper class that leverages Type Conversion API. See <a
+	 * href="http://docs.spring.io/spring/docs/current/spring-framework-reference/html/validation.html#core-convert"
+	 * >Spring's Type Conversion"</a> for more details.
 	 * <p/>
 	 * <p/>
 	 * Insert is used to initially store the object into the database. To update an existing object use the save method.
@@ -814,9 +785,9 @@ public interface MongoOperations {
 	 * <p/>
 	 * If you object has an "Id' property, it will be set with the generated Id from MongoDB. If your Id property is a
 	 * String then MongoDB ObjectId will be used to populate that string. Otherwise, the conversion from ObjectId to your
-	 * property type will be handled by Spring's BeanWrapper class that leverages Type Conversion API. See
-	 * <a href="http://docs.spring.io/spring/docs/current/spring-framework-reference/html/validation.html#core-convert" >
-	 * Spring's Type Conversion"</a> for more details.
+	 * property type will be handled by Spring's BeanWrapper class that leverages Type Conversion API. See <a
+	 * href="http://docs.spring.io/spring/docs/current/spring-framework-reference/html/validation.html#core-convert"
+	 * >Spring's Type Conversion"</a> for more details.
 	 * 
 	 * @param objectToSave the object to store in the collection
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/Aggregation.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,7 +26,6 @@ import org.springframework.data.domain.Sort.Direction;
 import org.springframework.data.mongodb.core.aggregation.ExposedFields.ExposedField;
 import org.springframework.data.mongodb.core.aggregation.ExposedFields.FieldReference;
 import org.springframework.data.mongodb.core.aggregation.Fields.AggregationField;
-import org.springframework.data.mongodb.core.aggregation.FieldsExposingAggregationOperation.InheritsFieldsAggregationOperation;
 import org.springframework.data.mongodb.core.query.Criteria;
 import org.springframework.data.mongodb.core.query.NearQuery;
 import org.springframework.data.mongodb.core.query.SerializationUtils;
@@ -38,12 +37,10 @@ import com.mongodb.DBObject;
 /**
  * An {@code Aggregation} is a representation of a list of aggregation steps to be performed by the MongoDB Aggregation
  * Framework.
- *
+ * 
  * @author Tobias Trelle
  * @author Thomas Darimont
  * @author Oliver Gierke
- * @author Mark Paluch
- * @author Alessio Fachechi
  * @since 1.3
  */
 public class Aggregation {
@@ -68,7 +65,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Aggregation} from the given {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param operations must not be {@literal null} or empty.
 	 */
 	public static Aggregation newAggregation(List<? extends AggregationOperation> operations) {
@@ -77,7 +74,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Aggregation} from the given {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param operations must not be {@literal null} or empty.
 	 */
 	public static Aggregation newAggregation(AggregationOperation... operations) {
@@ -87,7 +84,7 @@ public class Aggregation {
 	/**
 	 * Returns a copy of this {@link Aggregation} with the given {@link AggregationOptions} set. Note that options are
 	 * supported in MongoDB version 2.6+.
-	 *
+	 * 
 	 * @param options must not be {@literal null}.
 	 * @return
 	 * @since 1.6
@@ -100,7 +97,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link TypedAggregation} for the given type and {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param type must not be {@literal null}.
 	 * @param operations must not be {@literal null} or empty.
 	 */
@@ -110,7 +107,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link TypedAggregation} for the given type and {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param type must not be {@literal null}.
 	 * @param operations must not be {@literal null} or empty.
 	 */
@@ -120,7 +117,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Aggregation} from the given {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param aggregationOperations must not be {@literal null} or empty.
 	 */
 	protected Aggregation(AggregationOperation... aggregationOperations) {
@@ -140,7 +137,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Aggregation} from the given {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param aggregationOperations must not be {@literal null} or empty.
 	 */
 	protected Aggregation(List<AggregationOperation> aggregationOperations) {
@@ -149,14 +146,14 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Aggregation} from the given {@link AggregationOperation}s.
-	 *
+	 * 
 	 * @param aggregationOperations must not be {@literal null} or empty.
 	 * @param options must not be {@literal null} or empty.
 	 */
 	protected Aggregation(List<AggregationOperation> aggregationOperations, AggregationOptions options) {
 
 		Assert.notNull(aggregationOperations, "AggregationOperations must not be null!");
-		Assert.isTrue(!aggregationOperations.isEmpty(), "At least one AggregationOperation has to be provided");
+		Assert.isTrue(aggregationOperations.size() > 0, "At least one AggregationOperation has to be provided");
 		Assert.notNull(options, "AggregationOptions must not be null!");
 
 		this.operations = aggregationOperations;
@@ -165,7 +162,7 @@ public class Aggregation {
 
 	/**
 	 * A pointer to the previous {@link AggregationOperation}.
-	 *
+	 * 
 	 * @return
 	 */
 	public static String previousOperation() {
@@ -174,7 +171,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link ProjectionOperation} including the given fields.
-	 *
+	 * 
 	 * @param fields must not be {@literal null}.
 	 * @return
 	 */
@@ -184,7 +181,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link ProjectionOperation} includeing the given {@link Fields}.
-	 *
+	 * 
 	 * @param fields must not be {@literal null}.
 	 * @return
 	 */
@@ -194,7 +191,7 @@ public class Aggregation {
 
 	/**
 	 * Factory method to create a new {@link UnwindOperation} for the field with the given name.
-	 *
+	 * 
 	 * @param fieldName must not be {@literal null} or empty.
 	 * @return
 	 */
@@ -204,7 +201,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link GroupOperation} for the given fields.
-	 *
+	 * 
 	 * @param fields must not be {@literal null}.
 	 * @return
 	 */
@@ -214,7 +211,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link GroupOperation} for the given {@link Fields}.
-	 *
+	 * 
 	 * @param fields must not be {@literal null}.
 	 * @return
 	 */
@@ -224,7 +221,7 @@ public class Aggregation {
 
 	/**
 	 * Factory method to create a new {@link SortOperation} for the given {@link Sort}.
-	 *
+	 * 
 	 * @param sort must not be {@literal null}.
 	 * @return
 	 */
@@ -234,7 +231,7 @@ public class Aggregation {
 
 	/**
 	 * Factory method to create a new {@link SortOperation} for the given sort {@link Direction} and {@code fields}.
-	 *
+	 * 
 	 * @param direction must not be {@literal null}.
 	 * @param fields must not be {@literal null}.
 	 * @return
@@ -245,28 +242,17 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link SkipOperation} skipping the given number of elements.
-	 *
+	 * 
 	 * @param elementsToSkip must not be less than zero.
 	 * @return
-	 * @deprecated prepare to get this one removed in favor of {@link #skip(long)}.
 	 */
 	public static SkipOperation skip(int elementsToSkip) {
 		return new SkipOperation(elementsToSkip);
 	}
 
-	/**
-	 * Creates a new {@link SkipOperation} skipping the given number of elements.
-	 *
-	 * @param elementsToSkip must not be less than zero.
-	 * @return
-	 */
-	public static SkipOperation skip(long elementsToSkip) {
-		return new SkipOperation(elementsToSkip);
-	}
-
 	/**
 	 * Creates a new {@link LimitOperation} limiting the result to the given number of elements.
-	 *
+	 * 
 	 * @param maxElements must not be less than zero.
 	 * @return
 	 */
@@ -276,7 +262,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link MatchOperation} using the given {@link Criteria}.
-	 *
+	 * 
 	 * @param criteria must not be {@literal null}.
 	 * @return
 	 */
@@ -284,40 +270,12 @@ public class Aggregation {
 		return new MatchOperation(criteria);
 	}
 
-	/**
-	 * Creates a new {@link LookupOperation}.
-	 *
-	 * @param from must not be {@literal null}.
-	 * @param localField must not be {@literal null}.
-	 * @param foreignField must not be {@literal null}.
-	 * @param as must not be {@literal null}.
-	 * @return never {@literal null}.
-	 * @since 1.9
-	 */
-	public static LookupOperation lookup(String from, String localField, String foreignField, String as) {
-		return lookup(field(from), field(localField), field(foreignField), field(as));
-	}
-
-	/**
-	 * Creates a new {@link LookupOperation} for the given {@link Fields}.
-	 *
-	 * @param from must not be {@literal null}.
-	 * @param localField must not be {@literal null}.
-	 * @param foreignField must not be {@literal null}.
-	 * @param as must not be {@literal null}.
-	 * @return never {@literal null}.
-	 * @since 1.9
-	 */
-	public static LookupOperation lookup(Field from, Field localField, Field foreignField, Field as) {
-		return new LookupOperation(from, localField, foreignField, as);
-	}
-
 	/**
 	 * Creates a new {@link Fields} instance for the given field names.
-	 *
+	 * 
+	 * @see Fields#fields(String...)
 	 * @param fields must not be {@literal null}.
 	 * @return
-	 * @see Fields#fields(String...)
 	 */
 	public static Fields fields(String... fields) {
 		return Fields.fields(fields);
@@ -325,7 +283,7 @@ public class Aggregation {
 
 	/**
 	 * Creates a new {@link Fields} instance from the given field name and target reference.
-	 *
+	 * 
 	 * @param name must not be {@literal null} or empty.
 	 * @param target must not be {@literal null} or empty.
 	 * @return
@@ -337,7 +295,7 @@ public class Aggregation {
 	/**
 	 * Creates a new {@link GeoNearOperation} instance from the given {@link NearQuery} and the{@code distanceField}. The
 	 * {@code distanceField} defines output field that contains the calculated distance.
-	 *
+	 * 
 	 * @param query must not be {@literal null}.
 	 * @param distanceField must not be {@literal null} or empty.
 	 * @return
@@ -349,7 +307,7 @@ public class Aggregation {
 
 	/**
 	 * Returns a new {@link AggregationOptions.Builder}.
-	 *
+	 * 
 	 * @return
 	 * @since 1.6
 	 */
@@ -359,7 +317,7 @@ public class Aggregation {
 
 	/**
 	 * Converts this {@link Aggregation} specification to a {@link DBObject}.
-	 *
+	 * 
 	 * @param inputCollectionName the name of the input collection
 	 * @return the {@code DBObject} representing this aggregation
 	 */
@@ -373,14 +331,8 @@ public class Aggregation {
 			operationDocuments.add(operation.toDBObject(context));
 
 			if (operation instanceof FieldsExposingAggregationOperation) {
-
 				FieldsExposingAggregationOperation exposedFieldsOperation = (FieldsExposingAggregationOperation) operation;
-
-				if (operation instanceof InheritsFieldsAggregationOperation) {
-					context = new InheritingExposedFieldsAggregationOperationContext(exposedFieldsOperation.getFields(), context);
-				} else {
-					context = new ExposedFieldsAggregationOperationContext(exposedFieldsOperation.getFields(), context);
-				}
+				context = new ExposedFieldsAggregationOperationContext(exposedFieldsOperation.getFields(), rootContext);
 			}
 		}
 
@@ -404,7 +356,7 @@ public class Aggregation {
 
 	/**
 	 * Simple {@link AggregationOperationContext} that just returns {@link FieldReference}s as is.
-	 *
+	 * 
 	 * @author Oliver Gierke
 	 */
 	private static class NoOpAggregationOperationContext implements AggregationOperationContext {
@@ -439,7 +391,7 @@ public class Aggregation {
 
 	/**
 	 * Describes the system variables available in MongoDB aggregation framework pipeline expressions.
-	 *
+	 * 
 	 * @author Thomas Darimont
 	 * @see http://docs.mongodb.org/manual/reference/aggregation-variables
 	 */
@@ -452,7 +404,7 @@ public class Aggregation {
 		/**
 		 * Return {@literal true} if the given {@code fieldRef} denotes a well-known system variable, {@literal false}
 		 * otherwise.
-		 *
+		 * 
 		 * @param fieldRef may be {@literal null}.
 		 * @return
 		 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AggregationExpression.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2015-2016 the original author or authors.
+ * Copyright 2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,17 +20,16 @@ import com.mongodb.DBObject;
 /**
  * An {@link AggregationExpression} can be used with field expressions in aggregation pipeline stages like
  * {@code project} and {@code group}.
- *
+ * 
  * @author Thomas Darimont
  * @author Oliver Gierke
- * @author Christoph Strobl
  */
-public interface AggregationExpression {
+interface AggregationExpression {
 
 	/**
 	 * Turns the {@link AggregationExpression} into a {@link DBObject} within the given
 	 * {@link AggregationOperationContext}.
-	 *
+	 * 
 	 * @param context
 	 * @return
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AggregationFunctionExpressions.java

@@ -16,7 +16,6 @@
 package org.springframework.data.mongodb.core.aggregation;
 
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 
 import org.springframework.util.Assert;
@@ -57,7 +56,7 @@ public enum AggregationFunctionExpressions {
 	static class FunctionExpression implements AggregationExpression {
 
 		private final String name;
-		private final List<Object> values;
+		private final Object[] values;
 
 		/**
 		 * Creates a new {@link FunctionExpression} for the given name and values.
@@ -71,7 +70,7 @@ public enum AggregationFunctionExpressions {
 			Assert.notNull(values, "Values must not be null!");
 
 			this.name = name;
-			this.values = Arrays.asList(values);
+			this.values = values;
 		}
 
 		/*
@@ -81,10 +80,10 @@ public enum AggregationFunctionExpressions {
 		@Override
 		public DBObject toDbObject(AggregationOperationContext context) {
 
-			List<Object> args = new ArrayList<Object>(values.size());
+			List<Object> args = new ArrayList<Object>(values.length);
 
-			for (Object value : values) {
-				args.add(unpack(value, context));
+			for (int i = 0; i < values.length; i++) {
+				args.add(unpack(values[i], context));
 			}
 
 			return new BasicDBObject("$" + name, args);

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/AggregationResults.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2017 the original author or authors.
+ * Copyright 2013-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -29,7 +29,6 @@ import com.mongodb.DBObject;
  * @author Tobias Trelle
  * @author Oliver Gierke
  * @author Thomas Darimont
- * @author Mark Paluch
  * @param <T> The class in which the results are mapped onto.
  * @since 1.3
  */
@@ -47,8 +46,8 @@ public class AggregationResults<T> implements Iterable<T> {
 	 */
 	public AggregationResults(List<T> mappedResults, DBObject rawResults) {
 
-		Assert.notNull(mappedResults, "List of mapped results must not be null!");
-		Assert.notNull(rawResults, "Raw results must not be null!");
+		Assert.notNull(mappedResults);
+		Assert.notNull(rawResults);
 
 		this.mappedResults = Collections.unmodifiableList(mappedResults);
 		this.rawResults = rawResults;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/ExposedFieldsAggregationOperationContext.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -24,10 +24,9 @@ import com.mongodb.DBObject;
 /**
  * {@link AggregationOperationContext} that combines the available field references from a given
  * {@code AggregationOperationContext} and an {@link FieldsExposingAggregationOperation}.
- *
+ * 
  * @author Thomas Darimont
  * @author Oliver Gierke
- * @author Mark Paluch
  * @since 1.4
  */
 class ExposedFieldsAggregationOperationContext implements AggregationOperationContext {
@@ -38,12 +37,11 @@ class ExposedFieldsAggregationOperationContext implements AggregationOperationCo
 	/**
 	 * Creates a new {@link ExposedFieldsAggregationOperationContext} from the given {@link ExposedFields}. Uses the given
 	 * {@link AggregationOperationContext} to perform a mapping to mongo types if necessary.
-	 *
+	 * 
 	 * @param exposedFields must not be {@literal null}.
 	 * @param rootContext must not be {@literal null}.
 	 */
-	public ExposedFieldsAggregationOperationContext(ExposedFields exposedFields,
-			AggregationOperationContext rootContext) {
+	public ExposedFieldsAggregationOperationContext(ExposedFields exposedFields, AggregationOperationContext rootContext) {
 
 		Assert.notNull(exposedFields, "ExposedFields must not be null!");
 		Assert.notNull(rootContext, "RootContext must not be null!");
@@ -81,7 +79,7 @@ class ExposedFieldsAggregationOperationContext implements AggregationOperationCo
 
 	/**
 	 * Returns a {@link FieldReference} to the given {@link Field} with the given {@code name}.
-	 *
+	 * 
 	 * @param field may be {@literal null}
 	 * @param name must not be {@literal null}
 	 * @return
@@ -90,22 +88,6 @@ class ExposedFieldsAggregationOperationContext implements AggregationOperationCo
 
 		Assert.notNull(name, "Name must not be null!");
 
-		FieldReference exposedField = resolveExposedField(field, name);
-		if (exposedField != null) {
-			return exposedField;
-		}
-
-		throw new IllegalArgumentException(String.format("Invalid reference '%s'!", name));
-	}
-
-	/**
-	 * Resolves a {@link field}/{@link name} for a {@link FieldReference} if possible.
-	 *
-	 * @param field may be {@literal null}
-	 * @param name must not be {@literal null}
-	 * @return the resolved reference or {@literal null}
-	 */
-	protected FieldReference resolveExposedField(Field field, String name) {
 		ExposedField exposedField = exposedFields.getField(name);
 
 		if (exposedField != null) {
@@ -129,6 +111,7 @@ class ExposedFieldsAggregationOperationContext implements AggregationOperationCo
 				return new FieldReference(new ExposedField(name, true));
 			}
 		}
-		return null;
+
+		throw new IllegalArgumentException(String.format("Invalid reference '%s'!", name));
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/FieldsExposingAggregationOperation.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,28 +16,17 @@
 package org.springframework.data.mongodb.core.aggregation;
 
 /**
- * {@link AggregationOperation} that exposes {@link ExposedFields} that can be used for later aggregation pipeline
- * {@code AggregationOperation}s. A {@link FieldsExposingAggregationOperation} implementing the
- * {@link InheritsFieldsAggregationOperation} will expose fields from its parent operations. Not implementing
- * {@link InheritsFieldsAggregationOperation} will replace existing exposed fields.
- *
+ * {@link AggregationOperation} that exposes new {@link ExposedFields} that can be used for later aggregation pipeline
+ * {@code AggregationOperation}s.
+ * 
  * @author Thomas Darimont
- * @author Mark Paluch
  */
 public interface FieldsExposingAggregationOperation extends AggregationOperation {
 
 	/**
 	 * Returns the fields exposed by the {@link AggregationOperation}.
-	 *
+	 * 
 	 * @return will never be {@literal null}.
 	 */
 	ExposedFields getFields();
-
-	/**
-	 * Marker interface for {@link AggregationOperation} that inherits fields from previous operations.
-	 */
-	static interface InheritsFieldsAggregationOperation extends FieldsExposingAggregationOperation {
-
-	}
-
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/InheritingExposedFieldsAggregationOperationContext.java

@@ -1,65 +0,0 @@
-/*
- * Copyright 2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-package org.springframework.data.mongodb.core.aggregation;
-
-import org.springframework.data.mongodb.core.aggregation.ExposedFields.FieldReference;
-import org.springframework.util.Assert;
-
-/**
- * {@link ExposedFieldsAggregationOperationContext} that inherits fields from its parent
- * {@link AggregationOperationContext}.
- *
- * @author Mark Paluch
- */
-class InheritingExposedFieldsAggregationOperationContext extends ExposedFieldsAggregationOperationContext {
-
-	private final AggregationOperationContext previousContext;
-
-	/**
-	 * Creates a new {@link ExposedFieldsAggregationOperationContext} from the given {@link ExposedFields}. Uses the given
-	 * {@link AggregationOperationContext} to perform a mapping to mongo types if necessary.
-	 *
-	 * @param exposedFields must not be {@literal null}.
-	 * @param previousContext must not be {@literal null}.
-	 */
-	public InheritingExposedFieldsAggregationOperationContext(ExposedFields exposedFields,
-			AggregationOperationContext previousContext) {
-
-		super(exposedFields, previousContext);
-		Assert.notNull(previousContext, "PreviousContext must not be null!");
-		this.previousContext = previousContext;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.aggregation.ExposedFieldsAggregationOperationContext#resolveExposedField(org.springframework.data.mongodb.core.aggregation.Field, java.lang.String)
-	 */
-	@Override
-	protected FieldReference resolveExposedField(Field field, String name) {
-
-		FieldReference fieldReference = super.resolveExposedField(field, name);
-		if (fieldReference != null) {
-			return fieldReference;
-		}
-
-		if (field != null) {
-			return previousContext.getReference(field);
-		}
-
-		return previousContext.getReference(name);
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/LookupOperation.java

@@ -1,196 +0,0 @@
-/*
- * Copyright 2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core.aggregation;
-
-import org.springframework.data.mongodb.core.aggregation.ExposedFields.ExposedField;
-import org.springframework.data.mongodb.core.aggregation.FieldsExposingAggregationOperation.InheritsFieldsAggregationOperation;
-import org.springframework.util.Assert;
-
-import com.mongodb.BasicDBObject;
-import com.mongodb.DBObject;
-
-/**
- * Encapsulates the aggregation framework {@code $lookup}-operation. We recommend to use the static factory method
- * {@link Aggregation#lookup(String, String, String, String)} instead of creating instances of this class directly.
- *
- * @author Alessio Fachechi
- * @author Christoph Strobl
- * @author Mark Paluch
- * @see http://docs.mongodb.org/manual/reference/aggregation/lookup/#stage._S_lookup
- * @since 1.9
- */
-public class LookupOperation implements FieldsExposingAggregationOperation, InheritsFieldsAggregationOperation {
-
-	private Field from;
-	private Field localField;
-	private Field foreignField;
-	private ExposedField as;
-
-	/**
-	 * Creates a new {@link LookupOperation} for the given {@link Field}s.
-	 *
-	 * @param from must not be {@literal null}.
-	 * @param localField must not be {@literal null}.
-	 * @param foreignField must not be {@literal null}.
-	 * @param as must not be {@literal null}.
-	 */
-	public LookupOperation(Field from, Field localField, Field foreignField, Field as) {
-
-		Assert.notNull(from, "From must not be null!");
-		Assert.notNull(localField, "LocalField must not be null!");
-		Assert.notNull(foreignField, "ForeignField must not be null!");
-		Assert.notNull(as, "As must not be null!");
-
-		this.from = from;
-		this.localField = localField;
-		this.foreignField = foreignField;
-		this.as = new ExposedField(as, true);
-	}
-
-	private LookupOperation() {
-		// used by builder
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.aggregation.FieldsExposingAggregationOperation#getFields()
-	 */
-	@Override
-	public ExposedFields getFields() {
-		return ExposedFields.from(as);
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.aggregation.AggregationOperation#toDBObject(org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
-	 */
-	@Override
-	public DBObject toDBObject(AggregationOperationContext context) {
-
-		BasicDBObject lookupObject = new BasicDBObject();
-
-		lookupObject.append("from", from.getTarget());
-		lookupObject.append("localField", localField.getTarget());
-		lookupObject.append("foreignField", foreignField.getTarget());
-		lookupObject.append("as", as.getTarget());
-
-		return new BasicDBObject("$lookup", lookupObject);
-	}
-
-	/**
-	 * Get a builder that allows creation of {@link LookupOperation}.
-	 *
-	 * @return
-	 */
-	public static FromBuilder newLookup() {
-		return new LookupOperationBuilder();
-	}
-
-	public static interface FromBuilder {
-
-		/**
-		 * @param name the collection in the same database to perform the join with, must not be {@literal null} or empty.
-		 * @return
-		 */
-		LocalFieldBuilder from(String name);
-	}
-
-	public static interface LocalFieldBuilder {
-
-		/**
-		 * @param name the field from the documents input to the {@code $lookup} stage, must not be {@literal null} or
-		 *          empty.
-		 * @return
-		 */
-		ForeignFieldBuilder localField(String name);
-	}
-
-	public static interface ForeignFieldBuilder {
-
-		/**
-		 * @param name the field from the documents in the {@code from} collection, must not be {@literal null} or empty.
-		 * @return
-		 */
-		AsBuilder foreignField(String name);
-	}
-
-	public static interface AsBuilder {
-
-		/**
-		 * @param name the name of the new array field to add to the input documents, must not be {@literal null} or empty.
-		 * @return
-		 */
-		LookupOperation as(String name);
-	}
-
-	/**
-	 * Builder for fluent {@link LookupOperation} creation.
-	 *
-	 * @author Christoph Strobl
-	 * @since 1.9
-	 */
-	public static final class LookupOperationBuilder
-			implements FromBuilder, LocalFieldBuilder, ForeignFieldBuilder, AsBuilder {
-
-		private final LookupOperation lookupOperation;
-
-		private LookupOperationBuilder() {
-			this.lookupOperation = new LookupOperation();
-		}
-
-		/**
-		 * Creates new builder for {@link LookupOperation}.
-		 *
-		 * @return never {@literal null}.
-		 */
-		public static FromBuilder newBuilder() {
-			return new LookupOperationBuilder();
-		}
-
-		@Override
-		public LocalFieldBuilder from(String name) {
-
-			Assert.hasText(name, "'From' must not be null or empty!");
-			lookupOperation.from = Fields.field(name);
-			return this;
-		}
-
-		@Override
-		public LookupOperation as(String name) {
-
-			Assert.hasText(name, "'As' must not be null or empty!");
-			lookupOperation.as = new ExposedField(Fields.field(name), true);
-			return new LookupOperation(lookupOperation.from, lookupOperation.localField, lookupOperation.foreignField,
-					lookupOperation.as);
-		}
-
-		@Override
-		public AsBuilder foreignField(String name) {
-
-			Assert.hasText(name, "'ForeignField' must not be null or empty!");
-			lookupOperation.foreignField = Fields.field(name);
-			return this;
-		}
-
-		@Override
-		public ForeignFieldBuilder localField(String name) {
-
-			Assert.hasText(name, "'LocalField' must not be null or empty!");
-			lookupOperation.localField = Fields.field(name);
-			return this;
-		}
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/ProjectionOperation.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2017 the original author or authors.
+ * Copyright 2013-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,7 +20,6 @@ import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 
-
 import org.springframework.data.mongodb.core.aggregation.ExposedFields.ExposedField;
 import org.springframework.data.mongodb.core.aggregation.Fields.AggregationField;
 import org.springframework.data.mongodb.core.aggregation.ProjectionOperation.ProjectionOperationBuilder.FieldProjection;
@@ -553,7 +552,7 @@ public class ProjectionOperation implements FieldsExposingAggregationOperation {
 		/**
 		 * Generates an {@code $mod} expression that divides the value of the given field by the previously mentioned field
 		 * and returns the remainder.
-		 *
+		 * 
 		 * @param fieldReference
 		 * @return
 		 */
@@ -567,7 +566,7 @@ public class ProjectionOperation implements FieldsExposingAggregationOperation {
 			return project("size");
 		}
 
-		/*
+		/* 
 		 * (non-Javadoc)
 		 * @see org.springframework.data.mongodb.core.aggregation.AggregationOperation#toDBObject(org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 		 */
@@ -623,7 +622,6 @@ public class ProjectionOperation implements FieldsExposingAggregationOperation {
 		 * 
 		 * @author Oliver Gierke
 		 * @author Thomas Darimont
-		 * @author Mark Paluch
 		 */
 		static class FieldProjection extends Projection {
 
@@ -642,7 +640,7 @@ public class ProjectionOperation implements FieldsExposingAggregationOperation {
 
 			private FieldProjection(Field field, Object value) {
 
-				super(new ExposedField(field.getName(), true));
+				super(field);
 
 				this.field = field;
 				this.value = value;
@@ -734,7 +732,7 @@ public class ProjectionOperation implements FieldsExposingAggregationOperation {
 				this.values = Arrays.asList(values);
 			}
 
-			/*
+			/* 
 			 * (non-Javadoc)
 			 * @see org.springframework.data.mongodb.core.aggregation.ProjectionOperation.Projection#toDBObject(org.springframework.data.mongodb.core.aggregation.AggregationOperationContext)
 			 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/SkipOperation.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -38,7 +38,7 @@ public class SkipOperation implements AggregationOperation {
 	/**
 	 * Creates a new {@link SkipOperation} skipping the given number of elements.
 	 * 
-	 * @param skipCount number of documents to skip, must not be less than zero.
+	 * @param skipCount number of documents to skip.
 	 */
 	public SkipOperation(long skipCount) {
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/TypeBasedAggregationOperationContext.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2017 the original author or authors.
+ * Copyright 2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -34,7 +34,6 @@ import com.mongodb.DBObject;
  * property references into document field names.
  * 
  * @author Oliver Gierke
- * @author Mark Paluch
  * @since 1.3
  */
 public class TypeBasedAggregationOperationContext implements AggregationOperationContext {
@@ -96,7 +95,7 @@ public class TypeBasedAggregationOperationContext implements AggregationOperatio
 
 		PersistentPropertyPath<MongoPersistentProperty> propertyPath = mappingContext.getPersistentPropertyPath(
 				field.getTarget(), type);
-		Field mappedField = field(field.getName(),
+		Field mappedField = field(propertyPath.getLeafProperty().getName(),
 				propertyPath.toDotPath(MongoPersistentProperty.PropertyToFieldNameConverter.INSTANCE));
 
 		return new FieldReference(new ExposedField(mappedField, true));

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/AbstractMongoConverter.java

@@ -75,13 +75,15 @@ public abstract class AbstractMongoConverter implements MongoConverter, Initiali
 	 */
 	private void initializeConverters() {
 
-		conversionService.addConverter(ObjectIdToStringConverter.INSTANCE);
-		conversionService.addConverter(StringToObjectIdConverter.INSTANCE);
-
+		if (!conversionService.canConvert(ObjectId.class, String.class)) {
+			conversionService.addConverter(ObjectIdToStringConverter.INSTANCE);
+		}
+		if (!conversionService.canConvert(String.class, ObjectId.class)) {
+			conversionService.addConverter(StringToObjectIdConverter.INSTANCE);
+		}
 		if (!conversionService.canConvert(ObjectId.class, BigInteger.class)) {
 			conversionService.addConverter(ObjectIdToBigIntegerConverter.INSTANCE);
 		}
-
 		if (!conversionService.canConvert(BigInteger.class, ObjectId.class)) {
 			conversionService.addConverter(BigIntegerToObjectIdConverter.INSTANCE);
 		}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/ConverterRegistration.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -23,7 +23,6 @@ import org.springframework.util.Assert;
  * Conversion registration information.
  * 
  * @author Oliver Gierke
- * @author Mark Paluch
  */
 class ConverterRegistration {
 
@@ -40,7 +39,7 @@ class ConverterRegistration {
 	 */
 	public ConverterRegistration(ConvertiblePair convertiblePair, boolean isReading, boolean isWriting) {
 
-		Assert.notNull(convertiblePair, "ConvertiblePair must not be null!");
+		Assert.notNull(convertiblePair);
 
 		this.convertiblePair = convertiblePair;
 		this.reading = isReading;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/CustomConversions.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -42,6 +42,16 @@ import org.springframework.data.convert.ReadingConverter;
 import org.springframework.data.convert.ThreeTenBackPortConverters;
 import org.springframework.data.convert.WritingConverter;
 import org.springframework.data.mapping.model.SimpleTypeHolder;
+import org.springframework.data.mongodb.core.convert.MongoConverters.BigDecimalToStringConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.BigIntegerToStringConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.NamedMongoScriptToDBObjectConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.DBObjectToNamedMongoScriptCoverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.DBObjectToStringConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.StringToBigDecimalConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.StringToBigIntegerConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.StringToURLConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.TermToStringConverter;
+import org.springframework.data.mongodb.core.convert.MongoConverters.URLToStringConverter;
 import org.springframework.data.mongodb.core.mapping.MongoSimpleTypes;
 import org.springframework.data.util.CacheValue;
 import org.springframework.util.Assert;
@@ -56,7 +66,6 @@ import org.springframework.util.Assert;
  * @author Oliver Gierke
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class CustomConversions {
 
@@ -89,7 +98,7 @@ public class CustomConversions {
 	 */
 	public CustomConversions(List<?> converters) {
 
-		Assert.notNull(converters, "List of converters must not be null!");
+		Assert.notNull(converters);
 
 		this.readingPairs = new LinkedHashSet<ConvertiblePair>();
 		this.writingPairs = new LinkedHashSet<ConvertiblePair>();
@@ -103,7 +112,17 @@ public class CustomConversions {
 		// Add user provided converters to make sure they can override the defaults
 		toRegister.addAll(converters);
 		toRegister.add(CustomToStringConverter.INSTANCE);
-		toRegister.addAll(MongoConverters.getConvertersToRegister());
+		toRegister.add(BigDecimalToStringConverter.INSTANCE);
+		toRegister.add(StringToBigDecimalConverter.INSTANCE);
+		toRegister.add(BigIntegerToStringConverter.INSTANCE);
+		toRegister.add(StringToBigIntegerConverter.INSTANCE);
+		toRegister.add(URLToStringConverter.INSTANCE);
+		toRegister.add(StringToURLConverter.INSTANCE);
+		toRegister.add(DBObjectToStringConverter.INSTANCE);
+		toRegister.add(TermToStringConverter.INSTANCE);
+		toRegister.add(NamedMongoScriptToDBObjectConverter.INSTANCE);
+		toRegister.add(DBObjectToNamedMongoScriptCoverter.INSTANCE);
+
 		toRegister.addAll(JodaTimeConverters.getConvertersToRegister());
 		toRegister.addAll(GeoConverters.getConvertersToRegister());
 		toRegister.addAll(Jsr310Converters.getConvertersToRegister());
@@ -167,15 +186,14 @@ public class CustomConversions {
 			}
 
 			if (!added) {
-				throw new IllegalArgumentException(
-						"Given set contains element that is neither Converter nor ConverterFactory!");
+				throw new IllegalArgumentException("Given set contains element that is neither Converter nor ConverterFactory!");
 			}
 		}
 	}
 
 	/**
-	 * Registers a conversion for the given converter. Inspects either generics of {@link Converter} and
-	 * {@link ConverterFactory} or the {@link ConvertiblePair}s returned by a {@link GenericConverter}.
+	 * Registers a conversion for the given converter. Inspects either generics or the {@link ConvertiblePair}s returned
+	 * by a {@link GenericConverter}.
 	 * 
 	 * @param converter
 	 */
@@ -190,10 +208,6 @@ public class CustomConversions {
 			for (ConvertiblePair pair : genericConverter.getConvertibleTypes()) {
 				register(new ConverterRegistration(pair, isReading, isWriting));
 			}
-		} else if (converter instanceof ConverterFactory) {
-
-			Class<?>[] arguments = GenericTypeResolver.resolveTypeArguments(converter.getClass(), ConverterFactory.class);
-			register(new ConverterRegistration(arguments[0], arguments[1], isReading, isWriting));
 		} else if (converter instanceof Converter) {
 			Class<?>[] arguments = GenericTypeResolver.resolveTypeArguments(converter.getClass(), Converter.class);
 			register(new ConverterRegistration(arguments[0], arguments[1], isReading, isWriting));
@@ -346,8 +360,8 @@ public class CustomConversions {
 	private static Class<?> getCustomTarget(Class<?> sourceType, Class<?> requestedTargetType,
 			Collection<ConvertiblePair> pairs) {
 
-		Assert.notNull(sourceType, "Source Class must not be null!");
-		Assert.notNull(pairs, "Collection of ConvertiblePair must not be null!");
+		Assert.notNull(sourceType);
+		Assert.notNull(pairs);
 
 		if (requestedTargetType != null && pairs.contains(new ConvertiblePair(sourceType, requestedTargetType))) {
 			return requestedTargetType;
@@ -398,10 +412,6 @@ public class CustomConversions {
 
 		INSTANCE;
 
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.GenericConverter#getConvertibleTypes()
-		 */
 		public Set<ConvertiblePair> getConvertibleTypes() {
 
 			ConvertiblePair localeToString = new ConvertiblePair(Locale.class, String.class);
@@ -410,10 +420,6 @@ public class CustomConversions {
 			return new HashSet<ConvertiblePair>(Arrays.asList(localeToString, booleanToString));
 		}
 
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.GenericConverter#convert(java.lang.Object, org.springframework.core.convert.TypeDescriptor, org.springframework.core.convert.TypeDescriptor)
-		 */
 		public Object convert(Object source, TypeDescriptor sourceType, TypeDescriptor targetType) {
 			return source.toString();
 		}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DBObjectAccessor.java

@@ -75,7 +75,9 @@ class DBObjectAccessor {
 			String part = parts.next();
 
 			if (parts.hasNext()) {
-				dbObject = getOrCreateNestedDbObject(part, dbObject);
+				BasicDBObject nestedDbObject = new BasicDBObject();
+				dbObject.put(part, nestedDbObject);
+				dbObject = nestedDbObject;
 			} else {
 				dbObject.put(part, value);
 			}
@@ -114,48 +116,8 @@ class DBObjectAccessor {
 		return result;
 	}
 
-	/**
-	 * Returns whether the underlying {@link DBObject} has a value ({@literal null} or non-{@literal null}) for the given
-	 * {@link MongoPersistentProperty}.
-	 * 
-	 * @param property must not be {@literal null}.
-	 * @return
-	 */
-	public boolean hasValue(MongoPersistentProperty property) {
-
-		Assert.notNull(property, "Property must not be null!");
-
-		String fieldName = property.getFieldName();
-
-		if (!fieldName.contains(".")) {
-			return this.dbObject.containsField(fieldName);
-		}
-
-		String[] parts = fieldName.split("\\.");
-		Map<String, Object> source = this.dbObject;
-		Object result = null;
-
-		for (int i = 1; i < parts.length; i++) {
-
-			result = source.get(parts[i - 1]);
-			source = getAsMap(result);
-
-			if (source == null) {
-				return false;
-			}
-		}
-
-		return source.containsKey(parts[parts.length - 1]);
-	}
-
-	/**
-	 * Returns the given source object as map, i.e. {@link BasicDBObject}s and maps as is or {@literal null} otherwise.
-	 * 
-	 * @param source can be {@literal null}.
-	 * @return
-	 */
 	@SuppressWarnings("unchecked")
-	private static Map<String, Object> getAsMap(Object source) {
+	private Map<String, Object> getAsMap(Object source) {
 
 		if (source instanceof BasicDBObject) {
 			return (BasicDBObject) source;
@@ -167,26 +129,4 @@ class DBObjectAccessor {
 
 		return null;
 	}
-
-	/**
-	 * Returns the {@link DBObject} which either already exists in the given source under the given key, or creates a new
-	 * nested one, registers it with the source and returns it.
-	 * 
-	 * @param key must not be {@literal null} or empty.
-	 * @param source must not be {@literal null}.
-	 * @return
-	 */
-	private static DBObject getOrCreateNestedDbObject(String key, DBObject source) {
-
-		Object existing = source.get(key);
-
-		if (existing instanceof BasicDBObject) {
-			return (BasicDBObject) existing;
-		}
-
-		DBObject nested = new BasicDBObject();
-		source.put(key, nested);
-
-		return nested;
-	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultDbRefResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2016 the original author or authors.
+ * Copyright 2013-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -180,8 +180,8 @@ public class DefaultDbRefResolver implements DbRefResolver {
 	 * @author Oliver Gierke
 	 * @author Christoph Strobl
 	 */
-	static class LazyLoadingInterceptor
-			implements MethodInterceptor, org.springframework.cglib.proxy.MethodInterceptor, Serializable {
+	static class LazyLoadingInterceptor implements MethodInterceptor, org.springframework.cglib.proxy.MethodInterceptor,
+			Serializable {
 
 		private static final Method INITIALIZE_METHOD, TO_DBREF_METHOD, FINALIZE_METHOD;
 
@@ -387,8 +387,7 @@ public class DefaultDbRefResolver implements DbRefResolver {
 				} catch (RuntimeException ex) {
 
 					DataAccessException translatedException = this.exceptionTranslator.translateExceptionIfPossible(ex);
-					throw new LazyLoadingException("Unable to lazily resolve DBRef!",
-							translatedException != null ? translatedException : ex);
+					throw new LazyLoadingException("Unable to lazily resolve DBRef!", translatedException);
 				}
 			}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/DefaultMongoTypeMapper.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -45,9 +45,9 @@ import com.mongodb.DBObject;
 public class DefaultMongoTypeMapper extends DefaultTypeMapper<DBObject> implements MongoTypeMapper {
 
 	public static final String DEFAULT_TYPE_KEY = "_class";
-	@SuppressWarnings("rawtypes") //
+	@SuppressWarnings("rawtypes")//
 	private static final TypeInformation<List> LIST_TYPE_INFO = ClassTypeInformation.from(List.class);
-	@SuppressWarnings("rawtypes") //
+	@SuppressWarnings("rawtypes")//
 	private static final TypeInformation<Map> MAP_TYPE_INFO = ClassTypeInformation.from(Map.class);
 
 	private final TypeAliasAccessor<DBObject> accessor;
@@ -58,12 +58,12 @@ public class DefaultMongoTypeMapper extends DefaultTypeMapper<DBObject> implemen
 	}
 
 	public DefaultMongoTypeMapper(String typeKey) {
-		this(typeKey, Arrays.asList(new SimpleTypeInformationMapper()));
+		this(typeKey, Arrays.asList(SimpleTypeInformationMapper.INSTANCE));
 	}
 
 	public DefaultMongoTypeMapper(String typeKey, MappingContext<? extends PersistentEntity<?, ?>, ?> mappingContext) {
-		this(typeKey, new DBObjectTypeAliasAccessor(typeKey), mappingContext,
-				Arrays.asList(new SimpleTypeInformationMapper()));
+		this(typeKey, new DBObjectTypeAliasAccessor(typeKey), mappingContext, Arrays
+				.asList(SimpleTypeInformationMapper.INSTANCE));
 	}
 
 	public DefaultMongoTypeMapper(String typeKey, List<? extends TypeInformationMapper> mappers) {
@@ -71,8 +71,7 @@ public class DefaultMongoTypeMapper extends DefaultTypeMapper<DBObject> implemen
 	}
 
 	private DefaultMongoTypeMapper(String typeKey, TypeAliasAccessor<DBObject> accessor,
-			MappingContext<? extends PersistentEntity<?, ?>, ?> mappingContext,
-			List<? extends TypeInformationMapper> mappers) {
+			MappingContext<? extends PersistentEntity<?, ?>, ?> mappingContext, List<? extends TypeInformationMapper> mappers) {
 
 		super(accessor, mappingContext, mappers);
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/GeoConverters.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2017 the original author or authors.
+ * Copyright 2014-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -41,7 +41,6 @@ import org.springframework.data.mongodb.core.geo.GeoJsonPolygon;
 import org.springframework.data.mongodb.core.geo.Sphere;
 import org.springframework.data.mongodb.core.query.GeoCommand;
 import org.springframework.util.Assert;
-import org.springframework.util.NumberUtils;
 import org.springframework.util.ObjectUtils;
 
 import com.mongodb.BasicDBList;
@@ -54,7 +53,6 @@ import com.mongodb.DBObject;
  * @author Thomas Darimont
  * @author Oliver Gierke
  * @author Christoph Strobl
- * @author Thiago Diniz da Silveira
  * @since 1.5
  */
 abstract class GeoConverters {
@@ -113,17 +111,9 @@ abstract class GeoConverters {
 		@Override
 		public Point convert(DBObject source) {
 
-			if (source == null) {
-				return null;
-			}
-
 			Assert.isTrue(source.keySet().size() == 2, "Source must contain 2 elements");
 
-			if (source.containsField("type")) {
-				return DbObjectToGeoJsonPointConverter.INSTANCE.convert(source);
-			}
-
-			return new Point(toPrimitiveDoubleValue(source.get("x")), toPrimitiveDoubleValue(source.get("y")));
+			return source == null ? null : new Point((Double) source.get("x"), (Double) source.get("y"));
 		}
 	}
 
@@ -257,12 +247,9 @@ abstract class GeoConverters {
 			}
 
 			DBObject center = (DBObject) source.get("center");
-			Number radius = (Number) source.get("radius");
+			Double radius = (Double) source.get("radius");
 
-			Assert.notNull(center, "Center must not be null!");
-			Assert.notNull(radius, "Radius must not be null!");
-
-			Distance distance = new Distance(toPrimitiveDoubleValue(radius));
+			Distance distance = new Distance(radius);
 
 			if (source.containsField("metric")) {
 
@@ -272,6 +259,9 @@ abstract class GeoConverters {
 				distance = distance.in(Metrics.valueOf(metricString));
 			}
 
+			Assert.notNull(center, "Center must not be null!");
+			Assert.notNull(radius, "Radius must not be null!");
+
 			return new Circle(DbObjectToPointConverter.INSTANCE.convert(center), distance);
 		}
 	}
@@ -328,12 +318,9 @@ abstract class GeoConverters {
 			}
 
 			DBObject center = (DBObject) source.get("center");
-			Number radius = (Number) source.get("radius");
+			Double radius = (Double) source.get("radius");
 
-			Assert.notNull(center, "Center must not be null!");
-			Assert.notNull(radius, "Radius must not be null!");
-
-			Distance distance = new Distance(toPrimitiveDoubleValue(radius));
+			Distance distance = new Distance(radius);
 
 			if (source.containsField("metric")) {
 
@@ -343,6 +330,9 @@ abstract class GeoConverters {
 				distance = distance.in(Metrics.valueOf(metricString));
 			}
 
+			Assert.notNull(center, "Center must not be null!");
+			Assert.notNull(radius, "Radius must not be null!");
+
 			return new Sphere(DbObjectToPointConverter.INSTANCE.convert(center), distance);
 		}
 	}
@@ -601,8 +591,8 @@ abstract class GeoConverters {
 			Assert.isTrue(ObjectUtils.nullSafeEquals(source.get("type"), "Point"),
 					String.format("Cannot convert type '%s' to Point.", source.get("type")));
 
-			List<Number> dbl = (List<Number>) source.get("coordinates");
-			return new GeoJsonPoint(toPrimitiveDoubleValue(dbl.get(0)), toPrimitiveDoubleValue(dbl.get(1)));
+			List<Double> dbl = (List<Double>) source.get("coordinates");
+			return new GeoJsonPoint(dbl.get(0).doubleValue(), dbl.get(1).doubleValue());
 		}
 	}
 
@@ -834,10 +824,9 @@ abstract class GeoConverters {
 
 			Assert.isInstanceOf(List.class, point);
 
-			List<Number> coordinatesList = (List<Number>) point;
+			List<Double> coordinatesList = (List<Double>) point;
 
-			points.add(new GeoJsonPoint(toPrimitiveDoubleValue(coordinatesList.get(0)),
-					toPrimitiveDoubleValue(coordinatesList.get(1))));
+			points.add(new GeoJsonPoint(coordinatesList.get(0).doubleValue(), coordinatesList.get(1).doubleValue()));
 		}
 		return points;
 	}
@@ -852,10 +841,4 @@ abstract class GeoConverters {
 	static GeoJsonPolygon toGeoJsonPolygon(BasicDBList dbList) {
 		return new GeoJsonPolygon(toListOfPoint((BasicDBList) dbList.get(0)));
 	}
-
-	private static double toPrimitiveDoubleValue(Object value) {
-
-		Assert.isInstanceOf(Number.class, value, "Argument must be a Number.");
-		return NumberUtils.convertNumberToTargetClass((Number) value, Double.class).doubleValue();
-	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MappingMongoConverter.java

@@ -1,11 +1,11 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *      http://www.apache.org/licenses/LICENSE-2.0
+ *     http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -20,7 +20,6 @@ import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.HashSet;
-import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
@@ -30,10 +29,10 @@ import org.slf4j.LoggerFactory;
 import org.springframework.beans.BeansException;
 import org.springframework.context.ApplicationContext;
 import org.springframework.context.ApplicationContextAware;
-import org.springframework.core.CollectionFactory;
 import org.springframework.core.convert.ConversionException;
 import org.springframework.core.convert.ConversionService;
 import org.springframework.core.convert.support.DefaultConversionService;
+import org.springframework.data.convert.CollectionFactory;
 import org.springframework.data.convert.EntityInstantiator;
 import org.springframework.data.convert.TypeMapper;
 import org.springframework.data.mapping.Association;
@@ -96,7 +95,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 	/**
 	 * Creates a new {@link MappingMongoConverter} given the new {@link DbRefResolver} and {@link MappingContext}.
 	 * 
-	 * @param dbRefResolver must not be {@literal null}.
+	 * @param mongoDbFactory must not be {@literal null}.
 	 * @param mappingContext must not be {@literal null}.
 	 */
 	public MappingMongoConverter(DbRefResolver dbRefResolver,
@@ -259,14 +258,13 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 
 		// make sure id property is set before all other properties
 		Object idValue = null;
-		final DBObjectAccessor dbObjectAccessor = new DBObjectAccessor(dbo);
 
-		if (idProperty != null && dbObjectAccessor.hasValue(idProperty)) {
+		if (idProperty != null) {
 			idValue = getValueInternal(idProperty, dbo, evaluator, path);
 			accessor.setProperty(idProperty, idValue);
 		}
 
-		final ObjectPath currentPath = path.push(result, entity, idValue != null ? dbObjectAccessor.get(idProperty) : null);
+		final ObjectPath currentPath = path.push(result, entity, idValue);
 
 		// Set properties not already set in the constructor
 		entity.doWithProperties(new PropertyHandler<MongoPersistentProperty>() {
@@ -277,7 +275,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 					return;
 				}
 
-				if (entity.isConstructorArgument(prop) || !dbObjectAccessor.hasValue(prop)) {
+				if (!dbo.containsField(prop.getFieldName()) || entity.isConstructorArgument(prop)) {
 					return;
 				}
 
@@ -290,9 +288,9 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 			public void doWithAssociation(Association<MongoPersistentProperty> association) {
 
 				final MongoPersistentProperty property = association.getInverse();
-				Object value = dbObjectAccessor.get(property);
+				Object value = dbo.get(property.getFieldName());
 
-				if (value == null || entity.isConstructorArgument(property)) {
+				if (value == null) {
 					return;
 				}
 
@@ -310,16 +308,16 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 		return result;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see org.springframework.data.mongodb.core.convert.MongoWriter#toDBRef(java.lang.Object, org.springframework.data.mongodb.core.mapping.MongoPersistentProperty)
 	 */
-	public DBRef toDBRef(Object object, MongoPersistentProperty referringProperty) {
+	public DBRef toDBRef(Object object, MongoPersistentProperty referingProperty) {
 
 		org.springframework.data.mongodb.core.mapping.DBRef annotation = null;
 
-		if (referringProperty != null) {
-			annotation = referringProperty.getDBRef();
+		if (referingProperty != null) {
+			annotation = referingProperty.getDBRef();
 			Assert.isTrue(annotation != null, "The referenced property has to be mapped with @DBRef!");
 		}
 
@@ -328,14 +326,14 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 			return ((LazyLoadingProxy) object).toDBRef();
 		}
 
-		return createDBRef(object, referringProperty);
+		return createDBRef(object, referingProperty);
 	}
 
 	/**
 	 * Root entry method into write conversion. Adds a type discriminator to the {@link DBObject}. Shouldn't be called for
 	 * nested conversions.
 	 * 
-	 * @see org.springframework.data.mongodb.core.convert.MongoWriter#write(java.lang.Object, com.mongodb.DBObject)
+	 * @see org.springframework.data.mongodb.core.core.convert.MongoWriter#write(java.lang.Object, com.mongodb.DBObject)
 	 */
 	public void write(final Object obj, final DBObject dbo) {
 
@@ -478,7 +476,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 			DBRef dbRefObj = null;
 
 			/*
-			 * If we already have a LazyLoadingProxy, we use it's cached DBRef value instead of
+			 * If we already have a LazyLoadingProxy, we use it's cached DBRef value instead of 
 			 * unnecessarily initializing it only to convert it to a DBRef a few instructions later.
 			 */
 			if (obj instanceof LazyLoadingProxy) {
@@ -825,7 +823,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 
 	protected DBRef createDBRef(Object target, MongoPersistentProperty property) {
 
-		Assert.notNull(target, "Target object must not be null!");
+		Assert.notNull(target);
 
 		if (target instanceof DBRef) {
 			return (DBRef) target;
@@ -886,6 +884,10 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 
 		Class<?> collectionType = targetType.getType();
 
+		if (sourceValue.isEmpty()) {
+			return getPotentiallyConvertedSimpleRead(new HashSet<Object>(), collectionType);
+		}
+
 		TypeInformation<?> componentType = targetType.getComponentType();
 		Class<?> rawComponentType = componentType == null ? null : componentType.getType();
 
@@ -893,11 +895,9 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 		Collection<Object> items = targetType.getType().isArray() ? new ArrayList<Object>()
 				: CollectionFactory.createCollection(collectionType, rawComponentType, sourceValue.size());
 
-		if (sourceValue.isEmpty()) {
-			return getPotentiallyConvertedSimpleRead(items, collectionType);
-		}
+		for (int i = 0; i < sourceValue.size(); i++) {
 
-		for (Object dbObjItem : sourceValue) {
+			Object dbObjItem = sourceValue.get(i);
 
 			if (dbObjItem instanceof DBRef) {
 				items.add(
@@ -991,32 +991,20 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 		}
 
 		if (obj instanceof DBObject) {
-
 			DBObject newValueDbo = new BasicDBObject();
-
 			for (String vk : ((DBObject) obj).keySet()) {
-
 				Object o = ((DBObject) obj).get(vk);
 				newValueDbo.put(vk, convertToMongoType(o, typeHint));
 			}
-
 			return newValueDbo;
 		}
 
 		if (obj instanceof Map) {
-
-			Map<Object, Object> converted = new LinkedHashMap<Object, Object>();
-
-			for (Entry<Object, Object> entry : ((Map<Object, Object>) obj).entrySet()) {
-
-				TypeInformation<? extends Object> valueTypeHint = typeHint != null && typeHint.getMapValueType() != null
-						? typeHint.getMapValueType() : typeHint;
-
-				converted.put(getPotentiallyConvertedSimpleWrite(entry.getKey()).toString(),
-						convertToMongoType(entry.getValue(), valueTypeHint));
+			DBObject result = new BasicDBObject();
+			for (Map.Entry<Object, Object> entry : ((Map<Object, Object>) obj).entrySet()) {
+				result.put(entry.getKey().toString(), convertToMongoType(entry.getValue(), typeHint));
 			}
-
-			return new BasicDBObject(converted);
+			return result;
 		}
 
 		if (obj.getClass().isArray()) {
@@ -1053,7 +1041,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 
 	/**
 	 * Removes the type information from the entire conversion result.
-	 *
+	 * 
 	 * @param object
 	 * @param recursively whether to apply the removal recursively
 	 * @return
@@ -1114,22 +1102,22 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 		/**
 		 * Creates a new {@link MongoDbPropertyValueProvider} for the given source, {@link SpELExpressionEvaluator} and
 		 * {@link ObjectPath}.
-		 *
+		 * 
 		 * @param source must not be {@literal null}.
 		 * @param evaluator must not be {@literal null}.
 		 * @param path can be {@literal null}.
 		 */
 		public MongoDbPropertyValueProvider(DBObject source, SpELExpressionEvaluator evaluator, ObjectPath path) {
 
-			Assert.notNull(source, "DBObject must not be null!");
-			Assert.notNull(evaluator, "SpELExpressionEvaluator must not be null!");
+			Assert.notNull(source);
+			Assert.notNull(evaluator);
 
 			this.source = new DBObjectAccessor(source);
 			this.evaluator = evaluator;
 			this.path = path;
 		}
 
-		/*
+		/* 
 		 * (non-Javadoc)
 		 * @see org.springframework.data.convert.PropertyValueProvider#getPropertyValue(org.springframework.data.mapping.PersistentProperty)
 		 */
@@ -1172,7 +1160,7 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 			this.path = path;
 		}
 
-		/*
+		/* 
 		 * (non-Javadoc)
 		 * @see org.springframework.data.mapping.model.SpELExpressionParameterValueProvider#potentiallyConvertSpelValue(java.lang.Object, org.springframework.data.mapping.PreferredConstructor.Parameter)
 		 */
@@ -1209,6 +1197,10 @@ public class MappingMongoConverter extends AbstractMongoConverter implements App
 
 		Object object = dbref == null ? null : path.getPathItem(dbref.getId(), dbref.getCollectionName());
 
+		if (object != null) {
+			return (T) object;
+		}
+
 		return (T) (object != null ? object : read(type, readRef(dbref), path));
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoConverters.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,26 +19,16 @@ import java.math.BigDecimal;
 import java.math.BigInteger;
 import java.net.MalformedURLException;
 import java.net.URL;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.Currency;
-import java.util.List;
-import java.util.concurrent.atomic.AtomicInteger;
-import java.util.concurrent.atomic.AtomicLong;
 
 import org.bson.types.Code;
 import org.bson.types.ObjectId;
 import org.springframework.core.convert.ConversionFailedException;
 import org.springframework.core.convert.TypeDescriptor;
-import org.springframework.core.convert.converter.ConditionalConverter;
 import org.springframework.core.convert.converter.Converter;
-import org.springframework.core.convert.converter.ConverterFactory;
 import org.springframework.data.convert.ReadingConverter;
 import org.springframework.data.convert.WritingConverter;
 import org.springframework.data.mongodb.core.query.Term;
 import org.springframework.data.mongodb.core.script.NamedMongoScript;
-import org.springframework.util.Assert;
-import org.springframework.util.NumberUtils;
 import org.springframework.util.StringUtils;
 
 import com.mongodb.BasicDBObject;
@@ -59,36 +49,6 @@ abstract class MongoConverters {
 	 */
 	private MongoConverters() {}
 
-	/**
-	 * Returns the converters to be registered.
-	 * 
-	 * @return
-	 * @since 1.9
-	 */
-	public static Collection<Object> getConvertersToRegister() {
-
-		List<Object> converters = new ArrayList<Object>();
-
-		converters.add(BigDecimalToStringConverter.INSTANCE);
-		converters.add(StringToBigDecimalConverter.INSTANCE);
-		converters.add(BigIntegerToStringConverter.INSTANCE);
-		converters.add(StringToBigIntegerConverter.INSTANCE);
-		converters.add(URLToStringConverter.INSTANCE);
-		converters.add(StringToURLConverter.INSTANCE);
-		converters.add(DBObjectToStringConverter.INSTANCE);
-		converters.add(TermToStringConverter.INSTANCE);
-		converters.add(NamedMongoScriptToDBObjectConverter.INSTANCE);
-		converters.add(DBObjectToNamedMongoScriptCoverter.INSTANCE);
-		converters.add(CurrencyToStringConverter.INSTANCE);
-		converters.add(StringToCurrencyConverter.INSTANCE);
-		converters.add(AtomicIntegerToIntegerConverter.INSTANCE);
-		converters.add(AtomicLongToLongConverter.INSTANCE);
-		converters.add(LongToAtomicLongConverter.INSTANCE);
-		converters.add(IntegerToAtomicIntegerConverter.INSTANCE);
-
-		return converters;
-	}
-
 	/**
 	 * Simple singleton to convert {@link ObjectId}s to their {@link String} representation.
 	 * 
@@ -268,177 +228,4 @@ abstract class MongoConverters {
 			return builder.get();
 		}
 	}
-
-	/**
-	 * {@link Converter} implementation converting {@link Currency} into its ISO 4217 {@link String} representation.
-	 * 
-	 * @author Christoph Strobl
-	 * @since 1.9
-	 */
-	@WritingConverter
-	public static enum CurrencyToStringConverter implements Converter<Currency, String> {
-
-		INSTANCE;
-
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object)
-		 */
-		@Override
-		public String convert(Currency source) {
-			return source == null ? null : source.getCurrencyCode();
-		}
-	}
-
-	/**
-	 * {@link Converter} implementation converting ISO 4217 {@link String} into {@link Currency}.
-	 * 
-	 * @author Christoph Strobl
-	 * @since 1.9
-	 */
-	@ReadingConverter
-	public static enum StringToCurrencyConverter implements Converter<String, Currency> {
-
-		INSTANCE;
-
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object)
-		 */
-		@Override
-		public Currency convert(String source) {
-			return StringUtils.hasText(source) ? Currency.getInstance(source) : null;
-		}
-	}
-
-	/**
-	 * {@link ConverterFactory} implementation using {@link NumberUtils} for number conversion and parsing. Additionally
-	 * deals with {@link AtomicInteger} and {@link AtomicLong} by calling {@code get()} before performing the actual
-	 * conversion.
-	 * 
-	 * @author Christoph Strobl
-	 * @since 1.9
-	 */
-	@WritingConverter
-	public static enum NumberToNumberConverterFactory implements ConverterFactory<Number, Number>,ConditionalConverter {
-
-		INSTANCE;
-
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.ConverterFactory#getConverter(java.lang.Class)
-		 */
-		@Override
-		public <T extends Number> Converter<Number, T> getConverter(Class<T> targetType) {
-			return new NumberToNumberConverter<T>(targetType);
-		}
-
-		/*
-		 * (non-Javadoc)
-		 * @see org.springframework.core.convert.converter.ConditionalConverter#matches(org.springframework.core.convert.TypeDescriptor, org.springframework.core.convert.TypeDescriptor)
-		 */
-		@Override
-		public boolean matches(TypeDescriptor sourceType, TypeDescriptor targetType) {
-			return !sourceType.equals(targetType);
-		}
-
-		private final static class NumberToNumberConverter<T extends Number> implements Converter<Number, T> {
-
-			private final Class<T> targetType;
-
-			/**
-			 * Creates a new {@link NumberToNumberConverter} for the given target type.
-			 * 
-			 * @param targetType must not be {@literal null}.
-			 */
-			public NumberToNumberConverter(Class<T> targetType) {
-				
-				Assert.notNull(targetType, "Target type must not be null!");
-				
-				this.targetType = targetType;
-			}
-
-			/*
-			 * (non-Javadoc)
-			 * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object)
-			 */
-			@Override
-			public T convert(Number source) {
-
-				if (source instanceof AtomicInteger) {
-					return NumberUtils.convertNumberToTargetClass(((AtomicInteger) source).get(), this.targetType);
-				}
-
-				if (source instanceof AtomicLong) {
-					return NumberUtils.convertNumberToTargetClass(((AtomicLong) source).get(), this.targetType);
-				}
-
-				return NumberUtils.convertNumberToTargetClass(source, this.targetType);
-			}
-		}
-	}
-
-	/**
-	 * {@link ConverterFactory} implementation converting {@link AtomicLong} into {@link Long}.
-	 *
-	 * @author Christoph Strobl
-	 * @since 1.10
-	 */
-	@WritingConverter
-	public static enum AtomicLongToLongConverter implements Converter<AtomicLong, Long> {
-		INSTANCE;
-
-		@Override
-		public Long convert(AtomicLong source) {
-			return NumberUtils.convertNumberToTargetClass(source, Long.class);
-		}
-	}
-
-	/**
-	 * {@link ConverterFactory} implementation converting {@link AtomicInteger} into {@link Integer}.
-	 *
-	 * @author Christoph Strobl
-	 * @since 1.10
-	 */
-	@WritingConverter
-	public static enum AtomicIntegerToIntegerConverter implements Converter<AtomicInteger, Integer> {
-		INSTANCE;
-
-		@Override
-		public Integer convert(AtomicInteger source) {
-			return NumberUtils.convertNumberToTargetClass(source, Integer.class);
-		}
-	}
-
-	/**
-	 * {@link ConverterFactory} implementation converting {@link Long} into {@link AtomicLong}.
-	 *
-	 * @author Christoph Strobl
-	 * @since 1.10
-	 */
-	@ReadingConverter
-	public static enum LongToAtomicLongConverter implements Converter<Long, AtomicLong> {
-		INSTANCE;
-
-		@Override
-		public AtomicLong convert(Long source) {
-			return source != null ? new AtomicLong(source) : null;
-		}
-	}
-
-	/**
-	 * {@link ConverterFactory} implementation converting {@link Integer} into {@link AtomicInteger}.
-	 *
-	 * @author Christoph Strobl
-	 * @since 1.10
-	 */
-	@ReadingConverter
-	public static enum IntegerToAtomicIntegerConverter implements Converter<Integer, AtomicInteger> {
-		INSTANCE;
-
-		@Override
-		public AtomicInteger convert(Integer source) {
-			return source != null ? new AtomicInteger(source) : null;
-		}
-	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/MongoExampleMapper.java

@@ -1,262 +0,0 @@
-/*
- * Copyright 2015-2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core.convert;
-
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Set;
-import java.util.Stack;
-import java.util.regex.Pattern;
-
-import org.springframework.data.domain.Example;
-import org.springframework.data.domain.ExampleMatcher.NullHandler;
-import org.springframework.data.domain.ExampleMatcher.PropertyValueTransformer;
-import org.springframework.data.domain.ExampleMatcher.StringMatcher;
-import org.springframework.data.mapping.PropertyHandler;
-import org.springframework.data.mapping.context.MappingContext;
-import org.springframework.data.mongodb.core.mapping.MongoPersistentEntity;
-import org.springframework.data.mongodb.core.mapping.MongoPersistentProperty;
-import org.springframework.data.mongodb.core.query.MongoRegexCreator;
-import org.springframework.data.mongodb.core.query.SerializationUtils;
-import org.springframework.data.repository.core.support.ExampleMatcherAccessor;
-import org.springframework.data.repository.query.parser.Part.Type;
-import org.springframework.data.util.TypeInformation;
-import org.springframework.util.Assert;
-import org.springframework.util.ObjectUtils;
-import org.springframework.util.StringUtils;
-
-import com.mongodb.BasicDBObject;
-import com.mongodb.DBObject;
-
-/**
- * @author Christoph Strobl
- * @author Mark Paluch
- * @since 1.8
- */
-public class MongoExampleMapper {
-
-	private final MappingContext<? extends MongoPersistentEntity<?>, MongoPersistentProperty> mappingContext;
-	private final MongoConverter converter;
-	private final Map<StringMatcher, Type> stringMatcherPartMapping = new HashMap<StringMatcher, Type>();
-
-	public MongoExampleMapper(MongoConverter converter) {
-
-		this.converter = converter;
-		this.mappingContext = converter.getMappingContext();
-
-		stringMatcherPartMapping.put(StringMatcher.EXACT, Type.SIMPLE_PROPERTY);
-		stringMatcherPartMapping.put(StringMatcher.CONTAINING, Type.CONTAINING);
-		stringMatcherPartMapping.put(StringMatcher.STARTING, Type.STARTING_WITH);
-		stringMatcherPartMapping.put(StringMatcher.ENDING, Type.ENDING_WITH);
-		stringMatcherPartMapping.put(StringMatcher.REGEX, Type.REGEX);
-	}
-
-	/**
-	 * Returns the given {@link Example} as {@link DBObject} holding matching values extracted from
-	 * {@link Example#getProbe()}.
-	 *
-	 * @param example must not be {@literal null}.
-	 * @return
-	 */
-	public DBObject getMappedExample(Example<?> example) {
-
-		Assert.notNull(example, "Example must not be null!");
-
-		return getMappedExample(example, mappingContext.getPersistentEntity(example.getProbeType()));
-	}
-
-	/**
-	 * Returns the given {@link Example} as {@link DBObject} holding matching values extracted from
-	 * {@link Example#getProbe()}.
-	 *
-	 * @param example must not be {@literal null}.
-	 * @param entity must not be {@literal null}.
-	 * @return
-	 */
-	@SuppressWarnings({ "unchecked", "rawtypes" })
-	public DBObject getMappedExample(Example<?> example, MongoPersistentEntity<?> entity) {
-
-		Assert.notNull(example, "Example must not be null!");
-		Assert.notNull(entity, "MongoPersistentEntity must not be null!");
-
-		DBObject reference = (DBObject) converter.convertToMongoType(example.getProbe());
-
-		if (entity.hasIdProperty() && entity.getIdentifierAccessor(example.getProbe()).getIdentifier() == null) {
-			reference.removeField(entity.getIdProperty().getFieldName());
-		}
-
-		ExampleMatcherAccessor matcherAccessor = new ExampleMatcherAccessor(example.getMatcher());
-
-		applyPropertySpecs("", reference, example.getProbeType(), matcherAccessor);
-
-		this.converter.getTypeMapper().writeTypeRestrictions(reference, getTypesToMatch(example));
-
-		return ObjectUtils.nullSafeEquals(NullHandler.INCLUDE, matcherAccessor.getNullHandler()) ? reference
-				: new BasicDBObject(SerializationUtils.flattenMap(reference));
-	}
-
-	private Set<Class<?>> getTypesToMatch(Example<?> example) {
-
-		Set<Class<?>> types = new HashSet<Class<?>>();
-
-		for (TypeInformation<?> reference : mappingContext.getManagedTypes()) {
-			if (example.getProbeType().isAssignableFrom(reference.getType())) {
-				types.add(reference.getType());
-			}
-		}
-
-		return types;
-	}
-
-	private String getMappedPropertyPath(String path, Class<?> probeType) {
-
-		MongoPersistentEntity<?> entity = mappingContext.getPersistentEntity(probeType);
-
-		Iterator<String> parts = Arrays.asList(path.split("\\.")).iterator();
-
-		final Stack<MongoPersistentProperty> stack = new Stack<MongoPersistentProperty>();
-
-		List<String> resultParts = new ArrayList<String>();
-
-		while (parts.hasNext()) {
-
-			final String part = parts.next();
-			MongoPersistentProperty prop = entity.getPersistentProperty(part);
-
-			if (prop == null) {
-
-				entity.doWithProperties(new PropertyHandler<MongoPersistentProperty>() {
-
-					@Override
-					public void doWithPersistentProperty(MongoPersistentProperty property) {
-
-						if (property.getFieldName().equals(part)) {
-							stack.push(property);
-						}
-					}
-				});
-
-				if (stack.isEmpty()) {
-					return "";
-				}
-				prop = stack.pop();
-			}
-
-			resultParts.add(prop.getName());
-
-			if (prop.isEntity() && mappingContext.hasPersistentEntityFor(prop.getActualType())) {
-				entity = mappingContext.getPersistentEntity(prop.getActualType());
-			} else {
-				break;
-			}
-		}
-
-		return StringUtils.collectionToDelimitedString(resultParts, ".");
-
-	}
-
-	private void applyPropertySpecs(String path, DBObject source, Class<?> probeType,
-			ExampleMatcherAccessor exampleSpecAccessor) {
-
-		if (!(source instanceof BasicDBObject)) {
-			return;
-		}
-
-		Iterator<Map.Entry<String, Object>> iter = ((BasicDBObject) source).entrySet().iterator();
-
-		while (iter.hasNext()) {
-
-			Map.Entry<String, Object> entry = iter.next();
-			String propertyPath = StringUtils.hasText(path) ? path + "." + entry.getKey() : entry.getKey();
-			String mappedPropertyPath = getMappedPropertyPath(propertyPath, probeType);
-
-			if (isEmptyIdProperty(entry)) {
-				iter.remove();
-				continue;
-			}
-
-			if (exampleSpecAccessor.isIgnoredPath(propertyPath) || exampleSpecAccessor.isIgnoredPath(mappedPropertyPath)) {
-				iter.remove();
-				continue;
-			}
-
-			StringMatcher stringMatcher = exampleSpecAccessor.getDefaultStringMatcher();
-			Object value = entry.getValue();
-			boolean ignoreCase = exampleSpecAccessor.isIgnoreCaseEnabled();
-
-			if (exampleSpecAccessor.hasPropertySpecifiers()) {
-
-				mappedPropertyPath = exampleSpecAccessor.hasPropertySpecifier(propertyPath) ? propertyPath
-						: getMappedPropertyPath(propertyPath, probeType);
-
-				stringMatcher = exampleSpecAccessor.getStringMatcherForPath(mappedPropertyPath);
-				ignoreCase = exampleSpecAccessor.isIgnoreCaseForPath(mappedPropertyPath);
-			}
-
-			// TODO: should a PropertySpecifier outrule the later on string matching?
-			if (exampleSpecAccessor.hasPropertySpecifier(mappedPropertyPath)) {
-
-				PropertyValueTransformer valueTransformer = exampleSpecAccessor.getValueTransformerForPath(mappedPropertyPath);
-				value = valueTransformer.convert(value);
-				if (value == null) {
-					iter.remove();
-					continue;
-				}
-
-				entry.setValue(value);
-			}
-
-			if (entry.getValue() instanceof String) {
-				applyStringMatcher(entry, stringMatcher, ignoreCase);
-			} else if (entry.getValue() instanceof BasicDBObject) {
-				applyPropertySpecs(propertyPath, (BasicDBObject) entry.getValue(), probeType, exampleSpecAccessor);
-			}
-		}
-	}
-
-	private boolean isEmptyIdProperty(Entry<String, Object> entry) {
-		return entry.getKey().equals("_id") && entry.getValue() == null;
-	}
-
-	private void applyStringMatcher(Map.Entry<String, Object> entry, StringMatcher stringMatcher, boolean ignoreCase) {
-
-		BasicDBObject dbo = new BasicDBObject();
-
-		if (ObjectUtils.nullSafeEquals(StringMatcher.DEFAULT, stringMatcher)) {
-
-			if (ignoreCase) {
-				dbo.put("$regex", Pattern.quote((String) entry.getValue()));
-				entry.setValue(dbo);
-			}
-		} else {
-
-			Type type = stringMatcherPartMapping.get(stringMatcher);
-			String expression = MongoRegexCreator.INSTANCE.toRegularExpression((String) entry.getValue(), type);
-			dbo.put("$regex", expression);
-			entry.setValue(dbo);
-		}
-
-		if (ignoreCase) {
-			dbo.put("$options", "i");
-		}
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/QueryMapper.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,12 +27,10 @@ import org.bson.types.ObjectId;
 import org.springframework.core.convert.ConversionException;
 import org.springframework.core.convert.ConversionService;
 import org.springframework.core.convert.converter.Converter;
-import org.springframework.data.domain.Example;
 import org.springframework.data.mapping.Association;
 import org.springframework.data.mapping.PersistentEntity;
 import org.springframework.data.mapping.PropertyPath;
 import org.springframework.data.mapping.PropertyReferenceException;
-import org.springframework.data.mapping.context.InvalidPersistentPropertyPath;
 import org.springframework.data.mapping.context.MappingContext;
 import org.springframework.data.mapping.context.PersistentPropertyPath;
 import org.springframework.data.mapping.model.MappingException;
@@ -58,7 +56,6 @@ import com.mongodb.DBRef;
  * @author Patryk Wasik
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class QueryMapper {
 
@@ -67,13 +64,12 @@ public class QueryMapper {
 	static final ClassTypeInformation<?> NESTED_DOCUMENT = ClassTypeInformation.from(NestedDocument.class);
 
 	private enum MetaMapping {
-		FORCE, WHEN_PRESENT, IGNORE
+		FORCE, WHEN_PRESENT, IGNORE;
 	}
 
 	private final ConversionService conversionService;
 	private final MongoConverter converter;
 	private final MappingContext<? extends MongoPersistentEntity<?>, MongoPersistentProperty> mappingContext;
-	private final MongoExampleMapper exampleMapper;
 
 	/**
 	 * Creates a new {@link QueryMapper} with the given {@link MongoConverter}.
@@ -82,12 +78,11 @@ public class QueryMapper {
 	 */
 	public QueryMapper(MongoConverter converter) {
 
-		Assert.notNull(converter, "MongoConverter must not be null!");
+		Assert.notNull(converter);
 
 		this.conversionService = converter.getConversionService();
 		this.converter = converter;
 		this.mappingContext = converter.getMappingContext();
-		this.exampleMapper = new MongoExampleMapper(converter);
 	}
 
 	/**
@@ -124,20 +119,10 @@ public class QueryMapper {
 				continue;
 			}
 
-			try {
+			Field field = createPropertyField(entity, key, mappingContext);
+			Entry<String, Object> entry = getMappedObjectForField(field, query.get(key));
 
-				Field field = createPropertyField(entity, key, mappingContext);
-				Entry<String, Object> entry = getMappedObjectForField(field, query.get(key));
-				result.put(entry.getKey(), entry.getValue());
-			} catch (InvalidPersistentPropertyPath invalidPathException) {
-
-				// in case the object has not already been mapped
-				if (!(query.get(key) instanceof DBObject)) {
-					throw invalidPathException;
-				}
-
-				result.put(key, query.get(key));
-			}
+			result.put(entry.getKey(), entry.getValue());
 		}
 
 		return result;
@@ -241,7 +226,7 @@ public class QueryMapper {
 	protected DBObject getMappedKeyword(Keyword keyword, MongoPersistentEntity<?> entity) {
 
 		// $or/$nor
-		if (keyword.isOrOrNor() || (keyword.hasIterableValue() && !keyword.isGeometry())) {
+		if (keyword.isOrOrNor() || keyword.hasIterableValue()) {
 
 			Iterable<?> conditions = keyword.getValue();
 			BasicDBList newConditions = new BasicDBList();
@@ -254,10 +239,6 @@ public class QueryMapper {
 			return new BasicDBObject(keyword.getKey(), newConditions);
 		}
 
-		if (keyword.isSample()) {
-			return exampleMapper.getMappedExample(keyword.<Example<?>> getValue(), entity);
-		}
-
 		return new BasicDBObject(keyword.getKey(), convertSimpleOrDBObject(keyword.getValue(), entity));
 	}
 
@@ -317,7 +298,7 @@ public class QueryMapper {
 		}
 
 		if (isNestedKeyword(value)) {
-			return getMappedKeyword(new Keyword((DBObject) value), documentField.getPropertyEntity());
+			return getMappedKeyword(new Keyword((DBObject) value), null);
 		}
 
 		if (isAssociationConversionNecessary(documentField, value)) {
@@ -575,26 +556,6 @@ public class QueryMapper {
 			return key.matches(N_OR_PATTERN);
 		}
 
-		/**
-		 * Returns whether the current keyword is the {@code $geometry} keyword.
-		 * 
-		 * @return
-		 * @since 1.8
-		 */
-		public boolean isGeometry() {
-			return "$geometry".equalsIgnoreCase(key);
-		}
-
-		/**
-		 * Returns wheter the current keyword indicates a sample object.
-		 * 
-		 * @return
-		 * @since 1.8
-		 */
-		public boolean isSample() {
-			return "$sample".equalsIgnoreCase(key);
-		}
-
 		public boolean hasIterableValue() {
 			return value instanceof Iterable;
 		}
@@ -896,7 +857,7 @@ public class QueryMapper {
 		 * @return
 		 */
 		protected Converter<MongoPersistentProperty, String> getPropertyConverter() {
-			return new PositionParameterRetainingPropertyKeyConverter(name);
+			return PropertyToFieldNameConverter.INSTANCE;
 		}
 
 		/**
@@ -910,28 +871,6 @@ public class QueryMapper {
 			return new AssociationConverter(getAssociation());
 		}
 
-		/**
-		 * @author Christoph Strobl
-		 * @since 1.8
-		 */
-		static class PositionParameterRetainingPropertyKeyConverter implements Converter<MongoPersistentProperty, String> {
-
-			private final KeyMapper keyMapper;
-
-			public PositionParameterRetainingPropertyKeyConverter(String rawKey) {
-				this.keyMapper = new KeyMapper(rawKey);
-			}
-
-			/*
-			 * (non-Javadoc)
-			 * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object)
-			 */
-			@Override
-			public String convert(MongoPersistentProperty source) {
-				return keyMapper.mapPropertyName(source);
-			}
-		}
-
 		/* 
 		 * (non-Javadoc)
 		 * @see org.springframework.data.mongodb.core.convert.QueryMapper.Field#getTypeHint()
@@ -952,61 +891,6 @@ public class QueryMapper {
 
 			return NESTED_DOCUMENT;
 		}
-
-		/**
-		 * @author Christoph Strobl
-		 * @since 1.8
-		 */
-		static class KeyMapper {
-
-			private final Iterator<String> iterator;
-
-			public KeyMapper(String key) {
-
-				this.iterator = Arrays.asList(key.split("\\.")).iterator();
-				this.iterator.next();
-			}
-
-			/**
-			 * Maps the property name while retaining potential positional operator {@literal $}.
-			 * 
-			 * @param property
-			 * @return
-			 */
-			protected String mapPropertyName(MongoPersistentProperty property) {
-
-				StringBuilder mappedName = new StringBuilder(PropertyToFieldNameConverter.INSTANCE.convert(property));
-				boolean inspect = iterator.hasNext();
-
-				while (inspect) {
-
-					String partial = iterator.next();
-					boolean isPositional = (isPositionalParameter(partial) && (property.isMap() || property.isCollectionLike()));
-
-					if (isPositional) {
-						mappedName.append(".").append(partial);
-					}
-
-					inspect = isPositional && iterator.hasNext();
-				}
-
-				return mappedName.toString();
-			}
-
-			private static boolean isPositionalParameter(String partial) {
-
-				if ("$".equals(partial)) {
-					return true;
-				}
-
-				try {
-					Long.valueOf(partial);
-					return true;
-				} catch (NumberFormatException e) {
-					return false;
-				}
-			}
-		}
 	}
 
 	/**

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/convert/UpdateMapper.java

@@ -15,6 +15,8 @@
  */
 package org.springframework.data.mongodb.core.convert;
 
+import java.util.Arrays;
+import java.util.Iterator;
 import java.util.Map.Entry;
 
 import org.springframework.core.convert.converter.Converter;
@@ -22,11 +24,13 @@ import org.springframework.data.mapping.Association;
 import org.springframework.data.mapping.context.MappingContext;
 import org.springframework.data.mongodb.core.mapping.MongoPersistentEntity;
 import org.springframework.data.mongodb.core.mapping.MongoPersistentProperty;
+import org.springframework.data.mongodb.core.mapping.MongoPersistentProperty.PropertyToFieldNameConverter;
 import org.springframework.data.mongodb.core.query.Query;
 import org.springframework.data.mongodb.core.query.Update.Modifier;
 import org.springframework.data.mongodb.core.query.Update.Modifiers;
 import org.springframework.data.util.ClassTypeInformation;
 import org.springframework.data.util.TypeInformation;
+import org.springframework.util.Assert;
 
 import com.mongodb.BasicDBObject;
 import com.mongodb.DBObject;
@@ -209,7 +213,7 @@ public class UpdateMapper extends QueryMapper {
 		 */
 		@Override
 		protected Converter<MongoPersistentProperty, String> getPropertyConverter() {
-			return new PositionParameterRetainingPropertyKeyConverter(key);
+			return new UpdatePropertyConverter(key);
 		}
 
 		/*
@@ -221,6 +225,99 @@ public class UpdateMapper extends QueryMapper {
 			return new UpdateAssociationConverter(getAssociation(), key);
 		}
 
+		/**
+		 * Special mapper handling positional parameter {@literal $} within property names.
+		 * 
+		 * @author Christoph Strobl
+		 * @since 1.7
+		 */
+		private static class UpdateKeyMapper {
+
+			private final Iterator<String> iterator;
+
+			protected UpdateKeyMapper(String rawKey) {
+
+				Assert.hasText(rawKey, "Key must not be null or empty!");
+
+				this.iterator = Arrays.asList(rawKey.split("\\.")).iterator();
+				this.iterator.next();
+			}
+
+			/**
+			 * Maps the property name while retaining potential positional operator {@literal $}.
+			 * 
+			 * @param property
+			 * @return
+			 */
+			protected String mapPropertyName(MongoPersistentProperty property) {
+
+				String mappedName = PropertyToFieldNameConverter.INSTANCE.convert(property);
+
+				boolean inspect = iterator.hasNext();
+				while (inspect) {
+
+					String partial = iterator.next();
+
+					boolean isPositional = isPositionalParameter(partial);
+					if (isPositional) {
+						mappedName += "." + partial;
+					}
+
+					inspect = isPositional && iterator.hasNext();
+				}
+
+				return mappedName;
+			}
+
+			boolean isPositionalParameter(String partial) {
+
+				if (partial.equals("$")) {
+					return true;
+				}
+
+				try {
+					Long.valueOf(partial);
+					return true;
+				} catch (NumberFormatException e) {
+					return false;
+				}
+			}
+
+		}
+
+		/**
+		 * Special {@link Converter} for {@link MongoPersistentProperty} instances that will concatenate the {@literal $}
+		 * contained in the source update key.
+		 * 
+		 * @author Oliver Gierke
+		 * @author Christoph Strobl
+		 */
+		private static class UpdatePropertyConverter implements Converter<MongoPersistentProperty, String> {
+
+			private final UpdateKeyMapper mapper;
+
+			/**
+			 * Creates a new {@link UpdatePropertyConverter} with the given update key.
+			 * 
+			 * @param updateKey must not be {@literal null} or empty.
+			 */
+			public UpdatePropertyConverter(String updateKey) {
+
+				Assert.hasText(updateKey, "Update key must not be null or empty!");
+
+				this.mapper = new UpdateKeyMapper(updateKey);
+			}
+
+			/* 
+			 * (non-Javadoc)
+			 * @see org.springframework.core.convert.converter.Converter#convert(java.lang.Object)
+			 */
+			@Override
+			public String convert(MongoPersistentProperty property) {
+				return mapper.mapPropertyName(property);
+			}
+		}
+
 		/**
 		 * {@link Converter} retaining positional parameter {@literal $} for {@link Association}s.
 		 * 
@@ -228,7 +325,7 @@ public class UpdateMapper extends QueryMapper {
 		 */
 		protected static class UpdateAssociationConverter extends AssociationConverter {
 
-			private final KeyMapper mapper;
+			private final UpdateKeyMapper mapper;
 
 			/**
 			 * Creates a new {@link AssociationConverter} for the given {@link Association}.
@@ -238,7 +335,7 @@ public class UpdateMapper extends QueryMapper {
 			public UpdateAssociationConverter(Association<MongoPersistentProperty> association, String key) {
 
 				super(association);
-				this.mapper = new KeyMapper(key);
+				this.mapper = new UpdateKeyMapper(key);
 			}
 
 			/* 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/geo/Sphere.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2017 the original author or authors.
+ * Copyright 2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -29,7 +29,6 @@ import org.springframework.util.Assert;
  * Represents a geospatial sphere value.
  * 
  * @author Thomas Darimont
- * @author Mark Paluch
  * @since 1.5
  */
 public class Sphere implements Shape {
@@ -47,8 +46,8 @@ public class Sphere implements Shape {
 	@PersistenceConstructor
 	public Sphere(Point center, Distance radius) {
 
-		Assert.notNull(center, "Center point must not be null!");
-		Assert.notNull(radius, "Radius must not be null!");
+		Assert.notNull(center);
+		Assert.notNull(radius);
 		Assert.isTrue(radius.getValue() >= 0, "Radius must not be negative!");
 
 		this.center = center;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/GeoSpatialIndexed.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2016 the original author or authors.
+ * Copyright 2010-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,9 +27,8 @@ import java.lang.annotation.Target;
  * @author Laurent Canet
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
-@Target({ ElementType.FIELD, ElementType.ANNOTATION_TYPE })
+@Target(ElementType.FIELD)
 @Retention(RetentionPolicy.RUNTIME)
 public @interface GeoSpatialIndexed {
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/GeospatialIndex.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2016 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -39,7 +39,6 @@ public class GeospatialIndex implements IndexDefinition {
 	private GeoSpatialIndexType type = GeoSpatialIndexType.GEO_2D;
 	private Double bucketSize = 1.0;
 	private String additionalField;
-	private IndexFilter filter;
 
 	/**
 	 * Creates a new {@link GeospatialIndex} for the given field.
@@ -120,22 +119,6 @@ public class GeospatialIndex implements IndexDefinition {
 		return this;
 	}
 
-
-	/**
-	 * Only index the documents in a collection that meet a specified {@link IndexFilter filter expression}.
-	 *
-	 * @param filter can be {@literal null}.
-	 * @return
-	 * @see <a href=
-	 *      "https://docs.mongodb.com/manual/core/index-partial/">https://docs.mongodb.com/manual/core/index-partial/</a>
-	 * @since 1.10
-	 */
-	public GeospatialIndex partial(IndexFilter filter) {
-
-		this.filter = filter;
-		return this;
-	}
-
 	public DBObject getIndexKeys() {
 
 		DBObject dbo = new BasicDBObject();
@@ -203,10 +186,6 @@ public class GeospatialIndex implements IndexDefinition {
 				break;
 		}
 
-		if (filter != null) {
-			dbo.put("partialFilterExpression", filter.getFilterObject());
-		}
-
 		return dbo;
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/Index.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2016 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -17,7 +17,6 @@ package org.springframework.data.mongodb.core.index;
 
 import java.util.LinkedHashMap;
 import java.util.Map;
-import java.util.Map.Entry;
 import java.util.concurrent.TimeUnit;
 
 import org.springframework.data.domain.Sort.Direction;
@@ -45,7 +44,7 @@ public class Index implements IndexDefinition {
 		 * 
 		 * @deprecated since 1.7.
 		 */
-		@Deprecated //
+		@Deprecated//
 		DROP
 	}
 
@@ -63,8 +62,6 @@ public class Index implements IndexDefinition {
 
 	private long expire = -1;
 
-	private IndexFilter filter;
-
 	public Index() {}
 
 	public Index(String key, Direction direction) {
@@ -178,33 +175,11 @@ public class Index implements IndexDefinition {
 		return unique();
 	}
 
-	/**
-	 * Only index the documents in a collection that meet a specified {@link IndexFilter filter expression}.
-	 *
-	 * @param filter can be {@literal null}.
-	 * @return
-	 * @see <a href=
-	 *      "https://docs.mongodb.com/manual/core/index-partial/">https://docs.mongodb.com/manual/core/index-partial/</a>
-	 * @since 1.10
-	 */
-	public Index partial(IndexFilter filter) {
-
-		this.filter = filter;
-		return this;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.index.IndexDefinition#getIndexKeys()
-	 */
 	public DBObject getIndexKeys() {
-
 		DBObject dbo = new BasicDBObject();
-
-		for (Entry<String, Direction> entry : fieldSpec.entrySet()) {
-			dbo.put(entry.getKey(), Direction.ASC.equals(entry.getValue()) ? 1 : -1);
+		for (String k : fieldSpec.keySet()) {
+			dbo.put(k, fieldSpec.get(k).equals(Direction.ASC) ? 1 : -1);
 		}
-
 		return dbo;
 	}
 
@@ -230,9 +205,6 @@ public class Index implements IndexDefinition {
 			dbo.put("expireAfterSeconds", expire);
 		}
 
-		if (filter != null) {
-			dbo.put("partialFilterExpression", filter.getFilterObject());
-		}
 		return dbo;
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/IndexField.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2017 the original author or authors.
+ * Copyright 2012-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -44,13 +44,8 @@ public final class IndexField {
 
 	private IndexField(String key, Direction direction, Type type, Float weight) {
 
-		Assert.hasText(key, "Key must not be null or empty");
-
-		if (Type.GEO.equals(type) || Type.TEXT.equals(type)) {
-			Assert.isTrue(direction == null, "Geo/Text indexes must not have a direction!");
-		} else {
-			Assert.notNull(direction, "Default indexes require a direction");
-		}
+		Assert.hasText(key);
+		Assert.isTrue(direction != null ^ (Type.GEO.equals(type) || Type.TEXT.equals(type)));
 
 		this.key = key;
 		this.direction = direction;
@@ -63,21 +58,17 @@ public final class IndexField {
 	 * 
 	 * @deprecated use {@link #create(String, Direction)}.
 	 * @param key must not be {@literal null} or emtpy.
-	 * @param order must not be {@literal null}.
+	 * @param direction must not be {@literal null}.
 	 * @return
 	 */
 	@Deprecated
 	public static IndexField create(String key, Order order) {
-		
-		Assert.notNull(order, "Order must not be null!");
-
+		Assert.notNull(order);
 		return new IndexField(key, order.toDirection(), Type.DEFAULT);
 	}
 
 	public static IndexField create(String key, Direction order) {
-		
-		Assert.notNull(order, "Direction must not be null!");
-
+		Assert.notNull(order);
 		return new IndexField(key, order, Type.DEFAULT);
 	}
 
@@ -137,7 +128,7 @@ public final class IndexField {
 	}
 
 	/**
-	 * Returns whether the {@link IndexField} is a text index field.
+	 * Returns wheter the {@link IndexField} is a text index field.
 	 * 
 	 * @return true if type is {@link Type#TEXT}
 	 * @since 1.6
@@ -167,7 +158,7 @@ public final class IndexField {
 				&& this.type == that.type;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Object#hashCode()
 	 */
@@ -182,7 +173,7 @@ public final class IndexField {
 		return result;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Object#toString()
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/IndexFilter.java

@@ -1,35 +0,0 @@
-/*
- * Copyright 2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core.index;
-
-import com.mongodb.DBObject;
-
-/**
- * Use {@link IndexFilter} to create the partial filter expression used when creating
- * <a href="https://docs.mongodb.com/manual/core/index-partial/">Partial Indexes</a>.
- *
- * @author Christoph Strobl
- * @since 1.10
- */
-public interface IndexFilter {
-
-	/**
-	 * Get the raw (unmapped) filter expression.
-	 *
-	 * @return
-	 */
-	DBObject getFilterObject();
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/IndexInfo.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2017 the original author or authors.
+ * Copyright 2002-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,10 +15,7 @@
  */
 package org.springframework.data.mongodb.core.index;
 
-import static org.springframework.data.domain.Sort.Direction.*;
-
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
@@ -26,20 +23,13 @@ import java.util.List;
 import org.springframework.util.Assert;
 import org.springframework.util.ObjectUtils;
 
-import com.mongodb.DBObject;
-
 /**
  * @author Mark Pollack
  * @author Oliver Gierke
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class IndexInfo {
 
-	private static final Double ONE = Double.valueOf(1);
-	private static final Double MINUS_ONE = Double.valueOf(-1);
-	private static final Collection<String> TWO_D_IDENTIFIERS = Arrays.asList("2d", "2dsphere");
-
 	private final List<IndexField> indexFields;
 
 	private final String name;
@@ -47,7 +37,6 @@ public class IndexInfo {
 	private final boolean dropDuplicates;
 	private final boolean sparse;
 	private final String language;
-	private String partialFilterExpression;
 
 	/**
 	 * @deprecated Will be removed in 1.7. Please use {@link #IndexInfo(List, String, boolean, boolean, boolean, String)}
@@ -73,64 +62,6 @@ public class IndexInfo {
 		this.language = language;
 	}
 
-	/**
-	 * Creates new {@link IndexInfo} parsing required properties from the given {@literal sourceDocument}.
-	 *
-	 * @param sourceDocument
-	 * @return
-	 * @since 1.10
-	 */
-	public static IndexInfo indexInfoOf(DBObject sourceDocument) {
-
-		DBObject keyDbObject = (DBObject) sourceDocument.get("key");
-		int numberOfElements = keyDbObject.keySet().size();
-
-		List<IndexField> indexFields = new ArrayList<IndexField>(numberOfElements);
-
-		for (String key : keyDbObject.keySet()) {
-
-			Object value = keyDbObject.get(key);
-
-			if (TWO_D_IDENTIFIERS.contains(value)) {
-
-				indexFields.add(IndexField.geo(key));
-
-			} else if ("text".equals(value)) {
-
-				DBObject weights = (DBObject) sourceDocument.get("weights");
-
-				for (String fieldName : weights.keySet()) {
-					indexFields.add(IndexField.text(fieldName, Float.valueOf(weights.get(fieldName).toString())));
-				}
-
-			} else {
-
-				Double keyValue = new Double(value.toString());
-
-				if (ONE.equals(keyValue)) {
-					indexFields.add(IndexField.create(key, ASC));
-				} else if (MINUS_ONE.equals(keyValue)) {
-					indexFields.add(IndexField.create(key, DESC));
-				}
-			}
-		}
-
-		String name = sourceDocument.get("name").toString();
-
-		boolean unique = sourceDocument.containsField("unique") ? (Boolean) sourceDocument.get("unique") : false;
-		boolean dropDuplicates = sourceDocument.containsField("dropDups") ? (Boolean) sourceDocument.get("dropDups")
-				: false;
-		boolean sparse = sourceDocument.containsField("sparse") ? (Boolean) sourceDocument.get("sparse") : false;
-		String language = sourceDocument.containsField("default_language") ? (String) sourceDocument.get("default_language")
-				: "";
-		String partialFilter = sourceDocument.containsField("partialFilterExpression")
-				? sourceDocument.get("partialFilterExpression").toString() : "";
-
-		IndexInfo info = new IndexInfo(indexFields, name, unique, dropDuplicates, sparse, language);
-		info.partialFilterExpression = partialFilter;
-		return info;
-	}
-
 	/**
 	 * Returns the individual index fields of the index.
 	 * 
@@ -148,8 +79,7 @@ public class IndexInfo {
 	 */
 	public boolean isIndexForFields(Collection<String> keys) {
 
-		Assert.notNull(keys, "Collection of keys must not be null!");
-
+		Assert.notNull(keys);
 		List<String> indexKeys = new ArrayList<String>(indexFields.size());
 
 		for (IndexField field : indexFields) {
@@ -183,19 +113,10 @@ public class IndexInfo {
 		return language;
 	}
 
-	/**
-	 * @return
-	 * @since 1.0
-	 */
-	public String getPartialFilterExpression() {
-		return partialFilterExpression;
-	}
-
 	@Override
 	public String toString() {
 		return "IndexInfo [indexFields=" + indexFields + ", name=" + name + ", unique=" + unique + ", dropDuplicates="
-				+ dropDuplicates + ", sparse=" + sparse + ", language=" + language + ", partialFilterExpression="
-				+ partialFilterExpression + "]";
+				+ dropDuplicates + ", sparse=" + sparse + ", language=" + language + "]";
 	}
 
 	@Override
@@ -209,7 +130,6 @@ public class IndexInfo {
 		result = prime * result + (sparse ? 1231 : 1237);
 		result = prime * result + (unique ? 1231 : 1237);
 		result = prime * result + ObjectUtils.nullSafeHashCode(language);
-		result = prime * result + ObjectUtils.nullSafeHashCode(partialFilterExpression);
 		return result;
 	}
 
@@ -251,9 +171,6 @@ public class IndexInfo {
 		if (!ObjectUtils.nullSafeEquals(language, other.language)) {
 			return false;
 		}
-		if (!ObjectUtils.nullSafeEquals(partialFilterExpression, other.partialFilterExpression)) {
-			return false;
-		}
 		return true;
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/Indexed.java

@@ -29,9 +29,8 @@ import java.lang.annotation.Target;
  * @author Johno Crawford
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Jordi Llach
  */
-@Target({ElementType.ANNOTATION_TYPE, ElementType.FIELD})
+@Target(ElementType.FIELD)
 @Retention(RetentionPolicy.RUNTIME)
 public @interface Indexed {
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoMappingEventPublisher.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -34,7 +34,6 @@ import org.springframework.util.Assert;
  * 
  * @author Jon Brisbin
  * @author Oliver Gierke
- * @author Mark Paluch
  */
 public class MongoMappingEventPublisher implements ApplicationEventPublisher {
 
@@ -47,7 +46,7 @@ public class MongoMappingEventPublisher implements ApplicationEventPublisher {
 	 */
 	public MongoMappingEventPublisher(MongoPersistentEntityIndexCreator indexCreator) {
 
-		Assert.notNull(indexCreator, "MongoPersistentEntityIndexCreator must not be null!");
+		Assert.notNull(indexCreator);
 		this.indexCreator = indexCreator;
 	}
 
@@ -62,7 +61,7 @@ public class MongoMappingEventPublisher implements ApplicationEventPublisher {
 		}
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see org.springframework.context.ApplicationEventPublisher#publishEvent(java.lang.Object)
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexCreator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,7 +21,6 @@ import java.util.concurrent.ConcurrentHashMap;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.context.ApplicationListener;
-import org.springframework.dao.DataIntegrityViolationException;
 import org.springframework.data.mapping.PersistentEntity;
 import org.springframework.data.mapping.context.MappingContext;
 import org.springframework.data.mapping.context.MappingContextEvent;
@@ -30,12 +29,7 @@ import org.springframework.data.mongodb.core.index.MongoPersistentEntityIndexRes
 import org.springframework.data.mongodb.core.mapping.Document;
 import org.springframework.data.mongodb.core.mapping.MongoMappingContext;
 import org.springframework.data.mongodb.core.mapping.MongoPersistentEntity;
-import org.springframework.data.mongodb.util.MongoDbErrorCodes;
 import org.springframework.util.Assert;
-import org.springframework.util.ObjectUtils;
-
-import com.mongodb.DBObject;
-import com.mongodb.MongoException;
 
 /**
  * Component that inspects {@link MongoPersistentEntity} instances contained in the given {@link MongoMappingContext}
@@ -79,9 +73,9 @@ public class MongoPersistentEntityIndexCreator implements ApplicationListener<Ma
 	public MongoPersistentEntityIndexCreator(MongoMappingContext mappingContext, MongoDbFactory mongoDbFactory,
 			IndexResolver indexResolver) {
 
-		Assert.notNull(mappingContext, "MongoMappingContext must not be null!");
-		Assert.notNull(mongoDbFactory, "MongoDbFactory must not be null!");
-		Assert.notNull(indexResolver, "IndexResolver must not be null!");
+		Assert.notNull(mongoDbFactory);
+		Assert.notNull(mappingContext);
+		Assert.notNull(indexResolver);
 
 		this.mongoDbFactory = mongoDbFactory;
 		this.mappingContext = mappingContext;
@@ -135,34 +129,9 @@ public class MongoPersistentEntityIndexCreator implements ApplicationListener<Ma
 		}
 	}
 
-	void createIndex(IndexDefinitionHolder indexDefinition) {
-
-		try {
-
-			mongoDbFactory.getDb().getCollection(indexDefinition.getCollection()).createIndex(indexDefinition.getIndexKeys(),
-					indexDefinition.getIndexOptions());
-
-		} catch (MongoException ex) {
-
-			if (MongoDbErrorCodes.isDataIntegrityViolationCode(ex.getCode())) {
-
-				DBObject existingIndex = fetchIndexInformation(indexDefinition);
-				String message = "Cannot create index for '%s' in collection '%s' with keys '%s' and options '%s'.";
-
-				if (existingIndex != null) {
-					message += " Index already defined as '%s'.";
-				}
-
-				throw new DataIntegrityViolationException(
-						String.format(message, indexDefinition.getPath(), indexDefinition.getCollection(),
-								indexDefinition.getIndexKeys(), indexDefinition.getIndexOptions(), existingIndex),
-						ex);
-			}
-
-			RuntimeException exceptionToThrow = mongoDbFactory.getExceptionTranslator().translateExceptionIfPossible(ex);
-
-			throw exceptionToThrow != null ? exceptionToThrow : ex;
-		}
+	private void createIndex(IndexDefinitionHolder indexDefinition) {
+		mongoDbFactory.getDb().getCollection(indexDefinition.getCollection()).createIndex(indexDefinition.getIndexKeys(),
+				indexDefinition.getIndexOptions());
 	}
 
 	/**
@@ -174,28 +143,4 @@ public class MongoPersistentEntityIndexCreator implements ApplicationListener<Ma
 	public boolean isIndexCreatorFor(MappingContext<?, ?> context) {
 		return this.mappingContext.equals(context);
 	}
-
-	private DBObject fetchIndexInformation(IndexDefinitionHolder indexDefinition) {
-
-		if (indexDefinition == null) {
-			return null;
-		}
-
-		try {
-
-			Object indexNameToLookUp = indexDefinition.getIndexOptions().get("name");
-
-			for (DBObject index : mongoDbFactory.getDb().getCollection(indexDefinition.getCollection()).getIndexInfo()) {
-				if (ObjectUtils.nullSafeEquals(indexNameToLookUp, index.get("name"))) {
-					return index;
-				}
-			}
-
-		} catch (Exception e) {
-			LOGGER.debug(
-					String.format("Failed to load index information for collection '%s'.", indexDefinition.getCollection()), e);
-		}
-
-		return null;
-	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/MongoPersistentEntityIndexResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2017 the original author or authors.
+ * Copyright 2014-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,10 +27,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.dao.InvalidDataAccessApiUsageException;
 import org.springframework.data.domain.Sort;
-import org.springframework.data.mapping.Association;
-import org.springframework.data.mapping.AssociationHandler;
 import org.springframework.data.mapping.PropertyHandler;
-import org.springframework.data.mapping.model.MappingException;
 import org.springframework.data.mongodb.core.index.Index.Duplicates;
 import org.springframework.data.mongodb.core.index.MongoPersistentEntityIndexResolver.TextIndexIncludeOptions.IncludeStrategy;
 import org.springframework.data.mongodb.core.index.TextIndexDefinition.TextIndexDefinitionBuilder;
@@ -53,10 +50,9 @@ import com.mongodb.util.JSON;
  * indexed. <br />
  * All {@link MongoPersistentProperty} of the {@link MongoPersistentEntity} are inspected for potential indexes by
  * scanning related annotations.
- *
+ * 
  * @author Christoph Strobl
  * @author Thomas Darimont
- * @author Martin Macko
  * @since 1.5
  */
 public class MongoPersistentEntityIndexResolver implements IndexResolver {
@@ -67,7 +63,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 	/**
 	 * Create new {@link MongoPersistentEntityIndexResolver}.
-	 *
+	 * 
 	 * @param mappingContext must not be {@literal null}.
 	 */
 	public MongoPersistentEntityIndexResolver(MongoMappingContext mappingContext) {
@@ -88,7 +84,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 	 * Resolve the {@link IndexDefinition}s for given {@literal root} entity by traversing {@link MongoPersistentProperty}
 	 * scanning for index annotations {@link Indexed}, {@link CompoundIndex} and {@link GeospatialIndex}. The given
 	 * {@literal root} has therefore to be annotated with {@link Document}.
-	 *
+	 * 
 	 * @param root must not be null.
 	 * @return List of {@link IndexDefinitionHolder}. Will never be {@code null}.
 	 * @throws IllegalArgumentException in case of missing {@link Document} annotation marking root entities.
@@ -112,7 +108,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 				try {
 					if (persistentProperty.isEntity()) {
-						indexInformation.addAll(resolveIndexForClass(persistentProperty.getTypeInformation().getActualType(),
+						indexInformation.addAll(resolveIndexForClass(persistentProperty.getActualType(),
 								persistentProperty.getFieldName(), root.getCollection(), guard));
 					}
 
@@ -127,21 +123,19 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 			}
 		});
 
-		indexInformation.addAll(resolveIndexesForDbrefs("", root.getCollection(), root));
-
 		return indexInformation;
 	}
 
 	/**
 	 * Recursively resolve and inspect properties of given {@literal type} for indexes to be created.
-	 *
+	 * 
 	 * @param type
 	 * @param path The {@literal "dot} path.
 	 * @param collection
 	 * @return List of {@link IndexDefinitionHolder} representing indexes for given type and its referenced property
 	 *         types. Will never be {@code null}.
 	 */
-	private List<IndexDefinitionHolder> resolveIndexForClass(final TypeInformation<?> type, final String path,
+	private List<IndexDefinitionHolder> resolveIndexForClass(final Class<?> type, final String path,
 			final String collection, final CycleGuard guard) {
 
 		MongoPersistentEntity<?> entity = mappingContext.getPersistentEntity(type);
@@ -159,8 +153,8 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 				if (persistentProperty.isEntity()) {
 					try {
-						indexInformation.addAll(resolveIndexForClass(persistentProperty.getTypeInformation().getActualType(),
-								propertyDotPath, collection, guard));
+						indexInformation.addAll(resolveIndexForClass(persistentProperty.getActualType(), propertyDotPath,
+								collection, guard));
 					} catch (CyclicPropertyReferenceException e) {
 						LOGGER.info(e.getMessage());
 					}
@@ -174,8 +168,6 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 			}
 		});
 
-		indexInformation.addAll(resolveIndexesForDbrefs(path, collection, entity));
-
 		return indexInformation;
 	}
 
@@ -201,19 +193,18 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 		return createCompoundIndexDefinitions(dotPath, collection, entity);
 	}
 
-	private Collection<? extends IndexDefinitionHolder> potentiallyCreateTextIndexDefinition(
-			MongoPersistentEntity<?> root) {
+	private Collection<? extends IndexDefinitionHolder> potentiallyCreateTextIndexDefinition(MongoPersistentEntity<?> root) {
 
-		TextIndexDefinitionBuilder indexDefinitionBuilder = new TextIndexDefinitionBuilder()
-				.named(root.getType().getSimpleName() + "_TextIndex");
+		TextIndexDefinitionBuilder indexDefinitionBuilder = new TextIndexDefinitionBuilder().named(root.getType()
+				.getSimpleName() + "_TextIndex");
 
 		if (StringUtils.hasText(root.getLanguage())) {
 			indexDefinitionBuilder.withDefaultLanguage(root.getLanguage());
 		}
 
 		try {
-			appendTextIndexInformation("", indexDefinitionBuilder, root, new TextIndexIncludeOptions(IncludeStrategy.DEFAULT),
-					new CycleGuard());
+			appendTextIndexInformation("", indexDefinitionBuilder, root,
+					new TextIndexIncludeOptions(IncludeStrategy.DEFAULT), new CycleGuard());
 		} catch (CyclicPropertyReferenceException e) {
 			LOGGER.info(e.getMessage());
 		}
@@ -229,8 +220,9 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 	}
 
-	private void appendTextIndexInformation(final String dotPath, final TextIndexDefinitionBuilder indexDefinitionBuilder,
-			final MongoPersistentEntity<?> entity, final TextIndexIncludeOptions includeOptions, final CycleGuard guard) {
+	private void appendTextIndexInformation(final String dotPath,
+			final TextIndexDefinitionBuilder indexDefinitionBuilder, final MongoPersistentEntity<?> entity,
+			final TextIndexIncludeOptions includeOptions, final CycleGuard guard) {
 
 		entity.doWithProperties(new PropertyHandler<MongoPersistentProperty>() {
 
@@ -257,18 +249,19 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 						TextIndexIncludeOptions optionsForNestedType = includeOptions;
 						if (!IncludeStrategy.FORCE.equals(includeOptions.getStrategy()) && indexed != null) {
-							optionsForNestedType = new TextIndexIncludeOptions(IncludeStrategy.FORCE,
-									new TextIndexedFieldSpec(propertyDotPath, weight));
+							optionsForNestedType = new TextIndexIncludeOptions(IncludeStrategy.FORCE, new TextIndexedFieldSpec(
+									propertyDotPath, weight));
 						}
 
 						try {
 							appendTextIndexInformation(propertyDotPath, indexDefinitionBuilder,
 									mappingContext.getPersistentEntity(persistentProperty.getActualType()), optionsForNestedType, guard);
 						} catch (CyclicPropertyReferenceException e) {
-							LOGGER.info(e.getMessage());
+							LOGGER.info(e.getMessage(), e);
 						} catch (InvalidDataAccessApiUsageException e) {
-							LOGGER.info(String.format("Potentially invalid index structure discovered. Breaking operation for %s.",
-									entity.getName()), e);
+							LOGGER.info(
+									String.format("Potentially invalid index structure discovered. Breaking operation for %s.",
+											entity.getName()), e);
 						}
 					} else if (includeOptions.isForce() || indexed != null) {
 						indexDefinitionBuilder.onField(propertyDotPath, weight);
@@ -282,7 +275,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 	/**
 	 * Create {@link IndexDefinition} wrapped in {@link IndexDefinitionHolder} for {@link CompoundIndexes} of given type.
-	 *
+	 * 
 	 * @param dotPath The properties {@literal "dot"} path representation from its document root.
 	 * @param fallbackCollection
 	 * @param type
@@ -313,8 +306,8 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 	protected IndexDefinitionHolder createCompoundIndexDefinition(String dotPath, String fallbackCollection,
 			CompoundIndex index, MongoPersistentEntity<?> entity) {
 
-		CompoundIndexDefinition indexDefinition = new CompoundIndexDefinition(
-				resolveCompoundIndexKeyFromStringDefinition(dotPath, index.def()));
+		CompoundIndexDefinition indexDefinition = new CompoundIndexDefinition(resolveCompoundIndexKeyFromStringDefinition(
+				dotPath, index.def()));
 
 		if (!index.useGeneratedName()) {
 			indexDefinition.named(pathAwareIndexName(index.name(), dotPath, null));
@@ -362,7 +355,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 	/**
 	 * Creates {@link IndexDefinition} wrapped in {@link IndexDefinitionHolder} out of {@link Indexed} for given
 	 * {@link MongoPersistentProperty}.
-	 *
+	 * 
 	 * @param dotPath The properties {@literal "dot"} path representation from its document root.
 	 * @param collection
 	 * @param persitentProperty
@@ -403,7 +396,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 	/**
 	 * Creates {@link IndexDefinition} wrapped in {@link IndexDefinitionHolder} out of {@link GeoSpatialIndexed} for
 	 * {@link MongoPersistentProperty}.
-	 *
+	 * 
 	 * @param dotPath The properties {@literal "dot"} path representation from its document root.
 	 * @param collection
 	 * @param persistentProperty
@@ -438,49 +431,17 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 		if (StringUtils.hasText(dotPath)) {
 
-			nameToUse = StringUtils.hasText(nameToUse)
-					? (property != null ? dotPath.replace("." + property.getFieldName(), "") : dotPath) + "." + nameToUse
-					: dotPath;
+			nameToUse = StringUtils.hasText(nameToUse) ? (property != null ? dotPath.replace("." + property.getFieldName(),
+					"") : dotPath) + "." + nameToUse : dotPath;
 		}
 		return nameToUse;
 
 	}
 
-	private List<IndexDefinitionHolder> resolveIndexesForDbrefs(final String path, final String collection,
-			MongoPersistentEntity<?> entity) {
-
-		final List<IndexDefinitionHolder> indexes = new ArrayList<IndexDefinitionHolder>(0);
-		entity.doWithAssociations(new AssociationHandler<MongoPersistentProperty>() {
-
-			@Override
-			public void doWithAssociation(Association<MongoPersistentProperty> association) {
-
-				MongoPersistentProperty property = association.getInverse();
-
-				String propertyDotPath = (StringUtils.hasText(path) ? path + "." : "") + property.getFieldName();
-
-				if (property.isAnnotationPresent(GeoSpatialIndexed.class) || property.isAnnotationPresent(TextIndexed.class)) {
-					throw new MappingException(
-							String.format("Cannot create geospatial-/text- index on DBRef in collection '%s' for path '%s'.",
-									collection, propertyDotPath));
-				}
-
-				IndexDefinitionHolder indexDefinitionHolder = createIndexDefinitionHolderForProperty(propertyDotPath,
-						collection, property);
-
-				if (indexDefinitionHolder != null) {
-					indexes.add(indexDefinitionHolder);
-				}
-			}
-		});
-
-		return indexes;
-	}
-
 	/**
 	 * {@link CycleGuard} holds information about properties and the paths for accessing those. This information is used
 	 * to detect potential cycles within the references.
-	 *
+	 * 
 	 * @author Christoph Strobl
 	 */
 	static class CycleGuard {
@@ -530,24 +491,24 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 		/**
 		 * Path defines the property and its full path from the document root. <br />
 		 * A {@link Path} with {@literal spring.data.mongodb} would be created for the property {@code Three.mongodb}.
-		 *
+		 * 
 		 * <pre>
 		 * <code>
 		 * &#64;Document
 		 * class One {
 		 *   Two spring;
 		 * }
-		 *
+		 * 
 		 * class Two {
 		 *   Three data;
 		 * }
-		 *
+		 * 
 		 * class Three {
 		 *   String mongodb;
 		 * }
 		 * </code>
 		 * </pre>
-		 *
+		 * 
 		 * @author Christoph Strobl
 		 */
 		static class Path {
@@ -570,7 +531,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 			 * the current path. Given {@literal foo.bar.bar} cycles if {@literal foo.bar} has already been visited and
 			 * {@code class Bar} contains a property of type {@code Bar}. The previously mentioned path would not cycle if
 			 * {@code class Bar} contained a property of type {@code SomeEntity} named {@literal bar}.
-			 *
+			 * 
 			 * @param property
 			 * @param path
 			 * @return
@@ -619,7 +580,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 	/**
 	 * Implementation of {@link IndexDefinition} holding additional (property)path information used for creating the
 	 * index. The path itself is the properties {@literal "dot"} path representation from its root document.
-	 *
+	 * 
 	 * @author Christoph Strobl
 	 * @since 1.5
 	 */
@@ -631,7 +592,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 		/**
 		 * Create
-		 *
+		 * 
 		 * @param path
 		 */
 		public IndexDefinitionHolder(String path, IndexDefinition definition, String collection) {
@@ -647,7 +608,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 		/**
 		 * Get the {@literal "dot"} path used to create the index.
-		 *
+		 * 
 		 * @return
 		 */
 		public String getPath() {
@@ -656,7 +617,7 @@ public class MongoPersistentEntityIndexResolver implements IndexResolver {
 
 		/**
 		 * Get the {@literal raw} {@link IndexDefinition}.
-		 *
+		 * 
 		 * @return
 		 */
 		public IndexDefinition getIndexDefinition() {

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/PartialIndexFilter.java

@@ -1,75 +0,0 @@
-/*
- * Copyright 2016. the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core.index;
-
-import lombok.AccessLevel;
-import lombok.NonNull;
-import lombok.RequiredArgsConstructor;
-
-import org.springframework.data.mongodb.core.query.CriteriaDefinition;
-
-import com.mongodb.DBObject;
-
-/**
- * {@link IndexFilter} implementation for usage with plain {@link DBObject} as well as {@link CriteriaDefinition} filter
- * expressions.
- *
- * @author Christoph Strobl
- * @since 1.10
- */
-@RequiredArgsConstructor(access = AccessLevel.PRIVATE)
-public class PartialIndexFilter implements IndexFilter {
-
-	private final @NonNull Object filterExpression;
-
-	/**
-	 * Create new {@link PartialIndexFilter} for given {@link DBObject filter expression}.
-	 *
-	 * @param where must not be {@literal null}.
-	 * @return
-	 */
-	public static PartialIndexFilter of(DBObject where) {
-		return new PartialIndexFilter(where);
-	}
-
-	/**
-	 * Create new {@link PartialIndexFilter} for given {@link CriteriaDefinition filter expression}.
-	 *
-	 * @param where must not be {@literal null}.
-	 * @return
-	 */
-	public static PartialIndexFilter of(CriteriaDefinition where) {
-		return new PartialIndexFilter(where);
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see org.springframework.data.mongodb.core.index.IndexFilter#getFilterObject()
-	 */
-	public DBObject getFilterObject() {
-
-		if (filterExpression instanceof DBObject) {
-			return (DBObject) filterExpression;
-		}
-
-		if (filterExpression instanceof CriteriaDefinition) {
-			return ((CriteriaDefinition) filterExpression).getCriteriaObject();
-		}
-
-		throw new IllegalArgumentException(
-				String.format("Unknown type %s used as filter expression.", filterExpression.getClass()));
-	}
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/TextIndexDefinition.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2016 the original author or authors.
+ * Copyright 2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -39,7 +39,6 @@ public class TextIndexDefinition implements IndexDefinition {
 	private Set<TextIndexedFieldSpec> fieldSpecs;
 	private String defaultLanguage;
 	private String languageOverride;
-	private IndexFilter filter;
 
 	TextIndexDefinition() {
 		fieldSpecs = new LinkedHashSet<TextIndexedFieldSpec>();
@@ -130,10 +129,6 @@ public class TextIndexDefinition implements IndexDefinition {
 			options.put("language_override", languageOverride);
 		}
 
-		if (filter != null) {
-			options.put("partialFilterExpression", filter.getFilterObject());
-		}
-
 		return options;
 	}
 
@@ -293,8 +288,8 @@ public class TextIndexDefinition implements IndexDefinition {
 		public TextIndexDefinitionBuilder onField(String fieldname, Float weight) {
 
 			if (this.instance.fieldSpecs.contains(ALL_FIELDS)) {
-				throw new InvalidDataAccessApiUsageException(
-						String.format("Cannot add %s to field spec for all fields.", fieldname));
+				throw new InvalidDataAccessApiUsageException(String.format("Cannot add %s to field spec for all fields.",
+						fieldname));
 			}
 
 			this.instance.fieldSpecs.add(new TextIndexedFieldSpec(fieldname, weight));
@@ -323,30 +318,15 @@ public class TextIndexDefinition implements IndexDefinition {
 		public TextIndexDefinitionBuilder withLanguageOverride(String fieldname) {
 
 			if (StringUtils.hasText(this.instance.languageOverride)) {
-				throw new InvalidDataAccessApiUsageException(
-						String.format("Cannot set language override on %s as it is already defined on %s.", fieldname,
-								this.instance.languageOverride));
+				throw new InvalidDataAccessApiUsageException(String.format(
+						"Cannot set language override on %s as it is already defined on %s.", fieldname,
+						this.instance.languageOverride));
 			}
 
 			this.instance.languageOverride = fieldname;
 			return this;
 		}
 
-		/**
-		 * Only index the documents that meet the specified {@link IndexFilter filter expression}.
-		 *
-		 * @param filter can be {@literal null}.
-		 * @return
-		 * @see <a href=
-		 *      "https://docs.mongodb.com/manual/core/index-partial/">https://docs.mongodb.com/manual/core/index-partial/</a>
-		 * @since 1.10
-		 */
-		public TextIndexDefinitionBuilder partial(IndexFilter filter) {
-
-			this.instance.filter = filter;
-			return this;
-		}
-
 		public TextIndexDefinition build() {
 			return this.instance;
 		}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/index/TextIndexed.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2016 the original author or authors.
+ * Copyright 2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,11 +26,10 @@ import java.lang.annotation.Target;
  * all fields marked with {@link TextIndexed} are combined into one single index. <br />
  * 
  * @author Christoph Strobl
- * @author Mark Paluch
  * @since 1.6
  */
 @Documented
-@Target({ ElementType.FIELD, ElementType.ANNOTATION_TYPE })
+@Target({ ElementType.FIELD })
 @Retention(RetentionPolicy.RUNTIME)
 public @interface TextIndexed {
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/BasicMongoPersistentEntity.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -51,7 +51,6 @@ import org.springframework.util.StringUtils;
  * @author Oliver Gierke
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class BasicMongoPersistentEntity<T> extends BasicPersistentEntity<T, MongoPersistentProperty> implements
 		MongoPersistentEntity<T>, ApplicationContextAware {
@@ -78,7 +77,7 @@ public class BasicMongoPersistentEntity<T> extends BasicPersistentEntity<T, Mong
 		Class<?> rawType = typeInformation.getType();
 		String fallback = MongoCollectionUtils.getPreferredCollectionName(rawType);
 
-		Document document = this.findAnnotation(Document.class);
+		Document document = rawType.getAnnotation(Document.class);
 
 		this.expression = detectExpression(document);
 		this.context = new StandardEvaluationContext();
@@ -139,7 +138,7 @@ public class BasicMongoPersistentEntity<T> extends BasicPersistentEntity<T, Mong
 		return getTextScoreProperty() != null;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see org.springframework.data.mapping.model.BasicPersistentEntity#verify()
 	 */
@@ -201,7 +200,7 @@ public class BasicMongoPersistentEntity<T> extends BasicPersistentEntity<T, Mong
 	@Override
 	protected MongoPersistentProperty returnPropertyIfBetterIdPropertyCandidateOrNull(MongoPersistentProperty property) {
 
-		Assert.notNull(property, "MongoPersistentProperty must not be null!");
+		Assert.notNull(property);
 
 		if (!property.isIdProperty()) {
 			return null;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/Field.java

@@ -1,25 +1,8 @@
-/*
- * Copyright 2011-2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
 package org.springframework.data.mongodb.core.mapping;
 
 import java.lang.annotation.Documented;
-import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
-import java.lang.annotation.Target;
 
 /**
  * Annotation to define custom metadata for document fields.
@@ -28,7 +11,6 @@ import java.lang.annotation.Target;
  */
 @Documented
 @Retention(RetentionPolicy.RUNTIME)
-@Target({ ElementType.FIELD, ElementType.METHOD, ElementType.ANNOTATION_TYPE })
 public @interface Field {
 
 	/**

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/MongoSimpleTypes.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2017 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,8 +26,6 @@ import org.bson.types.Binary;
 import org.bson.types.CodeWScope;
 import org.bson.types.ObjectId;
 import org.springframework.data.mapping.model.SimpleTypeHolder;
-import org.springframework.data.mongodb.util.MongoClientVersion;
-import org.springframework.util.ClassUtils;
 
 import com.mongodb.DBObject;
 import com.mongodb.DBRef;
@@ -36,7 +34,6 @@ import com.mongodb.DBRef;
  * Simple constant holder for a {@link SimpleTypeHolder} enriched with Mongo specific simple types.
  * 
  * @author Oliver Gierke
- * @author Christoph Strobl
  */
 public abstract class MongoSimpleTypes {
 
@@ -57,17 +54,12 @@ public abstract class MongoSimpleTypes {
 		simpleTypes.add(Pattern.class);
 		simpleTypes.add(Binary.class);
 		simpleTypes.add(UUID.class);
-
-		if (MongoClientVersion.isMongo34Driver()) {
-			simpleTypes
-					.add(ClassUtils.resolveClassName("org.bson.types.Decimal128", MongoSimpleTypes.class.getClassLoader()));
-		}
-
 		MONGO_SIMPLE_TYPES = Collections.unmodifiableSet(simpleTypes);
 	}
 
 	private static final Set<Class<?>> MONGO_SIMPLE_TYPES;
 	public static final SimpleTypeHolder HOLDER = new SimpleTypeHolder(MONGO_SIMPLE_TYPES, true);
 
-	private MongoSimpleTypes() {}
+	private MongoSimpleTypes() {
+	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractDeleteEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2015 by the original author(s).
+ * Copyright 2013 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,7 +21,6 @@ import com.mongodb.DBObject;
  * Base class for delete events.
  * 
  * @author Martin Baumgartner
- * @author Christoph Strobl
  */
 public abstract class AbstractDeleteEvent<T> extends MongoMappingEvent<DBObject> {
 
@@ -32,25 +31,11 @@ public abstract class AbstractDeleteEvent<T> extends MongoMappingEvent<DBObject>
 	 * Creates a new {@link AbstractDeleteEvent} for the given {@link DBObject} and type.
 	 * 
 	 * @param dbo must not be {@literal null}.
-	 * @param type can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #AbstractDeleteEvent(DBObject, Class, String)}.
+	 * @param type , possibly be {@literal null}.
 	 */
-	@Deprecated
 	public AbstractDeleteEvent(DBObject dbo, Class<T> type) {
-		this(dbo, type, null);
-	}
 
-	/**
-	 * Creates a new {@link AbstractDeleteEvent} for the given {@link DBObject} and type.
-	 * 
-	 * @param dbo must not be {@literal null}.
-	 * @param type can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public AbstractDeleteEvent(DBObject dbo, Class<T> type, String collectionName) {
-
-		super(dbo, dbo, collectionName);
+		super(dbo, dbo);
 		this.type = type;
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AbstractMongoEventListener.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2015 by the original author(s).
+ * Copyright 2011-2013 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -28,7 +28,6 @@ import com.mongodb.DBObject;
  * @author Jon Brisbin
  * @author Oliver Gierke
  * @author Martin Baumgartner
- * @author Christoph Strobl
  */
 public abstract class AbstractMongoEventListener<E> implements ApplicationListener<MongoMappingEvent<?>> {
 
@@ -47,14 +46,14 @@ public abstract class AbstractMongoEventListener<E> implements ApplicationListen
 	 * (non-Javadoc)
 	 * @see org.springframework.context.ApplicationListener#onApplicationEvent(org.springframework.context.ApplicationEvent)
 	 */
-	@SuppressWarnings({ "rawtypes", "unchecked" })
+	@SuppressWarnings("rawtypes")
 	public void onApplicationEvent(MongoMappingEvent<?> event) {
 
 		if (event instanceof AfterLoadEvent) {
 			AfterLoadEvent<?> afterLoadEvent = (AfterLoadEvent<?>) event;
 
 			if (domainClass.isAssignableFrom(afterLoadEvent.getType())) {
-				onAfterLoad((AfterLoadEvent<E>) event);
+				onAfterLoad(event.getDBObject());
 			}
 
 			return;
@@ -66,18 +65,18 @@ public abstract class AbstractMongoEventListener<E> implements ApplicationListen
 
 			if (eventDomainType != null && domainClass.isAssignableFrom(eventDomainType)) {
 				if (event instanceof BeforeDeleteEvent) {
-					onBeforeDelete((BeforeDeleteEvent<E>) event);
+					onBeforeDelete(event.getDBObject());
 				}
 				if (event instanceof AfterDeleteEvent) {
-					onAfterDelete((AfterDeleteEvent<E>) event);
+					onAfterDelete(event.getDBObject());
 				}
 			}
 
 			return;
-
 		}
 
-		Object source = event.getSource();
+		@SuppressWarnings("unchecked")
+		E source = (E) event.getSource();
 
 		// Check for matching domain type and invoke callbacks
 		if (source != null && !domainClass.isAssignableFrom(source.getClass())) {
@@ -85,185 +84,55 @@ public abstract class AbstractMongoEventListener<E> implements ApplicationListen
 		}
 
 		if (event instanceof BeforeConvertEvent) {
-			onBeforeConvert((BeforeConvertEvent<E>) event);
+			onBeforeConvert(source);
 		} else if (event instanceof BeforeSaveEvent) {
-			onBeforeSave((BeforeSaveEvent<E>) event);
+			onBeforeSave(source, event.getDBObject());
 		} else if (event instanceof AfterSaveEvent) {
-			onAfterSave((AfterSaveEvent<E>) event);
+			onAfterSave(source, event.getDBObject());
 		} else if (event instanceof AfterConvertEvent) {
-			onAfterConvert((AfterConvertEvent<E>) event);
+			onAfterConvert(event.getDBObject(), source);
 		}
 	}
 
-	/**
-	 * Captures source element before conversion.
-	 * 
-	 * @param source will never be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onBeforeConvert(BeforeConvertEvent)}.
-	 */
-	@Deprecated
 	public void onBeforeConvert(E source) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onBeforeConvert({})", source);
 		}
 	}
 
-	/**
-	 * Captures {@link BeforeConvertEvent}.
-	 * 
-	 * @param event never {@literal null}.
-	 * @since 1.8
-	 */
-	public void onBeforeConvert(BeforeConvertEvent<E> event) {
-		onBeforeConvert(event.getSource());
-	}
-
-	/**
-	 * Captures source element and {@link com.mongodb.DBObject} representation before save.
-	 * 
-	 * @param source will never be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onBeforeSave(BeforeSaveEvent)}.
-	 */
-	@Deprecated
 	public void onBeforeSave(E source, DBObject dbo) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onBeforeSave({}, {})", source, dbo);
 		}
 	}
 
-	/**
-	 * Captures {@link BeforeSaveEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onBeforeSave(BeforeSaveEvent<E> event) {
-		onBeforeSave(event.getSource(), event.getDBObject());
-	}
-
-	/**
-	 * Captures source element and {@link com.mongodb.DBObject} representation after save.
-	 * 
-	 * @param source will never be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onAfterSave(AfterSaveEvent)}.
-	 */
-	@Deprecated
 	public void onAfterSave(E source, DBObject dbo) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onAfterSave({}, {})", source, dbo);
 		}
 	}
 
-	/**
-	 * Captures {@link AfterSaveEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onAfterSave(AfterSaveEvent<E> event) {
-		onAfterSave(event.getSource(), event.getDBObject());
-	}
-
-	/**
-	 * Captures raw {@link com.mongodb.DBObject} when read from MongoDB.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onAfterLoad(AfterLoadEvent)}.
-	 */
-	@Deprecated
 	public void onAfterLoad(DBObject dbo) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onAfterLoad({})", dbo);
 		}
 	}
 
-	/**
-	 * Captures {@link AfterLoadEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onAfterLoad(AfterLoadEvent<E> event) {
-		onAfterLoad(event.getDBObject());
-	}
-
-	/**
-	 * Captures raw {@link com.mongodb.DBObject} and converted domain type after conversion.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @param source will never be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onAfterConvert(AfterConvertEvent)}.
-	 */
-	@Deprecated
 	public void onAfterConvert(DBObject dbo, E source) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onAfterConvert({}, {})", dbo, source);
 		}
 	}
 
-	/**
-	 * Captures {@link AfterConvertEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onAfterConvert(AfterConvertEvent<E> event) {
-		onAfterConvert(event.getDBObject(), event.getSource());
-
-	}
-
-	/**
-	 * Captures {@link com.mongodb.DBObject} after delete.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onAfterDelete(AfterDeleteEvent)}.
-	 */
-	@Deprecated
 	public void onAfterDelete(DBObject dbo) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onAfterDelete({})", dbo);
 		}
 	}
 
-	/**
-	 * Captures {@link AfterDeleteEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onAfterDelete(AfterDeleteEvent<E> event) {
-		onAfterDelete(event.getDBObject());
-	}
-
-	/**
-	 * Capture {@link com.mongodb.DBObject} before delete.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #onBeforeDelete(BeforeDeleteEvent)}.
-	 */
-	@Deprecated
 	public void onBeforeDelete(DBObject dbo) {
-
 		if (LOG.isDebugEnabled()) {
 			LOG.debug("onBeforeDelete({})", dbo);
 		}
 	}
-
-	/**
-	 * Capture {@link BeforeDeleteEvent}.
-	 * 
-	 * @param event will never be {@literal null}.
-	 * @since 1.8
-	 */
-	public void onBeforeDelete(BeforeDeleteEvent<E> event) {
-		onBeforeDelete(event.getDBObject());
-	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AfterConvertEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -13,42 +13,20 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.springframework.data.mongodb.core.mapping.event;
 
 import com.mongodb.DBObject;
 
 /**
- * {@link MongoMappingEvent} thrown after convert of a document.
- * 
  * @author Jon Brisbin <jbrisbin@vmware.com>
- * @author Christoph Strobl
  */
 public class AfterConvertEvent<E> extends MongoMappingEvent<E> {
 
 	private static final long serialVersionUID = 1L;
 
-	/**
-	 * Creates new {@link AfterConvertEvent}.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @param source must not be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #AfterConvertEvent(DBObject, Object, String)}.
-	 */
-	@Deprecated
 	public AfterConvertEvent(DBObject dbo, E source) {
-		this(dbo, source, null);
-	}
-
-	/**
-	 * Creates new {@link AfterConvertEvent}.
-	 * 
-	 * @param dbo can be {@literal null}.
-	 * @param source must not be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public AfterConvertEvent(DBObject dbo, E source, String collectionName) {
-		super(source, dbo, collectionName);
+		super(source, dbo);
 	}
 
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AfterDeleteEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2015 by the original author(s).
+ * Copyright 2013 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,7 +22,6 @@ import com.mongodb.DBObject;
  * will be the query document <em>after</am> it has been mapped onto the domain type handled.
  * 
  * @author Martin Baumgartner
- * @author Christoph Strobl
  */
 public class AfterDeleteEvent<T> extends AbstractDeleteEvent<T> {
 
@@ -33,22 +32,8 @@ public class AfterDeleteEvent<T> extends AbstractDeleteEvent<T> {
 	 * 
 	 * @param dbo must not be {@literal null}.
 	 * @param type can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #AfterDeleteEvent(DBObject, Class, String)}.
 	 */
-	@Deprecated
 	public AfterDeleteEvent(DBObject dbo, Class<T> type) {
-		this(dbo, type, null);
-	}
-
-	/**
-	 * Creates a new {@link AfterDeleteEvent} for the given {@link DBObject}, type and collectionName.
-	 * 
-	 * @param dbo must not be {@literal null}.
-	 * @param type can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public AfterDeleteEvent(DBObject dbo, Class<T> type, String collectionName) {
-		super(dbo, type, collectionName);
+		super(dbo, type);
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AfterLoadEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -26,7 +26,6 @@ import com.mongodb.DBObject;
  * @author Oliver Gierke
  * @author Jon Brisbin
  * @author Christoph Leiter
- * @author Christoph Strobl
  */
 public class AfterLoadEvent<T> extends MongoMappingEvent<DBObject> {
 
@@ -37,25 +36,11 @@ public class AfterLoadEvent<T> extends MongoMappingEvent<DBObject> {
 	 * Creates a new {@link AfterLoadEvent} for the given {@link DBObject} and type.
 	 * 
 	 * @param dbo must not be {@literal null}.
-	 * @param type can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #AfterLoadEvent(DBObject, Class, String)}.
-	 */
-	@Deprecated
-	public AfterLoadEvent(DBObject dbo, Class<T> type) {
-		this(dbo, type, null);
-	}
-
-	/**
-	 * Creates a new {@link AfterLoadEvent} for the given {@link DBObject}, type and collectionName.
-	 * 
-	 * @param dbo must not be {@literal null}.
 	 * @param type must not be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
 	 */
-	public AfterLoadEvent(DBObject dbo, Class<T> type, String collectionName) {
+	public AfterLoadEvent(DBObject dbo, Class<T> type) {
 
-		super(dbo, dbo, collectionName);
+		super(dbo, dbo);
 
 		Assert.notNull(type, "Type must not be null!");
 		this.type = type;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/AfterSaveEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,37 +19,14 @@ package org.springframework.data.mongodb.core.mapping.event;
 import com.mongodb.DBObject;
 
 /**
- * {@link MongoMappingEvent} triggered after save of a document.
- * 
  * @author Jon Brisbin <jbrisbin@vmware.com>
- * @author Christoph Strobl
  */
 public class AfterSaveEvent<E> extends MongoMappingEvent<E> {
 
 	private static final long serialVersionUID = 1L;
 
-	/**
-	 * Creates new {@link AfterSaveEvent}
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #AfterSaveEvent(Object, DBObject, String)}.
-	 */
-	@Deprecated
 	public AfterSaveEvent(E source, DBObject dbo) {
 		super(source, dbo);
 	}
 
-	/**
-	 * Creates new {@link AfterSaveEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public AfterSaveEvent(E source, DBObject dbo, String collectionName) {
-		super(source, dbo, collectionName);
-	}
-
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/BeforeConvertEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2015 the original author or authors.
+ * Copyright 2011-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,31 +20,12 @@ package org.springframework.data.mongodb.core.mapping.event;
  * 
  * @author Jon Brisbin
  * @author Oliver Gierke
- * @author Christoph Strobl
  */
 public class BeforeConvertEvent<T> extends MongoMappingEvent<T> {
 
 	private static final long serialVersionUID = 252614269008845243L;
 
-	/**
-	 * Creates new {@link BeforeConvertEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #BeforeConvertEvent(Object, String)}.
-	 */
-	@Deprecated
 	public BeforeConvertEvent(T source) {
-		this(source, null);
-	}
-
-	/**
-	 * Creates new {@link BeforeConvertEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public BeforeConvertEvent(T source, String collectionName) {
-		super(source, null, collectionName);
+		super(source, null);
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/BeforeDeleteEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2015 by the original author(s).
+ * Copyright 2013 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,7 +22,6 @@ import com.mongodb.DBObject;
  * document <em>before</em> being mapped based on the domain class handled.
  * 
  * @author Martin Baumgartner
- * @author Christoph Strobl
  */
 public class BeforeDeleteEvent<T> extends AbstractDeleteEvent<T> {
 
@@ -33,22 +32,8 @@ public class BeforeDeleteEvent<T> extends AbstractDeleteEvent<T> {
 	 * 
 	 * @param dbo must not be {@literal null}.
 	 * @param type can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #BeforeDeleteEvent(DBObject, Class, String)}.
 	 */
-	@Deprecated
 	public BeforeDeleteEvent(DBObject dbo, Class<T> type) {
-		this(dbo, type, null);
-	}
-
-	/**
-	 * Creates a new {@link BeforeDeleteEvent} for the given {@link DBObject}, type and collectionName.
-	 * 
-	 * @param dbo must not be {@literal null}.
-	 * @param type can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public BeforeDeleteEvent(DBObject dbo, Class<T> type, String collectionName) {
-		super(dbo, type, collectionName);
+		super(dbo, type);
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/BeforeSaveEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -19,37 +19,14 @@ package org.springframework.data.mongodb.core.mapping.event;
 import com.mongodb.DBObject;
 
 /**
- * {@link MongoMappingEvent} triggered before save of a document.
- * 
  * @author Jon Brisbin <jbrisbin@vmware.com>
- * @author Christoph Strobl
  */
 public class BeforeSaveEvent<E> extends MongoMappingEvent<E> {
 
 	private static final long serialVersionUID = 1L;
 
-	/**
-	 * Creates new {@link BeforeSaveEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #BeforeSaveEvent(Object, DBObject, String)}.
-	 */
-	@Deprecated
 	public BeforeSaveEvent(E source, DBObject dbo) {
 		super(source, dbo);
 	}
 
-	/**
-	 * Creates new {@link BeforeSaveEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 * @since 1.8
-	 */
-	public BeforeSaveEvent(E source, DBObject dbo, String collectionName) {
-		super(source, dbo, collectionName);
-	}
-
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/LoggingEventListener.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,8 +15,6 @@
  */
 package org.springframework.data.mongodb.core.mapping.event;
 
-import static org.springframework.data.mongodb.core.query.SerializationUtils.*;
-
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.context.ApplicationListener;
@@ -25,10 +23,9 @@ import com.mongodb.DBObject;
 
 /**
  * {@link ApplicationListener} for Mongo mapping events logging the events.
- *
+ * 
  * @author Jon Brisbin
  * @author Martin Baumgartner
- * @author Christoph Strobl
  */
 public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 
@@ -49,7 +46,7 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onBeforeSave(Object source, DBObject dbo) {
-		LOGGER.info("onBeforeSave: {}, {}", source, serializeToJsonSafely(dbo));
+		LOGGER.info("onBeforeSave: {}, {}", source, dbo);
 	}
 
 	/*
@@ -58,7 +55,7 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onAfterSave(Object source, DBObject dbo) {
-		LOGGER.info("onAfterSave: {}, {}", source, serializeToJsonSafely(dbo));
+		LOGGER.info("onAfterSave: {}, {}", source, dbo);
 	}
 
 	/*
@@ -67,7 +64,7 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onAfterLoad(DBObject dbo) {
-		LOGGER.info("onAfterLoad: {}", serializeToJsonSafely(dbo));
+		LOGGER.info("onAfterLoad: {}", dbo);
 	}
 
 	/*
@@ -76,7 +73,7 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onAfterConvert(DBObject dbo, Object source) {
-		LOGGER.info("onAfterConvert: {}, {}", serializeToJsonSafely(dbo), source);
+		LOGGER.info("onAfterConvert: {}, {}", dbo, source);
 	}
 
 	/*
@@ -85,7 +82,7 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onAfterDelete(DBObject dbo) {
-		LOGGER.info("onAfterDelete: {}", serializeToJsonSafely(dbo));
+		LOGGER.info("onAfterDelete: {}", dbo);
 	}
 
 	/*
@@ -94,6 +91,6 @@ public class LoggingEventListener extends AbstractMongoEventListener<Object> {
 	 */
 	@Override
 	public void onBeforeDelete(DBObject dbo) {
-		LOGGER.info("onBeforeDelete: {}", serializeToJsonSafely(dbo));
+		LOGGER.info("onBeforeDelete: {}", dbo);
 	}
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapping/event/MongoMappingEvent.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015 by the original author(s).
+ * Copyright (c) 2011 by the original author(s).
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,69 +16,26 @@
 
 package org.springframework.data.mongodb.core.mapping.event;
 
+import com.mongodb.DBObject;
 import org.springframework.context.ApplicationEvent;
 
-import com.mongodb.DBObject;
-
 /**
- * Base {@link ApplicationEvent} triggered by Spring Data MongoDB.
- * 
  * @author Jon Brisbin <jbrisbin@vmware.com>
- * @author Christoph Strobl
  */
 public class MongoMappingEvent<T> extends ApplicationEvent {
 
 	private static final long serialVersionUID = 1L;
 	private final DBObject dbo;
-	private final String collectionName;
 
-	/**
-	 * Creates new {@link MongoMappingEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @deprecated since 1.8. Please use {@link #MongoMappingEvent(Object, DBObject, String)}.
-	 */
-	@Deprecated
 	public MongoMappingEvent(T source, DBObject dbo) {
-		this(source, dbo, null);
-	}
-
-	/**
-	 * Creates new {@link MongoMappingEvent}.
-	 * 
-	 * @param source must not be {@literal null}.
-	 * @param dbo can be {@literal null}.
-	 * @param collectionName can be {@literal null}.
-	 */
-	public MongoMappingEvent(T source, DBObject dbo, String collectionName) {
-
 		super(source);
 		this.dbo = dbo;
-		this.collectionName = collectionName;
 	}
 
-	/**
-	 * @return {@literal null} if not set.
-	 */
 	public DBObject getDBObject() {
 		return dbo;
 	}
 
-	/**
-	 * Get the collection the event refers to.
-	 * 
-	 * @return {@literal null} if not set.
-	 * @since 1.8
-	 */
-	public String getCollectionName() {
-		return collectionName;
-	}
-
-	/*
-	 * (non-Javadoc)
-	 * @see java.util.EventObject#getSource()
-	 */
 	@SuppressWarnings({ "unchecked" })
 	@Override
 	public T getSource() {

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapreduce/GroupByResults.java

@@ -1,11 +1,11 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011 - 2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *      http://www.apache.org/licenses/LICENSE-2.0
+ *			http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -27,7 +27,6 @@ import com.mongodb.DBObject;
  * 
  * @author Mark Pollack
  * @author Christoph Strobl
- * @author Mark Paluch
  * @param <T> The class in which the results are mapped onto, accessible via an {@link Iterator}.
  */
 public class GroupByResults<T> implements Iterable<T> {
@@ -41,12 +40,10 @@ public class GroupByResults<T> implements Iterable<T> {
 
 	public GroupByResults(List<T> mappedResults, DBObject rawResults) {
 
-		Assert.notNull(mappedResults, "List of mapped results must not be null!");
-		Assert.notNull(rawResults, "Raw results must not be null!");
-
+		Assert.notNull(mappedResults);
+		Assert.notNull(rawResults);
 		this.mappedResults = mappedResults;
 		this.rawResults = rawResults;
-
 		parseKeys();
 		parseCount();
 		parseServerUsed();

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapreduce/MapReduceOptions.java

@@ -45,8 +45,6 @@ public class MapReduceOptions {
 
 	private Boolean verbose = true;
 
-	private Integer limit;
-
 	private Map<String, Object> extraOptions = new HashMap<String, Object>();
 
 	/**
@@ -66,8 +64,6 @@ public class MapReduceOptions {
 	 * @return MapReduceOptions so that methods can be chained in a fluent API style
 	 */
 	public MapReduceOptions limit(int limit) {
-
-		this.limit = limit;
 		return this;
 	}
 
@@ -251,15 +247,6 @@ public class MapReduceOptions {
 		return this.scopeVariables;
 	}
 
-	/**
-	 * Get the maximum number of documents for the input into the map function.
-	 * 
-	 * @return {@literal null} if not set.
-	 */
-	public Integer getLimit() {
-		return limit;
-	}
-
 	public DBObject getOptionsObject() {
 		BasicDBObject cmd = new BasicDBObject();
 
@@ -277,10 +264,6 @@ public class MapReduceOptions {
 			cmd.put("scope", scopeVariables);
 		}
 
-		if (limit != null) {
-			cmd.put("limit", limit);
-		}
-
 		if (!extraOptions.keySet().isEmpty()) {
 			cmd.putAll(extraOptions);
 		}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapreduce/MapReduceResults.java

@@ -1,11 +1,11 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *      http://www.apache.org/licenses/LICENSE-2.0
+ *			http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -29,7 +29,6 @@ import com.mongodb.MapReduceOutput;
  * @author Mark Pollack
  * @author Oliver Gierke
  * @author Christoph Strobl
- * @author Mark Paluch
  * @param <T> The class in which the results are mapped onto, accessible via an iterator.
  */
 public class MapReduceResults<T> implements Iterable<T> {
@@ -50,8 +49,8 @@ public class MapReduceResults<T> implements Iterable<T> {
 	@Deprecated
 	public MapReduceResults(List<T> mappedResults, DBObject rawResults) {
 
-		Assert.notNull(mappedResults, "List of mapped results must not be null!");
-		Assert.notNull(rawResults, "Raw results must not be null!");
+		Assert.notNull(mappedResults);
+		Assert.notNull(rawResults);
 
 		this.mappedResults = mappedResults;
 		this.rawResults = rawResults;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/mapreduce/MapReduceTiming.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2015 the original author or authors.
+ * Copyright 2010-2011 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -17,10 +17,13 @@ package org.springframework.data.mongodb.core.mapreduce;
 
 public class MapReduceTiming {
 
-	private long mapTime, emitLoopTime, totalTime;
+	private long mapTime;
+
+	private long emitLoopTime;
+
+	private long totalTime;
 
 	public MapReduceTiming(long mapTime, long emitLoopTime, long totalTime) {
-
 		this.mapTime = mapTime;
 		this.emitLoopTime = emitLoopTime;
 		this.totalTime = totalTime;
@@ -38,52 +41,37 @@ public class MapReduceTiming {
 		return totalTime;
 	}
 
-	/*
-	 * (non-Javadoc)
-	 * @see java.lang.Object#toString()
-	 */
 	@Override
 	public String toString() {
 		return "MapReduceTiming [mapTime=" + mapTime + ", emitLoopTime=" + emitLoopTime + ", totalTime=" + totalTime + "]";
 	}
 
-	/*
-	 * (non-Javadoc)
-	 * @see java.lang.Object#hashCode()
-	 */
 	@Override
 	public int hashCode() {
-
 		final int prime = 31;
 		int result = 1;
-
 		result = prime * result + (int) (emitLoopTime ^ (emitLoopTime >>> 32));
 		result = prime * result + (int) (mapTime ^ (mapTime >>> 32));
 		result = prime * result + (int) (totalTime ^ (totalTime >>> 32));
-
 		return result;
 	}
 
-	/*
-	 * 
-	 * (non-Javadoc)
-	 * @see java.lang.Object#equals(java.lang.Object)
-	 */
 	@Override
 	public boolean equals(Object obj) {
-
-		if (this == obj) {
+		if (this == obj)
 			return true;
-		}
-
-		if (!(obj instanceof MapReduceTiming)) {
+		if (obj == null)
 			return false;
-		}
-
-		MapReduceTiming that = (MapReduceTiming) obj;
-
-		return this.emitLoopTime == that.emitLoopTime && //
-				this.mapTime == that.mapTime && //
-				this.totalTime == that.totalTime;
+		if (getClass() != obj.getClass())
+			return false;
+		MapReduceTiming other = (MapReduceTiming) obj;
+		if (emitLoopTime != other.emitLoopTime)
+			return false;
+		if (mapTime != other.mapTime)
+			return false;
+		if (totalTime != other.totalTime)
+			return false;
+		return true;
 	}
+
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/BasicQuery.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2016 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -28,7 +28,6 @@ import com.mongodb.util.JSON;
  * @author Oliver Gierke
  * @author Christoph Strobl
  * @author Thomas Darimont
- * @author John Willemin
  */
 public class BasicQuery extends Query {
 
@@ -71,19 +70,6 @@ public class BasicQuery extends Query {
 
 	@Override
 	public DBObject getFieldsObject() {
-
-		if (fieldsObject == null) {
-			return super.getFieldsObject();
-		}
-
-		if (super.getFieldsObject() != null) {
-
-			DBObject combinedFieldsObject = new BasicDBObject();
-			combinedFieldsObject.putAll(fieldsObject);
-			combinedFieldsObject.putAll(super.getFieldsObject());
-			return combinedFieldsObject;
-		}
-
 		return fieldsObject;
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/Criteria.java

@@ -22,11 +22,9 @@ import java.util.Arrays;
 import java.util.Collection;
 import java.util.LinkedHashMap;
 import java.util.List;
-import java.util.Map.Entry;
 import java.util.regex.Pattern;
 
 import org.bson.BSON;
-import org.springframework.data.domain.Example;
 import org.springframework.data.geo.Circle;
 import org.springframework.data.geo.Point;
 import org.springframework.data.geo.Shape;
@@ -50,7 +48,6 @@ import com.mongodb.DBObject;
  * @author Oliver Gierke
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class Criteria implements CriteriaDefinition {
 
@@ -90,30 +87,6 @@ public class Criteria implements CriteriaDefinition {
 		return new Criteria(key);
 	}
 
-	/**
-	 * Static factory method to create a {@link Criteria} matching an example object.
-	 * 
-	 * @param example must not be {@literal null}.
-	 * @return
-	 * @see Criteria#alike(Example)
-	 * @since 1.8
-	 */
-	public static Criteria byExample(Object example) {
-		return byExample(Example.of(example));
-	}
-
-	/**
-	 * Static factory method to create a {@link Criteria} matching an example object.
-	 * 
-	 * @param example must not be {@literal null}.
-	 * @return
-	 * @see Criteria#alike(Example)
-	 * @since 1.8
-	 */
-	public static Criteria byExample(Example<?> example) {
-		return new Criteria().alike(example);
-	}
-
 	/**
 	 * Static factory method to create a Criteria using the provided key
 	 * 
@@ -145,7 +118,7 @@ public class Criteria implements CriteriaDefinition {
 	}
 
 	private boolean lastOperatorWasNot() {
-		return !this.criteria.isEmpty() && "$not".equals(this.criteria.keySet().toArray()[this.criteria.size() - 1]);
+		return this.criteria.size() > 0 && "$not".equals(this.criteria.keySet().toArray()[this.criteria.size() - 1]);
 	}
 
 	/**
@@ -388,7 +361,7 @@ public class Criteria implements CriteriaDefinition {
 	 */
 	public Criteria regex(Pattern pattern) {
 
-		Assert.notNull(pattern, "Pattern must not be null!");
+		Assert.notNull(pattern);
 
 		if (lastOperatorWasNot()) {
 			return not(pattern);
@@ -399,9 +372,7 @@ public class Criteria implements CriteriaDefinition {
 	}
 
 	private Pattern toPattern(String regex, String options) {
-
-		Assert.notNull(regex, "Regex string must not be null!");
-
+		Assert.notNull(regex);
 		return Pattern.compile(regex, options == null ? 0 : BSON.regexFlags(options));
 	}
 
@@ -415,9 +386,7 @@ public class Criteria implements CriteriaDefinition {
 	 * @return
 	 */
 	public Criteria withinSphere(Circle circle) {
-
-		Assert.notNull(circle, "Circle must not be null!");
-
+		Assert.notNull(circle);
 		criteria.put("$geoWithin", new GeoCommand(new Sphere(circle)));
 		return this;
 	}
@@ -431,8 +400,7 @@ public class Criteria implements CriteriaDefinition {
 	 */
 	public Criteria within(Shape shape) {
 
-		Assert.notNull(shape, "Shape must not be null!");
-
+		Assert.notNull(shape);
 		criteria.put("$geoWithin", new GeoCommand(shape));
 		return this;
 	}
@@ -445,9 +413,7 @@ public class Criteria implements CriteriaDefinition {
 	 * @return
 	 */
 	public Criteria near(Point point) {
-
-		Assert.notNull(point, "Point must not be null!");
-
+		Assert.notNull(point);
 		criteria.put("$near", point);
 		return this;
 	}
@@ -461,31 +427,13 @@ public class Criteria implements CriteriaDefinition {
 	 * @return
 	 */
 	public Criteria nearSphere(Point point) {
-
-		Assert.notNull(point, "Point must not be null!");
-
+		Assert.notNull(point);
 		criteria.put("$nearSphere", point);
 		return this;
 	}
 
 	/**
-	 * Creates criterion using {@code $geoIntersects} operator which matches intersections of the given {@code geoJson}
-	 * structure and the documents one. Requires MongoDB 2.4 or better.
-	 * 
-	 * @param geoJson must not be {@literal null}.
-	 * @return
-	 * @since 1.8
-	 */
-	@SuppressWarnings("rawtypes")
-	public Criteria intersects(GeoJson geoJson) {
-
-		Assert.notNull(geoJson, "GeoJson must not be null!");
-		criteria.put("$geoIntersects", geoJson);
-		return this;
-	}
-
-	/**
-	 * Creates a geo-spatial criterion using a {@literal $maxDistance} operation, for use with $near
+	 * Creates a geospatical criterion using a {@literal $maxDistance} operation, for use with $near
 	 * 
 	 * @see http://docs.mongodb.org/manual/reference/operator/query/maxDistance/
 	 * @param maxDistance
@@ -533,20 +481,6 @@ public class Criteria implements CriteriaDefinition {
 		return this;
 	}
 
-	/**
-	 * Creates a criterion using the given object as a pattern.
-	 * 
-	 * @param sample
-	 * @return
-	 * @since 1.8
-	 */
-	public Criteria alike(Example<?> sample) {
-
-		criteria.put("$sample", sample);
-		this.criteriaChain.add(this);
-		return this;
-	}
-
 	/**
 	 * Creates an 'or' criteria using the $or operator for all of the provided criteria
 	 * <p>
@@ -631,10 +565,9 @@ public class Criteria implements CriteriaDefinition {
 		DBObject dbo = new BasicDBObject();
 		boolean not = false;
 
-		for (Entry<String, Object> entry : criteria.entrySet()) {
+		for (String k : this.criteria.keySet()) {
 
-			String key = entry.getKey();
-			Object value = entry.getValue();
+			Object value = this.criteria.get(k);
 
 			if (requiresGeoJsonFormat(value)) {
 				value = new BasicDBObject("$geometry", value);
@@ -642,14 +575,14 @@ public class Criteria implements CriteriaDefinition {
 
 			if (not) {
 				DBObject notDbo = new BasicDBObject();
-				notDbo.put(key, value);
+				notDbo.put(k, value);
 				dbo.put("$not", notDbo);
 				not = false;
 			} else {
-				if ("$not".equals(key) && value == null) {
+				if ("$not".equals(k) && value == null) {
 					not = true;
 				} else {
-					dbo.put(key, value);
+					dbo.put(k, value);
 				}
 			}
 		}
@@ -717,7 +650,7 @@ public class Criteria implements CriteriaDefinition {
 		return false;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Object#equals(java.lang.Object)
 	 */
@@ -780,7 +713,7 @@ public class Criteria implements CriteriaDefinition {
 		return ObjectUtils.nullSafeEquals(left, right);
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Object#hashCode()
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/Field.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2015 the original author or authors.
+ * Copyright 2010-2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -83,10 +83,14 @@ public class Field {
 
 	public DBObject getFieldsObject() {
 
-		DBObject dbo = new BasicDBObject(criteria);
+		DBObject dbo = new BasicDBObject();
 
-		for (Entry<String, Object> entry : slices.entrySet()) {
-			dbo.put(entry.getKey(), new BasicDBObject("$slice", entry.getValue()));
+		for (String k : criteria.keySet()) {
+			dbo.put(k, criteria.get(k));
+		}
+
+		for (String k : slices.keySet()) {
+			dbo.put(k, new BasicDBObject("$slice", slices.get(k)));
 		}
 
 		for (Entry<String, Criteria> entry : elemMatchs.entrySet()) {
@@ -130,8 +134,8 @@ public class Field {
 			return false;
 		}
 
-		boolean samePositionKey = this.postionKey == null ? that.postionKey == null
-				: this.postionKey.equals(that.postionKey);
+		boolean samePositionKey = this.postionKey == null ? that.postionKey == null : this.postionKey
+				.equals(that.postionKey);
 		boolean samePositionValue = this.positionValue == that.positionValue;
 
 		return samePositionKey && samePositionValue;

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/MongoRegexCreator.java

@@ -1,106 +0,0 @@
-/*
- * Copyright 2015-2016 the original author or authors.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.springframework.data.mongodb.core.query;
-
-import java.util.regex.Pattern;
-
-import org.springframework.data.repository.query.parser.Part.Type;
-import org.springframework.util.ObjectUtils;
-
-/**
- * @author Christoph Strobl
- * @author Mark Paluch
- * @since 1.8
- */
-public enum MongoRegexCreator {
-
-	INSTANCE;
-
-	private static final Pattern PUNCTATION_PATTERN = Pattern.compile("\\p{Punct}");
-
-	/**
-	 * Creates a regular expression String to be used with {@code $regex}.
-	 * 
-	 * @param source the plain String
-	 * @param type
-	 * @return {@literal source} when {@literal source} or {@literal type} is {@literal null}.
-	 */
-	public String toRegularExpression(String source, Type type) {
-
-		if (type == null || source == null) {
-			return source;
-		}
-
-		String regex = prepareAndEscapeStringBeforeApplyingLikeRegex(source, type);
-
-		switch (type) {
-			case STARTING_WITH:
-				regex = "^" + regex;
-				break;
-			case ENDING_WITH:
-				regex = regex + "$";
-				break;
-			case CONTAINING:
-			case NOT_CONTAINING:
-				regex = ".*" + regex + ".*";
-				break;
-			case SIMPLE_PROPERTY:
-			case NEGATING_SIMPLE_PROPERTY:
-				regex = "^" + regex + "$";
-			default:
-		}
-
-		return regex;
-	}
-
-	private String prepareAndEscapeStringBeforeApplyingLikeRegex(String source, Type type) {
-
-		if (ObjectUtils.nullSafeEquals(Type.REGEX, type)) {
-			return source;
-		}
-
-		if (!ObjectUtils.nullSafeEquals(Type.LIKE, type)) {
-			return PUNCTATION_PATTERN.matcher(source).find() ? Pattern.quote(source) : source;
-		}
-
-		if (source.equals("*")) {
-			return ".*";
-		}
-
-		StringBuilder sb = new StringBuilder();
-
-		boolean leadingWildcard = source.startsWith("*");
-		boolean trailingWildcard = source.endsWith("*");
-
-		String valueToUse = source.substring(leadingWildcard ? 1 : 0,
-				trailingWildcard ? source.length() - 1 : source.length());
-
-		if (PUNCTATION_PATTERN.matcher(valueToUse).find()) {
-			valueToUse = Pattern.quote(valueToUse);
-		}
-
-		if (leadingWildcard) {
-			sb.append(".*");
-		}
-		sb.append(valueToUse);
-		if (trailingWildcard) {
-			sb.append(".*");
-		}
-
-		return sb.toString();
-	}
-
-}

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/NearQuery.java

@@ -1,11 +1,11 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
  *
- *      http://www.apache.org/licenses/LICENSE-2.0
+ *			http://www.apache.org/licenses/LICENSE-2.0
  *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
@@ -49,11 +49,11 @@ public final class NearQuery {
 	/**
 	 * Creates a new {@link NearQuery}.
 	 * 
-	 * @param point must not be {@literal null}.
+	 * @param point
 	 */
 	private NearQuery(Point point, Metric metric) {
 
-		Assert.notNull(point, "Point must not be null!");
+		Assert.notNull(point);
 
 		this.point = point;
 		this.spherical = false;
@@ -108,6 +108,7 @@ public final class NearQuery {
 	 * @return
 	 */
 	public static NearQuery near(Point point, Metric metric) {
+		Assert.notNull(point);
 		return new NearQuery(point, metric);
 	}
 
@@ -184,8 +185,7 @@ public final class NearQuery {
 	 */
 	public NearQuery maxDistance(double maxDistance, Metric metric) {
 
-		Assert.notNull(metric, "Metric must not be null!");
-
+		Assert.notNull(metric);
 		return maxDistance(new Distance(maxDistance, metric));
 	}
 
@@ -198,7 +198,7 @@ public final class NearQuery {
 	 */
 	public NearQuery maxDistance(Distance distance) {
 
-		Assert.notNull(distance, "Distance must not be null!");
+		Assert.notNull(distance);
 
 		if (distance.getMetric() != Metrics.NEUTRAL) {
 			this.spherical(true);
@@ -241,8 +241,7 @@ public final class NearQuery {
 	 */
 	public NearQuery minDistance(double minDistance, Metric metric) {
 
-		Assert.notNull(metric, "Metric must not be null!");
-
+		Assert.notNull(metric);
 		return minDistance(new Distance(minDistance, metric));
 	}
 
@@ -256,7 +255,7 @@ public final class NearQuery {
 	 */
 	public NearQuery minDistance(Distance distance) {
 
-		Assert.notNull(distance, "Distance must not be null!");
+		Assert.notNull(distance);
 
 		if (distance.getMetric() != Metrics.NEUTRAL) {
 			this.spherical(true);

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/Query.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2017 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -58,7 +58,7 @@ public class Query {
 
 	/**
 	 * Static factory method to create a {@link Query} using the provided {@link CriteriaDefinition}.
-	 *
+	 * 
 	 * @param criteriaDefinition must not be {@literal null}.
 	 * @return
 	 * @since 1.6
@@ -71,7 +71,7 @@ public class Query {
 
 	/**
 	 * Creates a new {@link Query} using the given {@link CriteriaDefinition}.
-	 *
+	 * 
 	 * @param criteriaDefinition must not be {@literal null}.
 	 * @since 1.6
 	 */
@@ -81,7 +81,7 @@ public class Query {
 
 	/**
 	 * Adds the given {@link CriteriaDefinition} to the current {@link Query}.
-	 *
+	 * 
 	 * @param criteriaDefinition must not be {@literal null}.
 	 * @return
 	 * @since 1.6
@@ -94,9 +94,9 @@ public class Query {
 		if (existing == null) {
 			this.criteria.put(key, criteriaDefinition);
 		} else {
-			throw new InvalidMongoDbApiUsageException(
-					String.format("Due to limitations of the com.mongodb.BasicDBObject, you can't add a second '%s' criteria. "
-							+ "Query already contains '%s'", key, serializeToJsonSafely(existing.getCriteriaObject())));
+			throw new InvalidMongoDbApiUsageException("Due to limitations of the com.mongodb.BasicDBObject, "
+					+ "you can't add a second '" + key + "' criteria. " + "Query already contains '"
+					+ existing.getCriteriaObject() + "'.");
 		}
 
 		return this;
@@ -111,7 +111,7 @@ public class Query {
 
 	/**
 	 * Set number of documents to skip before returning results.
-	 *
+	 * 
 	 * @param skip
 	 * @return
 	 */
@@ -122,7 +122,7 @@ public class Query {
 
 	/**
 	 * Limit the number of returned documents to {@code limit}.
-	 *
+	 * 
 	 * @param limit
 	 * @return
 	 */
@@ -133,7 +133,7 @@ public class Query {
 
 	/**
 	 * Configures the query to use the given hint when being executed.
-	 *
+	 * 
 	 * @param name must not be {@literal null} or empty.
 	 * @return
 	 */
@@ -146,7 +146,7 @@ public class Query {
 	/**
 	 * Sets the given pagination information on the {@link Query} instance. Will transparently set {@code skip} and
 	 * {@code limit} as well as applying the {@link Sort} instance defined with the {@link Pageable}.
-	 *
+	 * 
 	 * @param pageable
 	 * @return
 	 */
@@ -164,7 +164,7 @@ public class Query {
 
 	/**
 	 * Adds a {@link Sort} to the {@link Query} instance.
-	 *
+	 * 
 	 * @param sort
 	 * @return
 	 */
@@ -199,7 +199,7 @@ public class Query {
 
 	/**
 	 * Restricts the query to only return documents instances that are exactly of the given types.
-	 *
+	 * 
 	 * @param type may not be {@literal null}
 	 * @param additionalTypes may not be {@literal null}
 	 * @return
@@ -221,8 +221,10 @@ public class Query {
 
 		DBObject dbo = new BasicDBObject();
 
-		for (CriteriaDefinition definition : criteria.values()) {
-			dbo.putAll(definition.getCriteriaObject());
+		for (String k : criteria.keySet()) {
+			CriteriaDefinition c = criteria.get(k);
+			DBObject cl = c.getCriteriaObject();
+			dbo.putAll(cl);
 		}
 
 		if (!restrictedTypes.isEmpty()) {
@@ -253,7 +255,7 @@ public class Query {
 
 	/**
 	 * Get the number of documents to skip.
-	 *
+	 * 
 	 * @return
 	 */
 	public int getSkip() {
@@ -262,7 +264,7 @@ public class Query {
 
 	/**
 	 * Get the maximum number of documents to be return.
-	 *
+	 * 
 	 * @return
 	 */
 	public int getLimit() {
@@ -388,7 +390,7 @@ public class Query {
 
 	/**
 	 * Tests whether the settings of the given {@link Query} are equal to this query.
-	 *
+	 * 
 	 * @param that
 	 * @return
 	 */
@@ -405,7 +407,7 @@ public class Query {
 		return criteriaEqual && fieldsEqual && sortEqual && hintEqual && skipEqual && limitEqual && metaEqual;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Object#hashCode()
 	 */
@@ -427,7 +429,7 @@ public class Query {
 
 	/**
 	 * Returns whether the given key is the one used to hold the type restriction information.
-	 *
+	 * 
 	 * @deprecated don't call this method as the restricted type handling will undergo some significant changes going
 	 *             forward.
 	 * @param key

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/SerializationUtils.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2016 the original author or authors.
+ * Copyright 2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,15 +16,12 @@
 package org.springframework.data.mongodb.core.query;
 
 import java.util.Collection;
-import java.util.Collections;
-import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Map.Entry;
 
 import org.springframework.core.convert.converter.Converter;
 
-import com.mongodb.BasicDBObject;
 import com.mongodb.DBObject;
 import com.mongodb.util.JSON;
 
@@ -32,7 +29,6 @@ import com.mongodb.util.JSON;
  * Utility methods for JSON serialization.
  * 
  * @author Oliver Gierke
- * @author Christoph Strobl
  */
 public abstract class SerializationUtils {
 
@@ -40,68 +36,6 @@ public abstract class SerializationUtils {
 
 	}
 
-	/**
-	 * Flattens out a given {@link DBObject}.
-	 * 
-	 * <pre>
-	 * <code>
-	 * {
-	 *   _id : 1
-	 *   nested : { value : "conflux"}
-	 * }
-	 * </code>
-	 * will result in 
-	 * <code>
-	 * {
-	 *   _id : 1
-	 *   nested.value : "conflux"
-	 * }
-	 * </code>
-	 * </pre>
-	 * 
-	 * @param source can be {@literal null}.
-	 * @return {@link Collections#emptyMap()} when source is {@literal null}
-	 * @since 1.8
-	 */
-	public static Map<String, Object> flattenMap(DBObject source) {
-
-		if (source == null) {
-			return Collections.emptyMap();
-		}
-
-		Map<String, Object> result = new HashMap<String, Object>();
-		toFlatMap("", source, result);
-		return result;
-	}
-
-	private static void toFlatMap(String currentPath, Object source, Map<String, Object> map) {
-
-		if (source instanceof BasicDBObject) {
-
-			BasicDBObject dbo = (BasicDBObject) source;
-			Iterator<Map.Entry<String, Object>> iter = dbo.entrySet().iterator();
-			String pathPrefix = currentPath.isEmpty() ? "" : currentPath + ".";
-
-			while (iter.hasNext()) {
-
-				Map.Entry<String, Object> entry = iter.next();
-
-				if (entry.getKey().startsWith("$")) {
-					if (map.containsKey(currentPath)) {
-						((BasicDBObject) map.get(currentPath)).put(entry.getKey(), entry.getValue());
-					} else {
-						map.put(currentPath, new BasicDBObject(entry.getKey(), entry.getValue()));
-					}
-				} else {
-
-					toFlatMap(pathPrefix + entry.getKey(), entry.getValue(), map);
-				}
-			}
-		} else {
-			map.put(currentPath, source);
-		}
-	}
-
 	/**
 	 * Serializes the given object into pseudo-JSON meaning it's trying to create a JSON representation as far as possible
 	 * but falling back to the given object's {@link Object#toString()} method if it's not serializable. Useful for

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/query/Update.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2015 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -254,7 +254,7 @@ public class Update {
 	 * @return
 	 */
 	public Update pullAll(String key, Object[] values) {
-		addMultiFieldOperation("$pullAll", key, Arrays.copyOf(values, values.length));
+		addFieldOperation("$pullAll", key, Arrays.copyOf(values, values.length));
 		return this;
 	}
 
@@ -327,22 +327,17 @@ public class Update {
 	}
 
 	public DBObject getUpdateObject() {
-		return new BasicDBObject(modifierOps);
+
+		DBObject dbo = new BasicDBObject();
+		for (String k : modifierOps.keySet()) {
+			dbo.put(k, modifierOps.get(k));
+		}
+		return dbo;
 	}
 
-	/**
-	 * This method is not called anymore rather override {@link #addMultiFieldOperation(String, String, Object)}.
-	 * 
-	 * @param operator
-	 * @param key
-	 * @param value
-	 * @deprectaed Use {@link #addMultiFieldOperation(String, String, Object)} instead.
-	 */
-	@Deprecated
 	protected void addFieldOperation(String operator, String key, Object value) {
 
 		Assert.hasText(key, "Key/Path for update must not be null or blank.");
-
 		modifierOps.put(operator, new BasicDBObject(key, value));
 		this.keysToUpdate.add(key);
 	}
@@ -360,8 +355,8 @@ public class Update {
 			if (existingValue instanceof BasicDBObject) {
 				keyValueMap = (BasicDBObject) existingValue;
 			} else {
-				throw new InvalidDataAccessApiUsageException(
-						"Modifier Operations should be a LinkedHashMap but was " + existingValue.getClass());
+				throw new InvalidDataAccessApiUsageException("Modifier Operations should be a LinkedHashMap but was "
+						+ existingValue.getClass());
 			}
 		}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/spel/ExpressionNode.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2017 the original author or authors.
+ * Copyright 2013 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -29,8 +29,6 @@ import org.springframework.util.Assert;
  * A value object for nodes in an expression. Allows iterating ove potentially available child {@link ExpressionNode}s.
  * 
  * @author Oliver Gierke
- * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class ExpressionNode implements Iterable<ExpressionNode> {
 
@@ -159,7 +157,7 @@ public class ExpressionNode implements Iterable<ExpressionNode> {
 	 */
 	public ExpressionNode getChild(int index) {
 
-		Assert.isTrue(index >= 0, "Index must be greater or equal to zero!");
+		Assert.isTrue(index >= 0);
 		return from(node.getChild(index), state);
 	}
 
@@ -185,7 +183,7 @@ public class ExpressionNode implements Iterable<ExpressionNode> {
 		return from(node, state);
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see java.lang.Iterable#iterator()
 	 */

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/gridfs/AntPath.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -24,7 +24,6 @@ import org.springframework.util.Assert;
  * Value object to abstract Ant paths.
  * 
  * @author Oliver Gierke
- * @author Mark Paluch
  */
 class AntPath {
 
@@ -39,9 +38,7 @@ class AntPath {
 	 * @param path must not be {@literal null}.
 	 */
 	public AntPath(String path) {
-
-		Assert.notNull(path, "Path must not be null!");
-
+		Assert.notNull(path);
 		this.path = path;
 	}
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/gridfs/GridFsTemplate.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2017 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -45,8 +45,6 @@ import com.mongodb.gridfs.GridFSInputFile;
  * @author Philipp Schneider
  * @author Thomas Darimont
  * @author Martin Baumgartner
- * @author Christoph Strobl
- * @author Mark Paluch
  */
 public class GridFsTemplate implements GridFsOperations, ResourcePatternResolver {
 
@@ -74,8 +72,8 @@ public class GridFsTemplate implements GridFsOperations, ResourcePatternResolver
 	 */
 	public GridFsTemplate(MongoDbFactory dbFactory, MongoConverter converter, String bucket) {
 
-		Assert.notNull(dbFactory, "MongoDbFactory must not be null!");
-		Assert.notNull(converter, "MongoConverter must not be null!");
+		Assert.notNull(dbFactory);
+		Assert.notNull(converter);
 
 		this.dbFactory = dbFactory;
 		this.converter = converter;
@@ -157,7 +155,7 @@ public class GridFsTemplate implements GridFsOperations, ResourcePatternResolver
 	 */
 	public GridFSFile store(InputStream content, String filename, String contentType, DBObject metadata) {
 
-		Assert.notNull(content, "InputStream must not be null!");
+		Assert.notNull(content);
 
 		GridFSInputFile file = getGridFs().createFile(content);
 
@@ -184,7 +182,7 @@ public class GridFsTemplate implements GridFsOperations, ResourcePatternResolver
 	public List<GridFSDBFile> find(Query query) {
 
 		if (query == null) {
-			return getGridFs().find(new BasicDBObject());
+			return getGridFs().find((DBObject) null);
 		}
 
 		DBObject queryObject = getMappedQuery(query.getQueryObject());

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/monitor/AbstractMonitor.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2015 the original author or authors.
+ * Copyright 2002-2012 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -29,7 +29,6 @@ import com.mongodb.MongoException;
  * Base class to encapsulate common configuration settings when connecting to a database
  * 
  * @author Mark Pollack
- * @author Oliver Gierke
  */
 public abstract class AbstractMonitor {
 

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/Meta.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2016 the original author or authors.
+ * Copyright 2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -25,11 +25,10 @@ import org.springframework.data.annotation.QueryAnnotation;
 
 /**
  * @author Christoph Strobl
- * @author Mark Paluch
  * @since 1.6
  */
 @Retention(RetentionPolicy.RUNTIME)
-@Target({ ElementType.METHOD, ElementType.ANNOTATION_TYPE })
+@Target(ElementType.METHOD)
 @Documented
 @QueryAnnotation
 public @interface Meta {

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/MongoRepository.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2010-2017 the original author or authors.
+ * Copyright 2010-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,23 +18,19 @@ package org.springframework.data.mongodb.repository;
 import java.io.Serializable;
 import java.util.List;
 
-import org.springframework.data.domain.Example;
 import org.springframework.data.domain.Sort;
 import org.springframework.data.repository.NoRepositoryBean;
 import org.springframework.data.repository.PagingAndSortingRepository;
-import org.springframework.data.repository.query.QueryByExampleExecutor;
 
 /**
  * Mongo specific {@link org.springframework.data.repository.Repository} interface.
- *
+ * 
  * @author Oliver Gierke
  * @author Christoph Strobl
  * @author Thomas Darimont
- * @author Mark Paluch
  */
 @NoRepositoryBean
-public interface MongoRepository<T, ID extends Serializable>
-		extends PagingAndSortingRepository<T, ID>, QueryByExampleExecutor<T> {
+public interface MongoRepository<T, ID extends Serializable> extends PagingAndSortingRepository<T, ID> {
 
 	/*
 	 * (non-Javadoc)
@@ -55,10 +51,10 @@ public interface MongoRepository<T, ID extends Serializable>
 	List<T> findAll(Sort sort);
 
 	/**
-	 * Inserts the given entity. Assumes the instance to be new to be able to apply insertion optimizations. Use
+	 * Inserts the given a given entity. Assumes the instance to be new to be able to apply insertion optimizations. Use
 	 * the returned instance for further operations as the save operation might have changed the entity instance
 	 * completely. Prefer using {@link #save(Object)} instead to avoid the usage of store-specific API.
-	 *
+	 * 
 	 * @param entity must not be {@literal null}.
 	 * @return the saved entity
 	 * @since 1.7
@@ -69,21 +65,10 @@ public interface MongoRepository<T, ID extends Serializable>
 	 * Inserts the given entities. Assumes the given entities to have not been persisted yet and thus will optimize the
 	 * insert over a call to {@link #save(Iterable)}. Prefer using {@link #save(Iterable)} to avoid the usage of store
 	 * specific API.
-	 *
+	 * 
 	 * @param entities must not be {@literal null}.
 	 * @return the saved entities
 	 * @since 1.7
 	 */
 	<S extends T> List<S> insert(Iterable<S> entities);
-
-	/* (non-Javadoc)
-	 * @see org.springframework.data.repository.query.QueryByExampleExecutor#findAll(org.springframework.data.domain.Example)
-	 */
-	<S extends T> List<S> findAll(Example<S> example);
-
-	/* (non-Javadoc)
-	 * @see org.springframework.data.repository.query.QueryByExampleExecutor#findAll(org.springframework.data.domain.Example, org.springframework.data.domain.Sort)
-	 */
-	<S extends T> List<S> findAll(Example<S> example, Sort sort);
-
 }

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/Query.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2016 the original author or authors.
+ * Copyright 2011-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,10 +30,9 @@ import org.springframework.data.annotation.QueryAnnotation;
  * @author Oliver Gierke
  * @author Thomas Darimont
  * @author Christoph Strobl
- * @author Mark Paluch
  */
 @Retention(RetentionPolicy.RUNTIME)
-@Target({ ElementType.METHOD, ElementType.ANNOTATION_TYPE })
+@Target(ElementType.METHOD)
 @Documented
 @QueryAnnotation
 public @interface Query {

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/repository/cdi/MongoRepositoryBean.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2017 the original author or authors.
+ * Copyright 2012-2014 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -53,11 +53,11 @@ public class MongoRepositoryBean<T> extends CdiRepositoryBean<T> {
 
 		super(qualifiers, repositoryType, beanManager, detector);
 
-		Assert.notNull(operations, "MongoOperations bean must not be null!");
+		Assert.notNull(operations);
 		this.operations = operations;
 	}
 
-	/*
+	/* 
 	 * (non-Javadoc)
 	 * @see org.springframework.data.repository.cdi.CdiRepositoryBean#create(javax.enterprise.context.spi.CreationalContext, java.lang.Class)
 	 */