Skip to content

Commit

Permalink
Source Iceberg PR - Build: Add missing @OverRide annotations (#3654)
Browse files Browse the repository at this point in the history
  • Loading branch information
Peter Vary committed Apr 1, 2022
1 parent ab3dae0 commit 237336d
Showing 1 changed file with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -415,6 +415,7 @@ public Map<String, String> properties() {
);
}

@Override
public String locationForCreateTableSQL(TableIdentifier identifier) {
return "LOCATION '" + warehouseLocation + TestTables.tablePath(identifier) + "' ";
}
Expand Down Expand Up @@ -498,23 +499,27 @@ private String getStringValueForInsert(Object value, Type type) {

enum TestTableType {
HADOOP_TABLE {
@Override
public TestTables instance(Configuration conf, TemporaryFolder temporaryFolder, String catalogName) {
return new HadoopTestTables(conf, temporaryFolder);
}
},
HADOOP_CATALOG {
@Override
public TestTables instance(Configuration conf, TemporaryFolder temporaryFolder, String catalogName)
throws IOException {
return new HadoopCatalogTestTables(conf, temporaryFolder, catalogName);
}
},
CUSTOM_CATALOG {
@Override
public TestTables instance(Configuration conf, TemporaryFolder temporaryFolder, String catalogName)
throws IOException {
return new CustomCatalogTestTables(conf, temporaryFolder, catalogName);
}
},
HIVE_CATALOG {
@Override
public TestTables instance(Configuration conf, TemporaryFolder temporaryFolder, String catalogName) {
return new HiveTestTables(conf, temporaryFolder, catalogName);
}
Expand Down

0 comments on commit 237336d

Please sign in to comment.