Add missing null check and @nullable

1. null check on variable `metadata` is missing in method `existsById`
2. `CrudMethodMetadata::getComment` should be `@Nullable`

Fix spring-projectsGH-2991

Author: quaff

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/CrudMethodMetadata.java

@@ -33,6 +33,7 @@
  * @author Mark Paluch
  * @author Jens Schauder
  * @author Greg Turnquist
+ * @author Yanming Zhou
  */
 public interface CrudMethodMetadata {
 
@@ -66,6 +67,7 @@ public interface CrudMethodMetadata {
 	 * @return
 	 * @since 3.0
 	 */
+	@Nullable
 	String getComment();
 
 	/**

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/CrudMethodMetadataPostProcessor.java

@@ -56,6 +56,7 @@
  * @author Christoph Strobl
  * @author Mark Paluch
  * @author Jens Schauder
+ * @author Yanming Zhou
  */
 class CrudMethodMetadataPostProcessor implements RepositoryProxyPostProcessor, BeanClassLoaderAware {
 
@@ -181,7 +182,7 @@ private static class DefaultCrudMethodMetadata implements CrudMethodMetadata {
 		private final @Nullable LockModeType lockModeType;
 		private final org.springframework.data.jpa.repository.support.QueryHints queryHints;
 		private final org.springframework.data.jpa.repository.support.QueryHints queryHintsForCount;
-		private final String comment;
+		private final @Nullable String comment;
 		private final Optional<EntityGraph> entityGraph;
 		private final Method method;
 

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java

@@ -373,7 +373,7 @@ public boolean existsById(ID id) {
 		Map<String, Object> hints = new HashMap<>();
 		getQueryHints().withFetchGraphs(em).forEach(hints::put);
 
-		if (metadata.getComment() != null && provider.getCommentHintKey() != null) {
+		if (metadata != null && metadata.getComment() != null && provider.getCommentHintKey() != null) {
 			hints.put(provider.getCommentHintKey(), provider.getCommentHintValue(metadata.getComment()));
 		}