renamed partition() -> partitioned(), primary() -> clustered() as appropriate

This commit is contained in:
Matthew Adams
2013-12-12 11:18:06 -06:00
parent 3c04ddf008
commit 4a1b5fa543
2 changed files with 18 additions and 18 deletions

View File

@@ -65,14 +65,14 @@ public class CreateTableCqlGenerator extends TableCqlGenerator<CreateTableSpecif
cql.append(" (");
List<ColumnSpecification> partitionKeys = new ArrayList<ColumnSpecification>();
List<ColumnSpecification> clusteredKeys = new ArrayList<ColumnSpecification>();
List<ColumnSpecification> clusterKeys = new ArrayList<ColumnSpecification>();
for (ColumnSpecification col : spec().getColumns()) {
col.toCql(cql).append(", ");
if (col.getKeyType() == PARTITIONED) {
partitionKeys.add(col);
} else if (col.getKeyType() == CLUSTERED) {
clusteredKeys.add(col);
clusterKeys.add(col);
}
}
@@ -91,11 +91,11 @@ public class CreateTableCqlGenerator extends TableCqlGenerator<CreateTableSpecif
// end partition key clause
}
if (!clusteredKeys.isEmpty()) {
if (!clusterKeys.isEmpty()) {
cql.append(", ");
}
appendColumnNames(cql, clusteredKeys);
appendColumnNames(cql, clusterKeys);
cql.append(")");
// end primary key clause
@@ -103,7 +103,7 @@ public class CreateTableCqlGenerator extends TableCqlGenerator<CreateTableSpecif
cql.append(")");
// end columns
StringBuilder ordering = createOrderingClause(clusteredKeys);
StringBuilder ordering = createOrderingClause(clusterKeys);
// begin options
// begin option clause
Map<String, Object> options = spec().getOptions();

View File

@@ -31,9 +31,9 @@ import com.datastax.driver.core.DataType;
* Builder class to help construct CQL statements that involve column manipulation. Not threadsafe.
* <p/>
* Use {@link #name(String)} and {@link #type(String)} to set the name and type of the column, respectively. To specify
* a <code>PRIMARY KEY</code> column, use {@link #primary()} or {@link #primary(Ordering)}. To specify that the
* <code>PRIMARY KEY</code> column is or is part of the partition key, use {@link #partition()} instead of
* {@link #primary()} or {@link #primary(Ordering)}.
* a <code>PRIMARY KEY</code> column, use {@link #clustered()} or {@link #clustered(Ordering)}. To specify that the
* <code>PRIMARY KEY</code> column is or is part of the partition key, use {@link #partitioned()} instead of
* {@link #clustered()} or {@link #clustered(Ordering)}.
*
* @author Matthew T. Adams
* @author Alex Shvid
@@ -77,8 +77,8 @@ public class ColumnSpecification {
*
* @return this
*/
public ColumnSpecification partition() {
return partition(true);
public ColumnSpecification partitioned() {
return partitioned(true);
}
/**
@@ -88,20 +88,20 @@ public class ColumnSpecification {
*
* @return this
*/
public ColumnSpecification partition(boolean partition) {
this.keyType = partition ? PARTITIONED : null;
public ColumnSpecification partitioned(boolean partitioned) {
this.keyType = partitioned ? PARTITIONED : null;
this.ordering = null;
return this;
}
/**
* Identifies this column as a primary key column with default ordering. Sets the column's {@link #keyType} to
* Identifies this column as a clustered key column with default ordering. Sets the column's {@link #keyType} to
* {@link PrimaryKeyType#CLUSTERED} and its {@link #ordering} to {@link #DEFAULT_ORDERING}.
*
* @return this
*/
public ColumnSpecification primary() {
return primary(DEFAULT_ORDERING);
public ColumnSpecification clustered() {
return clustered(DEFAULT_ORDERING);
}
/**
@@ -110,8 +110,8 @@ public class ColumnSpecification {
*
* @return this
*/
public ColumnSpecification primary(Ordering order) {
return primary(order, true);
public ColumnSpecification clustered(Ordering order) {
return clustered(order, true);
}
/**
@@ -121,7 +121,7 @@ public class ColumnSpecification {
*
* @return this
*/
public ColumnSpecification primary(Ordering order, boolean primary) {
public ColumnSpecification clustered(Ordering order, boolean primary) {
this.keyType = primary ? CLUSTERED : null;
this.ordering = primary ? order : null;
return this;