Skip to content

Commit de33e3b

Browse files
committed
Merge branch '5.3.x'
2 parents ed26962 + dbeae27 commit de33e3b

File tree

1 file changed

+10
-3
lines changed
  • spring-web/src/testFixtures/java/org/springframework/web/testfixture/http/server/reactive/bootstrap

1 file changed

+10
-3
lines changed

spring-web/src/testFixtures/java/org/springframework/web/testfixture/http/server/reactive/bootstrap/JettyHttpServer.java

+10-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2021 the original author or authors.
2+
* Copyright 2002-2022 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -27,6 +27,7 @@
2727

2828
/**
2929
* @author Rossen Stoyanchev
30+
* @author Sam Brannen
3031
*/
3132
public class JettyHttpServer extends AbstractHttpServer {
3233

@@ -75,7 +76,10 @@ protected void stopInternal() throws Exception {
7576
finally {
7677
try {
7778
if (this.jettyServer.isRunning()) {
78-
this.jettyServer.setStopTimeout(5000);
79+
// Do not configure a large stop timeout. For example, setting a stop timeout
80+
// of 5000 adds an additional 1-2 seconds to the runtime of each test using
81+
// the Jetty sever, resulting in 2-4 extra minutes of overall build time.
82+
this.jettyServer.setStopTimeout(100);
7983
this.jettyServer.stop();
8084
this.jettyServer.destroy();
8185
}
@@ -90,7 +94,10 @@ protected void stopInternal() throws Exception {
9094
protected void resetInternal() {
9195
try {
9296
if (this.jettyServer.isRunning()) {
93-
this.jettyServer.setStopTimeout(5000);
97+
// Do not configure a large stop timeout. For example, setting a stop timeout
98+
// of 5000 adds an additional 1-2 seconds to the runtime of each test using
99+
// the Jetty sever, resulting in 2-4 extra minutes of overall build time.
100+
this.jettyServer.setStopTimeout(100);
94101
this.jettyServer.stop();
95102
this.jettyServer.destroy();
96103
}

0 commit comments

Comments
 (0)