Polishing.

Add Javadoc. Reformat code.

See: #359
Original pull request: #1385

Author: mp911de

spring-data-cassandra/src/main/java/org/springframework/data/cassandra/core/CassandraAdminOperations.java

@@ -55,7 +55,6 @@ public interface CassandraAdminOperations extends CassandraOperations {
 	void createTable(boolean ifNotExists, CqlIdentifier tableName, Class<?> entityClass,
 			Map<String, Object> optionsByName);
 
-
 	/**
 	 * Drops a table based on the given {@link Class entity type}. The name of the table is derived from either the simple
 	 * name of the {@link Class entity class} or name of the table specified with the {@link Table} mapping annotation.

spring-data-cassandra/src/main/java/org/springframework/data/cassandra/core/cql/keyspace/TableOption.java

@@ -18,7 +18,6 @@
 import java.util.Map;
 
 import org.springframework.lang.Nullable;
-import org.springframework.util.StringUtils;
 
 /**
  * Enumeration that represents all known table options. If a table option is not listed here, but is supported by
@@ -86,12 +85,22 @@ public enum TableOption implements Option {
 		this.delegate = new DefaultOption(name, type, requiresValue, escapesValue, quotesValue);
 	}
 
+	/**
+	 * Look up {@link TableOption} by name using case-insensitive lookups.
+	 *
+	 * @param optionName name of the option.
+	 * @return the option.
+	 * @throws IllegalArgumentException if the option cannot be determined.
+	 * @since 3.4.13
+	 */
 	public static TableOption valueOfIgnoreCase(String optionName) {
+
 		for (TableOption value : values()) {
 			if (value.getName().equalsIgnoreCase(optionName)) {
 				return value;
 			}
 		}
+
 		throw new IllegalArgumentException(String.format("Unable to recognize specified Table option '%s'", optionName));
 	}
 

spring-data-cassandra/src/main/java/org/springframework/data/cassandra/core/cql/keyspace/TableSpecification.java

@@ -296,7 +296,7 @@ protected T column(CqlIdentifier name, DataType type, Optional<PrimaryKeyType> o
 
 		this.columns.add(column);
 
-		if (optionalKeyType.isEmpty()) {
+		if (!optionalKeyType.isPresent()) {
 			this.nonKeyColumns.add(column);
 		}
 

spring-data-cassandra/src/test/java/org/springframework/data/cassandra/core/CassandraAdminTemplateIntegrationTests.java

@@ -19,9 +19,9 @@
 
 import java.time.LocalDate;
 import java.util.Collection;
+import java.util.HashMap;
 import java.util.Map;
 
-import org.assertj.core.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.data.annotation.Id;
@@ -66,24 +66,23 @@ private KeyspaceMetadata getKeyspaceMetadata() {
 		return getSession().getKeyspace().flatMap(metadata::getKeyspace).get();
 	}
 
-	@Test
-	void givenAdminTemplate_whenCreateTableWithOptions_ThenCreatedTableContainsTheseOptions() {
-		cassandraAdminTemplate.createTable(
-				true,
-				CqlIdentifier.fromCql("someTable"),
-				SomeTable.class,
-				Map.of(
-					TableOption.COMMENT.getName(), "This is comment for table",
-					TableOption.BLOOM_FILTER_FP_CHANCE.getName(), "0.3"
-				)
-		);
-
-		TableMetadata someTable = getKeyspaceMetadata().getTables().values().stream().findFirst().orElse(null);
-
-		Assertions.assertThat(someTable).isNotNull();
-		Assertions.assertThat(someTable.getOptions().get(CqlIdentifier.fromCql(TableOption.COMMENT.getName())))
+	@Test // GH-359
+	void shouldApplyTableOptions() {
+
+		Map<String, Object> options = new HashMap<>();
+
+		options.put(TableOption.COMMENT.getName(), "This is comment for table");
+		options.put(TableOption.BLOOM_FILTER_FP_CHANCE.getName(), "0.3");
+
+		CqlIdentifier tableName = CqlIdentifier.fromCql("someTable");
+		cassandraAdminTemplate.createTable(true, tableName, SomeTable.class, options);
+
+		TableMetadata someTable = getKeyspaceMetadata().getTables().get(tableName);
+
+		assertThat(someTable).isNotNull();
+		assertThat(someTable.getOptions().get(CqlIdentifier.fromCql(TableOption.COMMENT.getName())))
 				.isEqualTo("This is comment for table");
-		Assertions.assertThat(someTable.getOptions().get(CqlIdentifier.fromCql(TableOption.BLOOM_FILTER_FP_CHANCE.getName())))
+		assertThat(someTable.getOptions().get(CqlIdentifier.fromCql(TableOption.BLOOM_FILTER_FP_CHANCE.getName())))
 				.isEqualTo(0.3);
 	}
 
@@ -117,8 +116,7 @@ void testDropTable() {
 	@Table("someTable")
 	private static class SomeTable {
 
-		@Id
-		private String name;
+		@Id private String name;
 		private Integer number;
 		private LocalDate createdAt;
 	}