Skip to content

Commit 226d19c

Browse files
committed
build fixes
1 parent abc124c commit 226d19c

File tree

6 files changed

+36
-13
lines changed

6 files changed

+36
-13
lines changed

fdb-relational-grpc/src/main/java/com/apple/foundationdb/relational/jdbc/RelationalArrayFacade.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@
4242
import java.sql.SQLException;
4343
import java.sql.SQLFeatureNotSupportedException;
4444
import java.sql.Types;
45+
import java.util.UUID;
4546

4647
/**
4748
* Facade over grpc protobuf objects that offers a {@link RelationalArray} view.
@@ -238,6 +239,11 @@ public RelationalArrayBuilder addLong(@Nonnull long value) throws SQLException {
238239
throw new SQLFeatureNotSupportedException();
239240
}
240241

242+
@Override
243+
public RelationalArrayBuilder addUuid(@Nonnull final UUID value) throws SQLException {
244+
throw new SQLFeatureNotSupportedException();
245+
}
246+
241247
@Override
242248
public RelationalArrayBuilder addStruct(RelationalStruct struct) throws SQLException {
243249
final var structFacade = struct.unwrap(RelationalStructFacade.class);

fdb-relational-grpc/src/main/java/com/apple/foundationdb/relational/jdbc/RelationalResultSetMetaDataFacade.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import com.apple.foundationdb.relational.api.StructMetaData;
2525
import com.apple.foundationdb.relational.api.RelationalResultSetMetaData;
2626
import com.apple.foundationdb.relational.api.exceptions.ErrorCode;
27+
import com.apple.foundationdb.relational.api.metadata.DataType;
2728
import com.apple.foundationdb.relational.jdbc.grpc.v1.ResultSetMetadata;
2829
import com.apple.foundationdb.relational.util.ExcludeFromJacocoGeneratedReport;
2930
import com.apple.foundationdb.relational.util.PositionalIndex;
@@ -47,6 +48,11 @@ public ArrayMetaData getArrayMetaData(int oneBasedColumn) throws SQLException {
4748
throw new SQLException("Not implemented", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());
4849
}
4950

51+
@Override
52+
public DataType.StructType getRelationalDataType() throws SQLException {
53+
throw new SQLException("Not implemented", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());
54+
}
55+
5056
@Override
5157
public StructMetaData getStructMetaData(int oneBasedColumn) throws SQLException {
5258
return new RelationalStructFacade.RelationalStructFacadeMetaData(

fdb-relational-grpc/src/main/java/com/apple/foundationdb/relational/jdbc/RelationalStructFacade.java

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import com.apple.foundationdb.relational.api.RelationalStruct;
2727
import com.apple.foundationdb.relational.api.RelationalStructBuilder;
2828
import com.apple.foundationdb.relational.api.exceptions.ErrorCode;
29+
import com.apple.foundationdb.relational.api.metadata.DataType;
2930
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.Column;
3031
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.ColumnMetadata;
3132
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.ListColumn;
@@ -420,11 +421,6 @@ public RelationalStructBuilder addBoolean(String fieldName, boolean b) throws SQ
420421
return this;
421422
}
422423

423-
@Override
424-
public RelationalStructBuilder addShort(String fieldName, short b) throws SQLException {
425-
throw new SQLException("Not implemented " + Thread.currentThread() .getStackTrace()[1] .getMethodName());
426-
}
427-
428424
@Override
429425
public RelationalStructBuilder addLong(String fieldName, long l) throws SQLException {
430426
int offset = addMetadata(ColumnMetadata.newBuilder()
@@ -465,7 +461,12 @@ public RelationalStructBuilder addString(String fieldName, @Nullable String s) t
465461
}
466462

467463
@Override
468-
public RelationalStructBuilder addObject(String fieldName, @Nullable Object obj, int targetSqlType) throws SQLException {
464+
public RelationalStructBuilder addUuid(final String fieldName, @Nullable final UUID uuid) throws SQLException {
465+
throw new SQLException("Not implemented " + Thread.currentThread() .getStackTrace()[1] .getMethodName());
466+
}
467+
468+
@Override
469+
public RelationalStructBuilder addObject(final String fieldName, @Nullable final Object obj) throws SQLException {
469470
throw new SQLException("Not implemented " + Thread.currentThread() .getStackTrace()[1] .getMethodName());
470471
}
471472

@@ -589,6 +590,11 @@ public int getLeadingPhantomColumnCount() {
589590
return -1000;
590591
}
591592

593+
@Override
594+
public DataType.StructType getRelationalDataType() throws SQLException {
595+
throw new SQLException("Not implemented", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());
596+
}
597+
592598
@Override
593599
public <T> T unwrap(Class<T> iface) throws SQLException {
594600
throw new SQLException("Not implemented", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());

fdb-relational-grpc/src/main/java/com/apple/foundationdb/relational/jdbc/TypeConversion.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,23 +21,22 @@
2121
package com.apple.foundationdb.relational.jdbc;
2222

2323
import com.apple.foundationdb.annotation.API;
24-
2524
import com.apple.foundationdb.relational.api.ArrayMetaData;
2625
import com.apple.foundationdb.relational.api.Continuation;
27-
import com.apple.foundationdb.relational.api.SqlTypeNamesSupport;
28-
import com.apple.foundationdb.relational.api.StructMetaData;
2926
import com.apple.foundationdb.relational.api.RelationalArray;
3027
import com.apple.foundationdb.relational.api.RelationalResultSet;
3128
import com.apple.foundationdb.relational.api.RelationalResultSetMetaData;
3229
import com.apple.foundationdb.relational.api.RelationalStruct;
3330
import com.apple.foundationdb.relational.api.RelationalStructMetaData;
31+
import com.apple.foundationdb.relational.api.StructMetaData;
3432
import com.apple.foundationdb.relational.api.exceptions.ErrorCode;
33+
import com.apple.foundationdb.relational.api.metadata.DataType;
3534
import com.apple.foundationdb.relational.jdbc.grpc.v1.KeySet;
3635
import com.apple.foundationdb.relational.jdbc.grpc.v1.KeySetValue;
3736
import com.apple.foundationdb.relational.jdbc.grpc.v1.ResultSet;
38-
import com.apple.foundationdb.relational.jdbc.grpc.v1.RpcContinuationReason;
3937
import com.apple.foundationdb.relational.jdbc.grpc.v1.ResultSetMetadata;
4038
import com.apple.foundationdb.relational.jdbc.grpc.v1.RpcContinuation;
39+
import com.apple.foundationdb.relational.jdbc.grpc.v1.RpcContinuationReason;
4140
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.Array;
4241
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.Column;
4342
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.ColumnMetadata;
@@ -335,7 +334,7 @@ public static Array toArray(@Nonnull java.sql.Array array) throws SQLException {
335334
* @throws SQLException in case of error
336335
*/
337336
public static Column toColumn(int columnType, @Nonnull Object obj) throws SQLException {
338-
if (columnType != SqlTypeNamesSupport.getSqlTypeCodeFromObject(obj)) {
337+
if (columnType != DataType.getDataTypeFromObject(obj).getJdbcSqlCode()) {
339338
throw new SQLException("Column element type does not match object type: " + columnType + " / " + obj.getClass().getSimpleName(),
340339
ErrorCode.WRONG_OBJECT_TYPE.getErrorCode());
341340
}

fdb-relational-grpc/src/test/java/com/apple/foundationdb/relational/jdbc/MockResultSetMetadata.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import com.apple.foundationdb.relational.api.StructMetaData;
2525
import com.apple.foundationdb.relational.api.RelationalResultSetMetaData;
2626
import com.apple.foundationdb.relational.api.exceptions.ErrorCode;
27+
import com.apple.foundationdb.relational.api.metadata.DataType;
2728

2829
import java.sql.SQLException;
2930
import java.util.List;
@@ -55,6 +56,11 @@ public ArrayMetaData getArrayMetaData(int oneBasedColumn) throws SQLException {
5556
throw new SQLException("Unsupported operation", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());
5657
}
5758

59+
@Override
60+
public DataType.StructType getRelationalDataType() throws SQLException {
61+
throw new SQLException("Unsupported operation", ErrorCode.UNSUPPORTED_OPERATION.getErrorCode());
62+
}
63+
5864
@Override
5965
public int getColumnCount() throws SQLException {
6066
return columnTypes.size();

fdb-relational-jdbc/src/main/java/com/apple/foundationdb/relational/jdbc/ParameterHelper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020

2121
package com.apple.foundationdb.relational.jdbc;
2222

23-
import com.apple.foundationdb.relational.api.SqlTypeNamesSupport;
2423
import com.apple.foundationdb.relational.api.exceptions.ErrorCode;
24+
import com.apple.foundationdb.relational.api.metadata.DataType;
2525
import com.apple.foundationdb.relational.jdbc.grpc.v1.Parameter;
2626
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.Column;
2727
import com.apple.foundationdb.relational.jdbc.grpc.v1.column.Uuid;
@@ -136,7 +136,7 @@ public static Parameter ofArray(final Array a) throws SQLException {
136136
}
137137

138138
public static Parameter ofObject(Object x) throws SQLException {
139-
final int typeCodeFromObject = SqlTypeNamesSupport.getSqlTypeCodeFromObject(x);
139+
final int typeCodeFromObject = DataType.getDataTypeFromObject(x).getJdbcSqlCode();
140140
switch (typeCodeFromObject) {
141141
case Types.BIGINT:
142142
return ofLong((Long)x);

0 commit comments

Comments
 (0)