Nixed Google Collections usage, fixed bug in uColSet checking.
git-svn-id: https://samskivert.googlecode.com/svn/trunk@2503 6335cc39-0255-0410-8fd6-9bcaacd3b74c
This commit is contained in:
@@ -21,12 +21,11 @@
|
|||||||
package com.samskivert.jdbc;
|
package com.samskivert.jdbc;
|
||||||
|
|
||||||
import java.util.Arrays;
|
import java.util.Arrays;
|
||||||
|
import java.util.Collections;
|
||||||
|
import java.util.HashSet;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
import java.util.Set;
|
import java.util.Set;
|
||||||
|
|
||||||
import com.google.common.base.Preconditions;
|
|
||||||
import com.google.common.collect.Sets;
|
|
||||||
|
|
||||||
import java.sql.Connection;
|
import java.sql.Connection;
|
||||||
import java.sql.ResultSet;
|
import java.sql.ResultSet;
|
||||||
import java.sql.Statement;
|
import java.sql.Statement;
|
||||||
@@ -124,11 +123,12 @@ public class HsqldbLiaison extends BaseLiaison
|
|||||||
String[][] uniqueConstraintColumns, String[] primaryKeyColumns)
|
String[][] uniqueConstraintColumns, String[] primaryKeyColumns)
|
||||||
throws SQLException
|
throws SQLException
|
||||||
{
|
{
|
||||||
Preconditions.checkArgument(columns.length == definitions.length,
|
if (columns.length != definitions.length) {
|
||||||
"Column name and definition number mismatch");
|
throw new IllegalArgumentException("Column name and definition number mismatch");
|
||||||
|
}
|
||||||
|
|
||||||
// make a set of unique constraints already provided
|
// make a set of unique constraints already provided
|
||||||
Set<List<String>> uColSet = Sets.newHashSet();
|
Set<List<String>> uColSet = new HashSet<List<String>>();
|
||||||
if (uniqueConstraintColumns != null) {
|
if (uniqueConstraintColumns != null) {
|
||||||
for (String[] uCols : uniqueConstraintColumns) {
|
for (String[] uCols : uniqueConstraintColumns) {
|
||||||
uColSet.add(Arrays.asList(uCols));
|
uColSet.add(Arrays.asList(uCols));
|
||||||
@@ -146,7 +146,7 @@ public class HsqldbLiaison extends BaseLiaison
|
|||||||
def.type, def.nullable, false, def.defaultValue);
|
def.type, def.nullable, false, def.defaultValue);
|
||||||
// if a uniqueness constraint for this column was not in the
|
// if a uniqueness constraint for this column was not in the
|
||||||
// uniqueConstraintColumns parameter, add such an entry
|
// uniqueConstraintColumns parameter, add such an entry
|
||||||
if (!uColSet.contains(Sets.newHashSet(columns[ii]))) {
|
if (!uColSet.contains(Collections.singletonList(columns[ii]))) {
|
||||||
String[] newConstraint = new String[] { columns[ii] };
|
String[] newConstraint = new String[] { columns[ii] };
|
||||||
uniqueConstraintColumns = (uniqueConstraintColumns == null) ?
|
uniqueConstraintColumns = (uniqueConstraintColumns == null) ?
|
||||||
new String[][] { newConstraint } :
|
new String[][] { newConstraint } :
|
||||||
|
|||||||
Reference in New Issue
Block a user