Merge branch 'master' into error-event

Author: ajakz

schemas-parse/src/main/java/org/cedar/schemas/parse/DefaultParser.java

@@ -13,7 +13,7 @@ public class DefaultParser {
    * Given a {@link ParsedRecord}, fill in any empty fields for which a default can be calculated. For instance
    * if the record's {@link Discovery} does not have a title, it will be set with the default, which is the the
    * {@link FileInformation#getName()} (see: {@link #defaultTitle(ParsedRecord.Builder)}).
-   * <br/>
+   * <br>
    * In general, only fields which have no value will be set with their defaults. The exception is the links,
    * which will have the defaults merged into the input values, with inputs taking precedence.
    * See: {@link #setDefaultLinks(ParsedRecord.Builder)}
@@ -278,15 +278,15 @@ private static boolean discoveryListEmpty(ParsedRecord.Builder builder, Function
 
   /**
    * <em>IMPORTANT!!</em>
-   * <br/>
+   * <br>
    * Apparently functions on an Avro builder which return a builder for a nested object,
    * for example {@link ParsedRecord.Builder#getDiscoveryBuilder}, are not always null-safe!
-   * <br/>
+   * <br>
    * If that nested object is nullable, and you built the {@code ParsedRecord} builder from another {@code ParsedRecord}
    * instance with {@link ParsedRecord#newBuilder(ParsedRecord)}, then it'll set {@code null} as the value
    * for the {@code Discovery} object, which will cause {@link ParsedRecord.Builder#hasDiscovery()} to return {@code true}
    * since the {@code Discovery} object was technically set (even though it was set to {@code null}).
-   * <br/>
+   * <br>
    * The result in that case is that {@link ParsedRecord.Builder#getDiscoveryBuilder()} throws a {@link NullPointerException}.
    * This function will check whether the {@code Discovery} is null and create a {@code Discovery.Builder} safely.
    *