Fix test pattern inconsistencies

- Remove @nullable from NullableJavaTimePOJO constructor params
  (matching existing NullablePOJO pattern)
- Use getLogicalTypeValue() instead of getValue() in nullable
  toRow test (matching testJavaTimeToRow pattern)
- Add testJavaTimeToRow/testJavaTimeFromRow to JavaBeanSchemaTest
  (matching existing testToRow/testFromRow coverage)

Author: sachinnn99

sdks/java/core/src/test/java/org/apache/beam/sdk/schemas/JavaBeanSchemaTest.java

@@ -203,6 +203,49 @@ public void testJavaTimeSchema() throws NoSuchSchemaException {
     assertTrue(schema.assignableToIgnoreNullable(icebergStyleSchema));
   }
 
+  @Test
+  public void testJavaTimeToRow() throws NoSuchSchemaException {
+    SchemaRegistry registry = SchemaRegistry.createDefault();
+    JavaTimeBean bean = new JavaTimeBean();
+    bean.setLocalDate(LocalDate.of(2024, 1, 15));
+    bean.setLocalTime(LocalTime.of(10, 30, 45));
+    bean.setLocalDateTime(LocalDateTime.of(2024, 1, 15, 10, 30, 45));
+    bean.setInstant(java.time.Instant.ofEpochSecond(1_705_315_845L, 123_456_789L));
+    bean.setUuid(UUID.fromString("11111111-2222-3333-4444-555555555555"));
+
+    Row row = registry.getToRowFunction(JavaTimeBean.class).apply(bean);
+
+    assertEquals(5, row.getFieldCount());
+    assertEquals(bean.getLocalDate(), row.getLogicalTypeValue("localDate", LocalDate.class));
+    assertEquals(bean.getLocalTime(), row.getLogicalTypeValue("localTime", LocalTime.class));
+    assertEquals(
+        bean.getLocalDateTime(), row.getLogicalTypeValue("localDateTime", LocalDateTime.class));
+    assertEquals(bean.getInstant(), row.getLogicalTypeValue("instant", java.time.Instant.class));
+    assertEquals(bean.getUuid(), row.getLogicalTypeValue("uuid", UUID.class));
+  }
+
+  @Test
+  public void testJavaTimeFromRow() throws NoSuchSchemaException {
+    SchemaRegistry registry = SchemaRegistry.createDefault();
+    LocalDate localDate = LocalDate.of(2024, 1, 15);
+    LocalTime localTime = LocalTime.of(10, 30, 45);
+    LocalDateTime localDateTime = LocalDateTime.of(2024, 1, 15, 10, 30, 45);
+    java.time.Instant instant = java.time.Instant.ofEpochSecond(1_705_315_845L, 123_456_789L);
+    UUID uuid = UUID.fromString("11111111-2222-3333-4444-555555555555");
+    Row row =
+        Row.withSchema(JAVA_TIME_BEAN_SCHEMA)
+            .addValues(localDate, localTime, localDateTime, instant, uuid)
+            .build();
+
+    JavaTimeBean bean = registry.getFromRowFunction(JavaTimeBean.class).apply(row);
+
+    assertEquals(localDate, bean.getLocalDate());
+    assertEquals(localTime, bean.getLocalTime());
+    assertEquals(localDateTime, bean.getLocalDateTime());
+    assertEquals(instant, bean.getInstant());
+    assertEquals(uuid, bean.getUuid());
+  }
+
   @Test
   public void testJavaTimeRoundTrip() throws NoSuchSchemaException {
     SchemaRegistry registry = SchemaRegistry.createDefault();

sdks/java/core/src/test/java/org/apache/beam/sdk/schemas/JavaFieldSchemaTest.java

@@ -330,11 +330,11 @@ public void testNullableJavaTimeToRow() throws NoSuchSchemaException {
     Row row = registry.getToRowFunction(NullableJavaTimePOJO.class).apply(pojo);
 
     assertEquals(5, row.getFieldCount());
-    assertNull(row.getValue("localDate"));
-    assertNull(row.getValue("localTime"));
-    assertNull(row.getValue("localDateTime"));
-    assertNull(row.getValue("instant"));
-    assertNull(row.getValue("uuid"));
+    assertNull(row.getLogicalTypeValue("localDate", LocalDate.class));
+    assertNull(row.getLogicalTypeValue("localTime", LocalTime.class));
+    assertNull(row.getLogicalTypeValue("localDateTime", LocalDateTime.class));
+    assertNull(row.getLogicalTypeValue("instant", java.time.Instant.class));
+    assertNull(row.getLogicalTypeValue("uuid", UUID.class));
   }
 
   @Test

sdks/java/core/src/test/java/org/apache/beam/sdk/schemas/utils/TestPOJOs.java

@@ -1349,11 +1349,11 @@ public static class NullableJavaTimePOJO {
     public NullableJavaTimePOJO() {}
 
     public NullableJavaTimePOJO(
-        @Nullable LocalDate localDate,
-        @Nullable LocalTime localTime,
-        @Nullable LocalDateTime localDateTime,
-        java.time.@Nullable Instant instant,
-        @Nullable UUID uuid) {
+        LocalDate localDate,
+        LocalTime localTime,
+        LocalDateTime localDateTime,
+        java.time.Instant instant,
+        UUID uuid) {
       this.localDate = localDate;
       this.localTime = localTime;
       this.localDateTime = localDateTime;