Skip to content

Commit c1cd281

Browse files
committed
Fix Beam to handle float32 arrays and pg schema.
This was missed in the earlier commits
1 parent 5e936f4 commit c1cd281

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

v1/src/main/java/org/apache/beam/sdk/io/gcp/spanner/MutationSizeEstimator.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,8 @@ private static long estimateArrayValue(Value v) {
142142
switch (v.getType().getArrayElementType().getCode()) {
143143
case BOOL:
144144
return v.getBoolArray().size();
145+
case FLOAT32:
146+
return 4L * v.getFloat32Array().size();
145147
case INT64:
146148
return 8L * v.getInt64Array().size();
147149
case FLOAT64:

v1/src/main/java/org/apache/beam/sdk/io/gcp/spanner/SpannerSchema.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -215,6 +215,9 @@ private static Type parseSpannerType(String spannerType, Dialect dialect) {
215215
if ("BIGINT".equals(spannerType)) {
216216
return Type.int64();
217217
}
218+
if ("REAL".equals(spannerType)) {
219+
return Type.float32();
220+
}
218221
if ("DOUBLE PRECISION".equals(spannerType)) {
219222
return Type.float64();
220223
}

0 commit comments

Comments
 (0)