Fix JavaDoc.

Self closing <p/> tags cause build failures now.

Closes #1059

Author: schauder

spring-data-relational/src/main/java/org/springframework/data/relational/core/conversion/DbAction.java

@@ -236,7 +236,7 @@ public String toString() {
 	 * <p>
 	 * Note that deletes for contained entities that reference the root are to be represented by separate
 	 * {@link DbAction}s.
-	 *
+	 * </p>
 	 * @param <T> type of the entity for which this represents a database interaction.
 	 */
 	final class DeleteRoot<T> implements DbAction<T> {
@@ -300,7 +300,7 @@ public String toString() {
 	 * <p>
 	 * Note that deletes for contained entities that reference the root are to be represented by separate
 	 * {@link DbAction}s.
-	 *
+	 * </p>
 	 * @param <T> type of the entity for which this represents a database interaction.
 	 */
 	final class DeleteAllRoot<T> implements DbAction<T> {
@@ -322,7 +322,6 @@ public String toString() {
 
 	/**
 	 * Represents an acquire lock statement for a aggregate root when only the ID is known.
-	 * <p>
 	 *
 	 * @param <T> type of the entity for which this represents a database interaction.
 	 */
@@ -392,7 +391,7 @@ interface WithDependingOn<T> extends WithPropertyPath<T>, WithEntity<T> {
 		 * Additional values to be set during insert or update statements.
 		 * <p>
 		 * Values come from parent entities but one might also add values manually.
-		 *
+		 * </p>
 		 * @return guaranteed to be not {@code null}.
 		 */
 		Map<PersistentPropertyPath<RelationalPersistentProperty>, Object> getQualifiers();

spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/Embedded.java

@@ -27,10 +27,11 @@
 
 /**
  * The annotation to configure a value object as embedded in the current table.
- * <p />
+ * <p>
  * Depending on the {@link OnEmpty value} of {@link #onEmpty()} the property is set to {@literal null} or an empty
  * instance in the case all embedded values are {@literal null} when reading from the result set.
- *
+ * </p>
+ * 
  * @author Bastian Wilhelm
  * @author Christoph Strobl
  * @since 1.1
@@ -42,9 +43,10 @@
 
 	/**
 	 * Set the load strategy for the embedded object if all contained fields yield {@literal null} values.
-	 * <p />
+	 * <p>
 	 * {@link Nullable @Embedded.Nullable} and {@link Empty @Embedded.Empty} offer shortcuts for this.
-	 *
+	 * </p>
+	 * 
 	 * @return never {@link} null.
 	 */
 	OnEmpty onEmpty();

spring-data-relational/src/main/java/org/springframework/data/relational/core/mapping/event/BeforeDeleteCallback.java

@@ -37,7 +37,7 @@ public interface BeforeDeleteCallback<T> extends EntityCallback<T> {
 	 * account for deleting. Only transient fields of the entity should be changed in this callback.
 	 *
 	 * @param aggregate the aggregate.
-	 * @param aggregateChange the associated {@link DefaultAggregateChange}.
+	 * @param aggregateChange the associated {@link MutableAggregateChange}.
 	 * @return the aggregate to be deleted.
 	 */
 	T onBeforeDelete(T aggregate, MutableAggregateChange<T> aggregateChange);

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/AssignValue.java

@@ -19,9 +19,9 @@
 
 /**
  * Assign a {@link Expression} to a {@link Column}.
- * <p/>
+ * <p>
  * Results in a rendered assignment: {@code <column> = <value>} (e.g. {@code col = 'foo'}.
- *
+ * </p>
  * @author Mark Paluch
  * @since 1.1
  */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/AsteriskFromTable.java

@@ -17,15 +17,16 @@
 
 /**
  * {@link Segment} to select all columns from a {@link Table}.
- * <p/>
- * * Renders to: {@code
+ * <p>
+ * Renders to: {@code
  *
 <table>
  * .*} as in {@code SELECT
  *
 <table>
  * .* FROM …}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @since 1.1
  * @see Table#asterisk()

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Between.java

@@ -19,9 +19,10 @@
 
 /**
  * BETWEEN {@link Condition} comparing between {@link Expression}s.
- * <p/>
+ * <p>
  * Results in a rendered condition: {@code <left> BETWEEN <begin> AND <end>}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @since 2.2
  */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Column.java

@@ -20,9 +20,10 @@
 
 /**
  * Column name within a {@code SELECT … FROM} clause.
- * <p/>
+ * <p>
  * Renders to: {@code <name>} or {@code <table(alias)>.<name>}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @author Jens Schauder
  * @since 1.1

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Comparison.java

@@ -19,9 +19,10 @@
 
 /**
  * Comparing {@link Condition} comparing two {@link Expression}s.
- * <p/>
+ * <p>
  * Results in a rendered condition: {@code <left> <comparator> <right>} (e.g. {@code col = 'predicate'}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @author Jens Schauder
  * @since 1.1
@@ -61,11 +62,13 @@ public static Comparison create(Expression leftColumnOrExpression, String compar
 
 	/**
 	 * Creates a new {@link Comparison} from simple {@literal StringP} arguments
-	 * @param unqualifiedColumnName gets turned in a {@link Expressions#just(String)} and is expected to be an unqualified unique column name but also could be an verbatim expression. Must not be {@literal null}.
+	 * 
+	 * @param unqualifiedColumnName gets turned in a {@link Expressions#just(String)} and is expected to be an unqualified
+	 *          unique column name but also could be an verbatim expression. Must not be {@literal null}.
 	 * @param comparator must not be {@literal null}.
 	 * @param rightValue is considered a {@link Literal}. Must not be {@literal null}.
-	 * @return a new {@literal Comparison} of the first with the third argument using the second argument as comparison operator. Guaranteed to be not {@literal null}.
-	 *
+	 * @return a new {@literal Comparison} of the first with the third argument using the second argument as comparison
+	 *         operator. Guaranteed to be not {@literal null}.
 	 * @since 2.3
 	 */
 	public static Comparison create(String unqualifiedColumnName, String comparator, Object rightValue) {

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/DeleteValidator.java

@@ -17,9 +17,10 @@
 
 /**
  * Validator for {@link Delete} statements.
- * <p/>
+ * <p>
  * Validates that all {@link Column}s using a table qualifier have a table import from the {@code FROM} clause.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @since 1.1
  */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Join.java

@@ -17,12 +17,13 @@
 
 /**
  * {@link Segment} for a {@code JOIN} declaration.
- * <p/>
+ * <p>
  * Renders to: {@code JOIN
  *
 <table>
  *  ON <condition>}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @since 1.1
  */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Like.java

@@ -19,9 +19,9 @@
 
 /**
  * LIKE {@link Condition} comparing two {@link Expression}s.
- * <p/>
+ * <p>
  * Results in a rendered condition: {@code <left> LIKE <right>}.
- *
+ * </p>
  * @author Mark Paluch
  * @since 1.1
  */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/SQL.java

@@ -22,9 +22,10 @@
 /**
  * Utility to create SQL {@link Segment}s. Typically used as entry point to the Statement Builder. Objects and dependent
  * objects created by the Query AST are immutable except for builders.
- * <p/>
+ * <p>
  * The Statement Builder API is intended for framework usage to produce SQL required for framework operations.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @author Jens Schauder
  * @since 1.1

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Segment.java

@@ -26,9 +26,10 @@ public interface Segment extends Visitable {
 
 	/**
 	 * Check whether this {@link Segment} is equal to another {@link Segment}.
-	 * <p/>
+	 * <p>
 	 * Equality is typically given if the {@link #toString()} representation matches.
-	 *
+	 * </p>
+	 * 
 	 * @param other the reference object with which to compare.
 	 * @return {@literal true} if this object is the same as the {@code other} argument; {@literal false} otherwise.
 	 */
@@ -37,22 +38,24 @@ public interface Segment extends Visitable {
 
 	/**
 	 * Generate a hash code from this{@link Segment}.
-	 * <p/>
+	 * <p>
 	 * Hashcode typically derives from the {@link #toString()} representation so two {@link Segment}s yield the same
 	 * {@link #hashCode()} if their {@link #toString()} representation matches.
-	 *
+	 * </p>
+	 * 
 	 * @return a hash code value for this object.
 	 */
 	@Override
 	int hashCode();
 
 	/**
 	 * Return a SQL string representation of this {@link Segment}.
-	 * <p/>
+	 * <p>
 	 * The representation is intended for debugging purposes and an approximation to the generated SQL. While it might
 	 * work in the context of a specific dialect, you should not that the {@link #toString()} representation works across
 	 * multiple databases.
-	 *
+	 * </p>
+	 * 
 	 * @return a SQL string representation of this {@link Segment}.
 	 */
 	@Override

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/SelectValidator.java

@@ -21,10 +21,10 @@
 
 /**
  * Validator for {@link Select} statements.
- * <p/>
+ * <p>
  * Validates that all {@link Column}s using a table qualifier have a table import from either the {@code FROM} or
  * {@code JOIN} clause.
- *
+ * </p>
  * @author Mark Paluch
  * @author Jens Schauder
  * @since 1.1

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/Table.java

@@ -25,9 +25,10 @@
 /**
  * Represents a table reference within an SQL statement. Typically used to denote {@code FROM} or {@code JOIN} or to
  * prefix a {@link Column}.
- * <p/>
+ * <p>
  * Renders to: {@code <name>} or {@code <name> AS <name>}.
- *
+ * </p>
+ * 
  * @author Mark Paluch
  * @since 1.1
  */
@@ -114,10 +115,10 @@ public Table as(SqlIdentifier alias) {
 
 	/**
 	 * Creates a new {@link Column} associated with this {@link Table}.
-	 * <p/>
+	 * <p>
 	 * Note: This {@link Table} does not track column creation and there is no possibility to enumerate all
 	 * {@link Column}s that were created for this table.
-	 *
+	 *</p>
 	 * @param name column name, must not be {@literal null} or empty.
 	 * @return a new {@link Column} associated with this {@link Table}.
 	 */
@@ -130,10 +131,10 @@ public Column column(String name) {
 
 	/**
 	 * Creates a new {@link Column} associated with this {@link Table}.
-	 * <p/>
+	 * <p>
 	 * Note: This {@link Table} does not track column creation and there is no possibility to enumerate all
 	 * {@link Column}s that were created for this table.
-	 *
+	 *</p>
 	 * @param name column name, must not be {@literal null} or empty.
 	 * @return a new {@link Column} associated with this {@link Table}.
 	 * @since 2.0
@@ -147,10 +148,10 @@ public Column column(SqlIdentifier name) {
 
 	/**
 	 * Creates a {@link List} of {@link Column}s associated with this {@link Table}.
-	 * <p/>
+	 * <p>
 	 * Note: This {@link Table} does not track column creation and there is no possibility to enumerate all
 	 * {@link Column}s that were created for this table.
-	 *
+	 *</p>
 	 * @param names column names, must not be {@literal null} or empty.
 	 * @return a new {@link List} of {@link Column}s associated with this {@link Table}.
 	 */
@@ -163,10 +164,10 @@ public List<Column> columns(String... names) {
 
 	/**
 	 * Creates a {@link List} of {@link Column}s associated with this {@link Table}.
-	 * <p/>
+	 * <p>
 	 * Note: This {@link Table} does not track column creation and there is no possibility to enumerate all
 	 * {@link Column}s that were created for this table.
-	 *
+	 *</p>
 	 * @param names column names, must not be {@literal null} or empty.
 	 * @return a new {@link List} of {@link Column}s associated with this {@link Table}.
 	 * @since 2.0
@@ -185,10 +186,10 @@ public List<Column> columns(SqlIdentifier... names) {
 
 	/**
 	 * Creates a {@link List} of {@link Column}s associated with this {@link Table}.
-	 * <p/>
+	 * <p>
 	 * Note: This {@link Table} does not track column creation and there is no possibility to enumerate all
 	 * {@link Column}s that were created for this table.
-	 *
+	 *</p>
 	 * @param names column names, must not be {@literal null} or empty.
 	 * @return a new {@link List} of {@link Column}s associated with this {@link Table}.
 	 */

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/UpdateBuilder.java

@@ -29,7 +29,7 @@ public interface UpdateBuilder {
 	/**
 	 * Configure the {@link Table} to which the update is applied.
 	 *
-	 * @param count the top count.
+	 * @param table the table to update.
 	 * @return {@code this} {@link SelectBuilder}.
 	 */
 	UpdateAssign table(Table table);

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/package-info.java

@@ -3,9 +3,10 @@
  * Statement Builder implementation. Use {@link org.springframework.data.relational.core.sql.StatementBuilder} to create
  * statements and {@link org.springframework.data.relational.core.sql.SQL} to create SQL objects. Objects and dependent
  * objects created by the Statement Builder are immutable except for builders.
- * <p/>
+ * <p>
  * The Statement Builder API is intended for framework usage to produce SQL required for framework operations.
- *
+ * </p>
+ * 
  * @since 1.1
  */
 @NonNullApi