Skip to content

Commit 903c118

Browse files
committed
Upgraded Spark to 2.8.0
1 parent 0fa8730 commit 903c118

File tree

5 files changed

+15
-11
lines changed

5 files changed

+15
-11
lines changed

aws-serverless-java-container-spark/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
</parent>
1616

1717
<properties>
18-
<spark.version>2.7.2</spark.version>
18+
<spark.version>2.8.0</spark.version>
1919
</properties>
2020

2121
<dependencies>

aws-serverless-java-container-spark/src/main/java/com/amazonaws/serverless/proxy/spark/embeddedserver/LambdaEmbeddedServer.java

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
import org.slf4j.Logger;
77
import org.slf4j.LoggerFactory;
8+
import spark.ExceptionMapper;
89
import spark.embeddedserver.EmbeddedServer;
910
import spark.embeddedserver.jetty.websocket.WebSocketHandlerWrapper;
1011
import spark.http.matching.MatcherFilter;
@@ -24,6 +25,7 @@ public class LambdaEmbeddedServer
2425
//-------------------------------------------------------------
2526

2627
private Routes applicationRoutes;
28+
private ExceptionMapper exceptionMapper;
2729
private MatcherFilter sparkFilter;
2830
private StaticFilesConfiguration staticFilesConfiguration;
2931
private boolean hasMultipleHandler;
@@ -34,14 +36,15 @@ public class LambdaEmbeddedServer
3436
// Constructors
3537
//-------------------------------------------------------------
3638

37-
LambdaEmbeddedServer(Routes routes, StaticFilesConfiguration filesConfig, boolean multipleHandlers) {
39+
LambdaEmbeddedServer(Routes routes, StaticFilesConfiguration filesConfig, ExceptionMapper exceptionMapper, boolean multipleHandlers) {
3840
Timer.start("SPARK_EMBEDDED_SERVER_CONSTRUCTOR");
3941
applicationRoutes = routes;
4042
staticFilesConfiguration = filesConfig;
4143
hasMultipleHandler = multipleHandlers;
44+
this.exceptionMapper = exceptionMapper;
4245

4346
// try to initialize the filter here.
44-
sparkFilter = new MatcherFilter(applicationRoutes, staticFilesConfiguration, true, hasMultipleHandler);
47+
sparkFilter = new MatcherFilter(applicationRoutes, staticFilesConfiguration, exceptionMapper, true, hasMultipleHandler);
4548
Timer.stop("SPARK_EMBEDDED_SERVER_CONSTRUCTOR");
4649
}
4750

@@ -50,17 +53,17 @@ public class LambdaEmbeddedServer
5053
// Implementation - EmbeddedServer
5154
//-------------------------------------------------------------
5255
@Override
53-
public int ignite(String s, int i, SslStores sslStores, int i1, int i2, int i3)
56+
public int ignite(String host, int port, SslStores sslStores, int maxThreads, int minThreads, int threadIdleTimeoutMillis)
5457
throws ContainerInitializationException {
5558
Timer.start("SPARK_EMBEDDED_IGNITE");
5659
log.info("Starting Spark server, ignoring port and host");
5760
// if not initialized yet
5861
if (sparkFilter == null) {
59-
sparkFilter = new MatcherFilter(applicationRoutes, staticFilesConfiguration, true, hasMultipleHandler);
62+
sparkFilter = new MatcherFilter(applicationRoutes, staticFilesConfiguration, exceptionMapper, true, hasMultipleHandler);
6063
}
6164
sparkFilter.init(null);
6265
Timer.stop("SPARK_EMBEDDED_IGNITE");
63-
return i;
66+
return port;
6467
}
6568

6669

aws-serverless-java-container-spark/src/main/java/com/amazonaws/serverless/proxy/spark/embeddedserver/LambdaEmbeddedServerFactory.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import com.amazonaws.serverless.proxy.internal.testutils.Timer;
44

55
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
6+
import spark.ExceptionMapper;
67
import spark.embeddedserver.EmbeddedServer;
78
import spark.embeddedserver.EmbeddedServerFactory;
89
import spark.route.Routes;
@@ -39,10 +40,10 @@ public LambdaEmbeddedServerFactory(LambdaEmbeddedServer server) {
3940

4041

4142
@Override
42-
public EmbeddedServer create(Routes routes, StaticFilesConfiguration staticFilesConfiguration, boolean multipleHandlers) {
43+
public EmbeddedServer create(Routes routes, StaticFilesConfiguration staticFilesConfiguration, ExceptionMapper exceptionMapper, boolean multipleHandlers) {
4344
Timer.start("SPARK_SERVER_FACTORY_CREATE");
4445
if (embeddedServer == null) {
45-
LambdaEmbeddedServerFactory.embeddedServer = new LambdaEmbeddedServer(routes, staticFilesConfiguration, multipleHandlers);
46+
LambdaEmbeddedServerFactory.embeddedServer = new LambdaEmbeddedServer(routes, staticFilesConfiguration, exceptionMapper, multipleHandlers);
4647
}
4748
Timer.stop("SPARK_SERVER_FACTORY_CREATE");
4849
return embeddedServer;

aws-serverless-java-container-spark/src/test/java/com/amazonaws/serverless/proxy/spark/embeddedserver/LambdaEmbeddedServerTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010

1111
public class LambdaEmbeddedServerTest {
12-
private static LambdaEmbeddedServer server = new LambdaEmbeddedServer(null, null, false);
12+
private static LambdaEmbeddedServer server = new LambdaEmbeddedServer(null, null, null, false);
1313

1414
@Test
1515
public void webSocket_configureWebSocket_noException() {

samples/spark/pet-store/pom.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,14 @@
2727
<maven.compiler.source>1.8</maven.compiler.source>
2828
<maven.compiler.target>1.8</maven.compiler.target>
2929
<jackson.version>2.9.4</jackson.version>
30-
<spark.version>2.7.2</spark.version>
30+
<spark.version>2.8.0</spark.version>
3131
</properties>
3232

3333
<dependencies>
3434
<dependency>
3535
<groupId>com.amazonaws.serverless</groupId>
3636
<artifactId>aws-serverless-java-container-spark</artifactId>
37-
<version>[0.1,)</version>
37+
<version>[0.2,)</version>
3838
</dependency>
3939

4040
<!-- https://mvnrepository.com/artifact/com.sparkjava/spark-core -->

0 commit comments

Comments
 (0)