Skip to content

feat: Support CRaC and priming of powertools metrics and idempotency-dynamodb #1861

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
597d123
Automatic priming of powertools-metrics
subhash686 May 8, 2025
ec43605
Merge remote-tracking branch 'upstream/v2' into support_crac_on_power…
subhash686 May 8, 2025
91ee9df
Fixed resource name and added unit tests and javadoc comments
subhash686 May 14, 2025
e10a624
Invoke prime Dynamo persistent store
subhash686 May 17, 2025
747dad8
Merge branch 'v2' into support_crac_on_powertools_metrics
subhash686 May 28, 2025
c8d2404
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jun 12, 2025
7ba4cfa
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jun 13, 2025
8b2dba4
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jun 24, 2025
85513ea
Fixed Sonar issues
subhash686 Jul 2, 2025
e1906e9
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jul 7, 2025
0c80204
Update classesloaded.txt
subhash686 Jul 9, 2025
babd855
Merge branch 'main' into support_crac_on_powertools_metrics
phipag Jul 9, 2025
789564b
Moved priming to MetricsFactory and DynamoDBPersistenceStore, added P…
subhash686 Jul 22, 2025
4cebb6b
Update classesloaded.txt
subhash686 Jul 22, 2025
6336252
Update LambdaMetricsAspect.java
subhash686 Jul 22, 2025
27dad71
Update DynamoDBPersistenceStore.java
subhash686 Jul 22, 2025
685877e
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jul 22, 2025
ec37b49
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jul 28, 2025
a1e1545
Improved priming.md documentation and static initialization of classe…
subhash686 Jul 30, 2025
6261d06
Merge branch 'support_crac_on_powertools_metrics' of https://github.c…
subhash686 Jul 30, 2025
844dad7
Fixed a Sonarqube finding and added more unit tests to ensure all ava…
subhash686 Jul 31, 2025
9b6e5f9
Merge branch 'main' into support_crac_on_powertools_metrics
subhash686 Jul 31, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
59 changes: 59 additions & 0 deletions Priming.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
# Automatic Priming for AWS Lambda Powertools Java

## Table of Contents
- [Overview](#overview)
- [Implementation Steps](#general-implementation-steps)
- [Known Issues](#known-issues-and-solutions)
- [Reference Implementation](#reference-implementation)

## Overview
Priming is the process of preloading dependencies and initializing resources during the INIT phase, rather than during the INVOKE phase to further optimize startup performance with SnapStart.
This is required because Java frameworks that use dependency injection load classes into memory when these classes are explicitly invoked, which typically happens during Lambda’s INVOKE phase.

This documentation provides guidance for automatic class priming in Powertools for AWS Lambda Java modules.


## Implementation Steps
Classes are proactively loaded using Java runtime hooks which are part of the open source [CRaC (Coordinated Restore at Checkpoint) project](https://openjdk.org/projects/crac/).
Implementations across the project use the `beforeCheckpoint()` hook, to prime Snapstart-enabled Java functions via Class Priming.
In order to generate the `classloaded.txt` file for a Java module in this project, follow these general steps.

1. **Add Maven Profile**
- Add maven test profile with the following VM argument for generating classes loaded files.
```shell
-Xlog:class+load=info:classesloaded.txt
```
- You can find an example of this in `generate-classesloaded-file` profile in this [pom.xml](powertools-metrics/pom.xml).

2. **Generate classes loaded file**
- Run tests with `-Pgenerate-classesloaded-file` profile.
```shell
mvn -Pgenerate-classesloaded-file clean test
```
- This will generate a file named `classesloaded.txt` in the target directory of the module.

3. **Cleanup the file**
- The classes loaded file generated in Step 2 has the format
`[0.054s][info][class,load] java.lang.Object source: shared objects file`
but we are only interested in `java.lang.Object` - the fully qualified class name.
- To strip the lines to include only the fully qualified class name,
Use the following regex to replace with empty string.
- `^\[[\[\]0-9.a-z,]+ ` (to replace the left part)
- `( source: )[0-9a-z :/._$-]+` (to replace the right part)

4. **Add file to resources**
- Move the cleaned-up file to the corresponding `src/main/resources` directory of the module. See [example](powertools-metrics/src/main/resources/classesloaded.txt).

5. **Register and checkpoint**
- A class, usually the entry point of the module, should register the CRaC resource in the constructor. [Example](powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/MetricsFactory.java)
- Note that AspectJ aspect is not suitable for this purpose, as it does not work with CRaC.
- Add the `beforeCheckpoint()` hook in the same class to invoke `ClassPreLoader.preloadClasses()`. The `ClassPreLoader` class is implemented in `powertools-common` module.
- This will ensure that the classes are already pre-loaded by the Snapstart RESTORE operation leading to a shorter INIT duration.


## Known Issues
- This is a manual process at the moment, but it can be automated in the future.
- `classesloaded.txt` file includes test classes as well because the file is generated while running tests. This is not a problem because all the classes that are not found are ignored by `ClassPreLoader.preloadClasses()`. Also `beforeCheckpoint()` hook is not time-sensitive, it only runs once when a new Lambda version gets published.

## Reference Implementation
Working example is available in the [powertools-metrics](powertools-metrics/src/main/java/software/amazon/lambda/powertools/metrics/MetricsFactory.java).
6 changes: 6 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,7 @@
<mockito.version>5.18.0</mockito.version>
<mockito-junit-jupiter.version>5.18.0</mockito-junit-jupiter.version>
<junit-pioneer.version>2.3.0</junit-pioneer.version>
<crac.version>1.4.0</crac.version>

<!-- As we have a .mvn directory at the root of the project, this will evaluate to the root directory
regardless of where maven is run - sub-module, or root. -->
Expand Down Expand Up @@ -264,6 +265,11 @@
<artifactId>logback-ecs-encoder</artifactId>
<version>${elastic.version}</version>
</dependency>
<dependency>
<groupId>org.crac</groupId>
<artifactId>crac</artifactId>
<version>${crac.version}</version>
</dependency>
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-api</artifactId>
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,91 @@
/*
* Copyright 2023 Amazon.com, Inc. or its affiliates.
* 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 software.amazon.lambda.powertools.common.internal;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.nio.charset.StandardCharsets;
import java.net.URL;
import java.net.URLConnection;
import java.util.Enumeration;

/**
* Used to preload classes to support automatic priming for SnapStart
*/
public final class ClassPreLoader {
public static final String CLASSES_FILE = "classesloaded.txt";

private ClassPreLoader() {
// Hide default constructor
}

/**
* Initializes the classes listed in the classesloaded resource
*/
public static void preloadClasses() {
try {
Enumeration<URL> files = ClassPreLoader.class.getClassLoader().getResources(CLASSES_FILE);
// If there are multiple files, preload classes from all of them
while (files.hasMoreElements()) {
URL url = files.nextElement();
URLConnection conn = url.openConnection();
conn.setUseCaches(false);
InputStream is = conn.getInputStream();

Check failure on line 46 in powertools-common/src/main/java/software/amazon/lambda/powertools/common/internal/ClassPreLoader.java

View workflow job for this annotation

GitHub Actions / pmd_analyse

Ensure that resources like this InputStream object are closed after use

Ensure that resources (like `java.sql.Connection`, `java.sql.Statement`, and `java.sql.ResultSet` objects and any subtype of `java.lang.AutoCloseable`) are always closed after use. Failing to do so might result in resource leaks. Note: It suffices to configure the super type, e.g. `java.lang.AutoCloseable`, so that this rule automatically triggers on any subtype (e.g. `java.io.FileInputStream`). Additionally specifying `java.sql.Connection` helps in detecting the types, if the type resolution / auxclasspath is not correctly setup. Note: Since PMD 6.16.0 the default value for the property `types` contains `java.lang.AutoCloseable` and detects now cases where the standard `java.io.*Stream` classes are involved. In order to restore the old behaviour, just remove "AutoCloseable" from the types. CloseResource (Priority: 1, Ruleset: Error Prone) https://docs.pmd-code.org/snapshot/pmd_rules_java_errorprone.html#closeresource
preloadClassesFromStream(is);
}
} catch (IOException ignored) {
// No action is required if preloading fails for any reason
}
}

/**
* Loads the list of classes passed as a stream
*
* @param is
*/
private static void preloadClassesFromStream(InputStream is) {
try (is;
InputStreamReader isr = new InputStreamReader(is, StandardCharsets.UTF_8);
BufferedReader reader = new BufferedReader(isr)) {
String line;
while ((line = reader.readLine()) != null) {
int idx = line.indexOf('#');
if (idx != -1) {
line = line.substring(0, idx);
}
final String className = line.stripTrailing();
if (!className.isBlank()) {
loadClassIfFound(className);
}
}
} catch (Exception ignored) {
// No action is required if preloading fails for any reason
}
}

/**
* Initializes the class with given name if found, ignores otherwise
*
* @param className
*/
private static void loadClassIfFound(String className) {
try {
Class.forName(className, true, ClassPreLoader.class.getClassLoader());
} catch (ClassNotFoundException e) {
// No action is required if the class with given name cannot be found
}

Check failure on line 89 in powertools-common/src/main/java/software/amazon/lambda/powertools/common/internal/ClassPreLoader.java

View workflow job for this annotation

GitHub Actions / pmd_analyse

Avoid empty catch blocks

Empty Catch Block finds instances where an exception is caught, but nothing is done. In most circumstances, this swallows an exception which should either be acted on or reported. EmptyCatchBlock (Priority: 1, Ruleset: Error Prone) https://docs.pmd-code.org/snapshot/pmd_rules_java_errorprone.html#emptycatchblock
}
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
package software.amazon.lambda.powertools.common.internal;

import static org.junit.jupiter.api.Assertions.*;

import org.junit.jupiter.api.Test;

class ClassPreLoaderTest {

// Making this volatile so the Thread Context doesn't need any special handling
static volatile boolean dummyClassLoaded = false;

Check failure on line 10 in powertools-common/src/test/java/software/amazon/lambda/powertools/common/internal/ClassPreLoaderTest.java

View workflow job for this annotation

GitHub Actions / pmd_analyse

Use of modifier volatile is not recommended.

Use of the keyword 'volatile' is generally used to fine tune a Java application, and therefore, requires a good expertise of the Java Memory Model. Moreover, its range of action is somewhat misknown. Therefore, the volatile keyword should not be used for maintenance purpose and portability. AvoidUsingVolatile (Priority: 1, Ruleset: Multithreading) https://docs.pmd-code.org/snapshot/pmd_rules_java_multithreading.html#avoidusingvolatile

/**
* Dummy class to be loaded by ClassPreLoader in test.
* <b>The class name is referenced in <i>powertools-common/src/test/resources/classesloaded.txt</i></b>
* This class is used to verify that the ClassPreLoader can load valid classes.
* The static block sets a flag to indicate that the class has been loaded.
*/
static class DummyClass {
static {
dummyClassLoaded = true;
}
}
@Test
void preloadClasses_shouldIgnoreInvalidClassesAndLoadValidClasses() {

dummyClassLoaded = false;
// powertools-common/src/test/resources/classesloaded.txt has a class that does not exist
// Verify that the missing class did not throw any exception
assertDoesNotThrow(ClassPreLoader::preloadClasses);

// When the classloaded.txt is a mixed bag of valid and invalid classes, Valid class must load
assertTrue(dummyClassLoaded, "DummyClass should be loaded");
}
}
2 changes: 2 additions & 0 deletions powertools-common/src/test/resources/classesloaded.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
software.amazon.lambda.powertools.common.internal.NonExistingClass
software.amazon.lambda.powertools.common.internal.ClassPreLoaderTest$DummyClass
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,10 @@
</exclusion>
</exclusions>
</dependency>
<dependency>
<groupId>org.crac</groupId>
<artifactId>crac</artifactId>
</dependency>

<!-- Test dependencies -->
<dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@

package software.amazon.lambda.powertools.idempotency.persistence.dynamodb;

import org.crac.Core;
import org.crac.Resource;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import software.amazon.awssdk.core.client.config.ClientOverrideConfiguration;
Expand All @@ -37,6 +39,7 @@
import software.amazon.lambda.powertools.idempotency.persistence.PersistenceStore;

import java.time.Instant;
import java.time.temporal.ChronoUnit;
import java.util.AbstractMap;
import java.util.HashMap;
import java.util.Map;
Expand All @@ -52,7 +55,7 @@
* DynamoDB version of the {@link PersistenceStore}. Will store idempotency data in DynamoDB.<br>
* Use the {@link Builder} to create a new instance.
*/
public class DynamoDBPersistenceStore extends BasePersistenceStore implements PersistenceStore {
public final class DynamoDBPersistenceStore extends BasePersistenceStore implements PersistenceStore, Resource {

public static final String IDEMPOTENCY = "idempotency";
private static final Logger LOG = LoggerFactory.getLogger(DynamoDBPersistenceStore.class);
Expand Down Expand Up @@ -109,6 +112,39 @@
this.dynamoDbClient = null;
}
}
Core.getGlobalContext().register(this);
}

/**
* Primes the persistent store by invoking the get record method with a key that doesn't exist.
*
* @param context
* @throws Exception
*/
@Override
public void beforeCheckpoint(org.crac.Context<? extends Resource> context) throws Exception {
try {
String primingRecordKey = "__invoke_prime__";
Instant now = Instant.now();
long expiry = now.plus(3600, ChronoUnit.SECONDS).getEpochSecond();
DataRecord primingDataRecord = new DataRecord(
primingRecordKey,
DataRecord.Status.COMPLETED,
expiry,
null, // no data
null // no validation
);
putRecord(primingDataRecord, Instant.now());
getRecord(primingRecordKey);
deleteRecord(primingRecordKey);
} catch (Exception unknown) {
// This is unexpected but we must continue without any interruption
}

Check failure on line 142 in powertools-idempotency/powertools-idempotency-dynamodb/src/main/java/software/amazon/lambda/powertools/idempotency/persistence/dynamodb/DynamoDBPersistenceStore.java

View workflow job for this annotation

GitHub Actions / pmd_analyse

Avoid empty catch blocks

Empty Catch Block finds instances where an exception is caught, but nothing is done. In most circumstances, this swallows an exception which should either be acted on or reported. EmptyCatchBlock (Priority: 1, Ruleset: Error Prone) https://docs.pmd-code.org/snapshot/pmd_rules_java_errorprone.html#emptycatchblock
}

@Override
public void afterRestore(org.crac.Context<? extends Resource> context) throws Exception {
// This is a no-op, as we don't need to do anything after restore
}

public static Builder builder() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,9 @@
// Make sure file is cleaned up before running tests
try {
FileChannel.open(Paths.get("target/logfile.json"), StandardOpenOption.WRITE).truncate(0).close();
} catch (NoSuchFileException e) {
// file may not exist on the first launch
}

Check failure on line 87 in powertools-logging/powertools-logging-logback/src/test/java/software/amazon/lambda/powertools/logging/internal/LambdaJsonEncoderTest.java

View workflow job for this annotation

GitHub Actions / pmd_analyse

Avoid empty catch blocks

Empty Catch Block finds instances where an exception is caught, but nothing is done. In most circumstances, this swallows an exception which should either be acted on or reported. EmptyCatchBlock (Priority: 1, Ruleset: Error Prone) https://docs.pmd-code.org/snapshot/pmd_rules_java_errorprone.html#emptycatchblock
}

@AfterEach
Expand Down Expand Up @@ -368,7 +368,7 @@
String result = new String(encoded, StandardCharsets.UTF_8);

// THEN
assertThat(result).contains("\"thread\":\"main\",\"thread_id\":1,\"thread_priority\":5");
assertThat(result).contains("\"thread\":\"main\",\"thread_id\":"+ Thread.currentThread().getId() +",\"thread_priority\":5");
}

@Test
Expand Down
21 changes: 21 additions & 0 deletions powertools-metrics/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,10 @@
<artifactId>aspectjrt</artifactId>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>org.crac</groupId>
<artifactId>crac</artifactId>
</dependency>
<dependency>
<groupId>software.amazon.lambda</groupId>
<artifactId>powertools-common</artifactId>
Expand Down Expand Up @@ -114,6 +118,23 @@
</dependencies>

<profiles>
<profile>
<id>generate-classesloaded-file</id>
<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<argLine>-Xlog:class+load=info:classesloaded.txt
--add-opens java.base/java.util=ALL-UNNAMED
--add-opens java.base/java.lang=ALL-UNNAMED
</argLine>
</configuration>
</plugin>
</plugins>
</build>
</profile>
<profile>
<id>generate-graalvm-files</id>
<dependencies>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@

package software.amazon.lambda.powertools.metrics;

import org.crac.Core;
import org.crac.Resource;
import software.amazon.lambda.powertools.common.internal.ClassPreLoader;
import software.amazon.lambda.powertools.common.internal.LambdaConstants;
import software.amazon.lambda.powertools.common.internal.LambdaHandlerProcessor;
import software.amazon.lambda.powertools.metrics.model.DimensionSet;
Expand All @@ -23,11 +26,16 @@
/**
* Factory for accessing the singleton Metrics instance
*/
public final class MetricsFactory {
public final class MetricsFactory implements Resource {
private static MetricsProvider provider = new EmfMetricsProvider();
private static Metrics metrics;

private MetricsFactory() {
// Dummy instance to register MetricsFactory with CRaC
private static final MetricsFactory INSTANCE = new MetricsFactory();

// Static block to ensure CRaC registration happens at class loading time
static {
Core.getGlobalContext().register(INSTANCE);
}

/**
Expand Down Expand Up @@ -68,4 +76,15 @@ public static synchronized void setMetricsProvider(MetricsProvider metricsProvid
// Reset the metrics instance so it will be recreated with the new provider
metrics = null;
}

@Override
public void beforeCheckpoint(org.crac.Context<? extends Resource> context) throws Exception {
MetricsFactory.getMetricsInstance();
ClassPreLoader.preloadClasses();
}

@Override
public void afterRestore(org.crac.Context<? extends Resource> context) throws Exception {
// No action needed after restore
}
}
Loading