Update target Beam version to 2.73.0 (#3760)

* Update target Beam version to 2.73.0

* follow validation PR change

* try fixing date logical type string

* attempt 2

* gemini suggestion

* spotless + regen

Author: jrmccluskey

pom.xml

@@ -51,8 +51,8 @@
     <jacoco.version>0.8.13</jacoco.version>
 
     <!-- Beam and linked versions -->
-    <beam.version>2.72.0</beam.version>
-    <beam-python.version>2.72.0</beam-python.version>
+    <beam.version>2.73.0</beam.version>
+    <beam-python.version>2.73.0</beam-python.version>
     <beam-maven-repo></beam-maven-repo>
 
     <!-- Common dependency versions -->

python/src/main/python/job-builder-server/requirements.txt

@@ -139,31 +139,25 @@ public static List<Object> sourceTextToTargetObjects(
             }
           case LOGICAL_TYPE:
             {
-              switch (type.getLogicalType().getIdentifier()) {
-                case NanosDuration.IDENTIFIER:
-                  castVals.add(asDuration(objVal));
-                  break;
-                case org.apache.beam.sdk.schemas.logicaltypes.Date.IDENTIFIER:
-                  castVals.add(asDate(objVal));
-                  break;
-                case org.apache.beam.sdk.schemas.logicaltypes.DateTime.IDENTIFIER:
-                  castVals.add(asDateTime(objVal, LocalDateTime::from));
-                  break;
-                case IsoDateTime.IDENTIFIER:
-                  castVals.add(asDateTime(objVal));
-                  break;
-                case Time.IDENTIFIER:
-                  castVals.add(asTime(objVal));
-                  break;
-                default:
-                  {
-                    var message =
-                        String.format(
-                            "Mapping '%s' types from text sources is not supported.", typeName);
-                    LOG.warn(message);
-                    castVals.add(null);
-                    break;
-                  }
+              String identifier = type.getLogicalType().getIdentifier();
+              if (NanosDuration.IDENTIFIER.equals(identifier)) {
+                castVals.add(asDuration(objVal));
+              } else if (org.apache.beam.sdk.schemas.logicaltypes.Date.IDENTIFIER.equals(
+                  identifier)) {
+                castVals.add(asDate(objVal));
+              } else if (org.apache.beam.sdk.schemas.logicaltypes.DateTime.IDENTIFIER.equals(
+                  identifier)) {
+                castVals.add(asDateTime(objVal, LocalDateTime::from));
+              } else if (IsoDateTime.IDENTIFIER.equals(identifier)) {
+                castVals.add(asDateTime(objVal));
+              } else if (Time.IDENTIFIER.equals(identifier)) {
+                castVals.add(asTime(objVal));
+              } else {
+                var message =
+                    String.format(
+                        "Mapping '%s' types from text sources is not supported.", typeName);
+                LOG.warn(message);
+                castVals.add(null);
               }
 
               break;