Skip to content

Commit

Permalink
[Improve][Zeta] Remove default jvm memory option (apache#8068)
Browse files Browse the repository at this point in the history
  • Loading branch information
Hisoka-X authored and hawk9821 committed Nov 18, 2024
1 parent e452189 commit 087fc7f
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 7 deletions.
4 changes: 2 additions & 2 deletions config/jvm_master_options
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
#

# JVM Heap
-Xms2g
-Xmx2g
# -Xms2g
# -Xmx2g

# JVM Dump
-XX:+HeapDumpOnOutOfMemoryError
Expand Down
4 changes: 2 additions & 2 deletions config/jvm_options
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
#

# JVM Heap
-Xms2g
-Xmx2g
# -Xms2g
# -Xmx2g

# JVM Dump
-XX:+HeapDumpOnOutOfMemoryError
Expand Down
4 changes: 2 additions & 2 deletions config/jvm_worker_options
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@
#

# JVM Heap
-Xms2g
-Xmx2g
# -Xms2g
# -Xmx2g

# JVM Dump
-XX:+HeapDumpOnOutOfMemoryError
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,9 @@
import java.util.TreeSet;
import java.util.stream.Collectors;

import static org.apache.seatunnel.e2e.common.container.TestContainerId.FLINK_1_18;
import static org.apache.seatunnel.e2e.common.container.TestContainerId.SPARK_3_3;

@Slf4j
public final class ContainerUtil {

Expand Down Expand Up @@ -317,7 +320,9 @@ public static List<TestContainer> discoverTestContainers() {
.filter(container -> container.identifier().isTestInPR())
.filter(
container -> {
if (testAllContainer) {
if (testAllContainer
|| container.identifier().equals(FLINK_1_18)
|| container.identifier().equals(SPARK_3_3)) {
return true;
}
if (testZetaContainer) {
Expand Down

0 comments on commit 087fc7f

Please sign in to comment.