Fix for issue #1117 - Unique causes transaction abort

- fix error causing wrong column to be used for UNIQUE
- remove inappropriately verbose log message related to foreign keys

Author: pervazea

src/catalog/catalog.cpp

@@ -265,7 +265,6 @@ ResultType Catalog::CreateTable(const std::string &database_name,
         table_oid, column.GetName(), column_id, column.GetOffset(),
         column.GetType(), column.IsInlined(), column.GetConstraints(),
         pool_.get(), txn);
-    column_id++;
 
     // Create index on unique single column
     if (column.IsUnique()) {
@@ -276,6 +275,7 @@ ResultType Catalog::CreateTable(const std::string &database_name,
       LOG_DEBUG("Added a UNIQUE index on %s in %s.", col_name.c_str(),
                 table_name.c_str());
     }
+    column_id++;
   }
   CreatePrimaryIndex(database_object->GetDatabaseOid(), table_oid, txn);
   return ResultType::SUCCESS;

src/storage/data_table.cpp

@@ -591,7 +591,6 @@ bool DataTable::CheckForeignKeyConstraints(
 
       // The foreign key constraints only refer to the primary key
       if (index->GetIndexType() == IndexConstraintType::PRIMARY_KEY) {
-        LOG_INFO("BEGIN checking referred table");
 
         std::vector<oid_t> key_attrs = foreign_key->GetSourceColumnIds();
         std::unique_ptr<catalog::Schema> foreign_key_schema(