#820

Committed 16 Apr 2026 04:09PM UTC coverage: 87.706% (-0.07%) from 87.774%

Build # #820

Build Type

push

Committed by

Commit Message

Upgrade to Java 21 and Spring Boot 3.2.12 (#201)

* chore: upgrade to Java 21 and Spring Boot 3.2.12

- Bump Java source/target/release to 21 across all modules
- Upgrade Spring Boot from 2.7.9 to 3.2.12
- Migrate javax.* to jakarta.* (persistence, servlet, annotation)
- Add CrudRepository to repository interfaces (Spring Data 3.x breaking change)
- Update springdoc-openapi from 1.x to 2.x (GroupedOpenApi, ParameterObject)
- Fix Testcontainers LocalStack API: getEndpointOverride() replaces getEndpointConfiguration()
- Fix Awaitility 4.x: replace org.awaitility.Duration with java.time.Duration
- Fix Hadoop+Java21: disable FileSystem cache to avoid UserGroupInformation incompatibility
- Add JVM --add-opens flags for Hadoop/reflection compatibility in tests
- Exclude JPA auto-configuration from @WebMvcTest context in API tests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Update gha workflows to use java 21

* fix: fix integration tests for Java 21 / Spring Boot 3.2.12 upgrade

- Replace JUnit 4 @Rule with @Container on LocalStack S3 containers in
  BeekeeperMetadataCleanupIntegrationTest and BeekeeperDryRunMetadataCleanupIntegrationTest
  to fix ExceptionInInitializerError caused by static block running before Testcontainers start
- Add properties.sqs.endpoint and properties.sqs.region Spring properties to
  CommonBeans.messageReader() so the SqsMessageReader uses a custom AmazonSQS client
  with explicit LocalStack endpoint and matching region (us-west-2), fixing
  QueueDoesNotExistException caused by LocalStack's region-sensitive SQS queue lookup
- Set properties.sqs.region in BeekeeperUnreferencedPathSchedulerApiaryIntegrationTest
  and BeekeeperExpiredMetadataSchedulerApiaryIntegrationTest to match the region used
  when creating SQS queues in LocalStack
- Apply Spotless formatting to all changed integration test files

All 71 integration tests now pass (1 skipped, pre-existing).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.... (continued)

Coverage Stats

10 of 12 new or added lines in 3 files covered. (83.33%)

115 existing lines in 33 files now uncovered.

1541 of 1757 relevant lines covered (87.71%)

0.88 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

90.91

/beekeeper-core/src/main/java/com/expediagroup/beekeeper/core/validation/S3PathValidator.java

/**
 * Copyright (C) 2019-2022 Expedia, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.expediagroup.beekeeper.core.validation;

import org.apache.commons.lang3.StringUtils;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class S3PathValidator {

  private static final Logger log = LoggerFactory.getLogger(S3PathValidator.class);

  /**
   * The minimum number of "/" chars for partition location: s3://basePath/table/partition = 4
   */
  public static boolean validPartitionPath(String location) {
    boolean valid = getNumSlashes(location) >= 4;
    if (!valid) {
      log.warn("Partition \"{}\" doesn't have the correct number of levels in the path", location);
    }
    return valid;
  }

  /**
   * The minimum number of "/" chars for table location: s3://basePath/table = 3
   */
  public static boolean validTablePath(String location) {
    boolean valid = getNumSlashes(location) >= 3;
    if (!valid) {
      log.warn("Table \"{}\" doesn't have the correct number of levels in the path", location);
    }
    return valid;
  }

  private static int getNumSlashes(String location) {
    return StringUtils.countMatches(location, "/");
  }
}

1	/**
2	* Copyright (C) 2019-2022 Expedia, Inc.
3	*
4	* Licensed under the Apache License, Version 2.0 (the "License");
5	* you may not use this file except in compliance with the License.
6	* You may obtain a copy of the License at
7	*
8	* http://www.apache.org/licenses/LICENSE-2.0
9	*
10	* Unless required by applicable law or agreed to in writing, software
11	* distributed under the License is distributed on an "AS IS" BASIS,
12	* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13	* See the License for the specific language governing permissions and
14	* limitations under the License.
15	*/
16	package com.expediagroup.beekeeper.core.validation;
17
18	import org.apache.commons.lang3.StringUtils;
19	import org.slf4j.Logger;
UNCOV 20	import org.slf4j.LoggerFactory;	×
21
22	public class S3PathValidator {	1✔
23
24	private static final Logger log = LoggerFactory.getLogger(S3PathValidator.class);
25
26	/**	1✔
27	* The minimum number of "/" chars for partition location: s3://basePath/table/partition = 4	1✔
28	*/	1✔
29	public static boolean validPartitionPath(String location) {
30	boolean valid = getNumSlashes(location) >= 4;	1✔
31	if (!valid) {
32	log.warn("Partition \"{}\" doesn't have the correct number of levels in the path", location);
33	}
34	return valid;
35	}	1✔
36		1✔
37	/**	1✔
38	* The minimum number of "/" chars for table location: s3://basePath/table = 3
39	*/	1✔
40	public static boolean validTablePath(String location) {
41	boolean valid = getNumSlashes(location) >= 3;
42	if (!valid) {
43	log.warn("Table \"{}\" doesn't have the correct number of levels in the path", location);	1✔
44	}
45	return valid;
46	}
47
48	private static int getNumSlashes(String location) {
49	return StringUtils.countMatches(location, "/");
50	}
51	}

ExpediaGroup / beekeeper / #820

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous