Skip to content

Commit

Permalink
fix[apache#2546]:Running IT use cases under Windows requires replacin…
Browse files Browse the repository at this point in the history
…g \ with / and start-seatunnel-flink-new-connector.sh need to use '#!/bin/sh' (apache#2547)
  • Loading branch information
liugddx authored and MRYOG committed Sep 16, 2022
1 parent 2e09417 commit 95a492d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/bin/bash
#!/bin/sh
#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements. See the NOTICE file distributed with
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -116,8 +116,9 @@ public Container.ExecResult executeSeaTunnelFlinkJob(String confFile) throws IOE

// Running IT use cases under Windows requires replacing \ with /
String conf = targetConfInContainer.replaceAll("\\\\", "/");
String binPath = Paths.get(SEATUNNEL_HOME, "bin", SEATUNNEL_FLINK_BIN).toString().replaceAll("\\\\", "/");
final List<String> command = new ArrayList<>();
command.add(Paths.get(SEATUNNEL_HOME, "bin", SEATUNNEL_FLINK_BIN).toString());
command.add(binPath);
command.add("--config " + conf);

Container.ExecResult execResult = jobManager.execInContainer("bash", "-c", String.join(" ", command));
Expand Down

0 comments on commit 95a492d

Please sign in to comment.