Skip to content

Commit

Permalink
[FLINK-12495][python][client] Move PythonGatewayServer into flink-cli…
Browse files Browse the repository at this point in the history
…ents.

This closes apache#8423
  • Loading branch information
sunjincheng121 committed May 13, 2019
1 parent 3f532e1 commit d1542e9
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 9 deletions.
7 changes: 7 additions & 0 deletions flink-clients/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,13 @@ under the License.
<artifactId>commons-cli</artifactId>
</dependency>

<!-- Python API dependencies -->
<dependency>
<groupId>net.sf.py4j</groupId>
<artifactId>py4j</artifactId>
<version>${py4j.version}</version>
</dependency>

<!-- test dependencies -->

<dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
* limitations under the License.
*/

package org.apache.flink.api.python;
package org.apache.flink.client.python;

import py4j.GatewayServer;

Expand Down
7 changes: 0 additions & 7 deletions flink-core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -86,13 +86,6 @@ under the License.
<artifactId>flink-shaded-guava</artifactId>
</dependency>

<!-- Python API dependencies -->
<dependency>
<groupId>net.sf.py4j</groupId>
<artifactId>py4j</artifactId>
<version>0.10.8.1</version>
</dependency>

<!-- ================== test dependencies ================== -->

<dependency>
Expand Down
2 changes: 1 addition & 1 deletion flink-python/pyflink/java_gateway.py
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ def launch_gateway():
raise Exception("Windows system is not supported currently.")
script = "./bin/pyflink-gateway-server.sh"
command = [os.path.join(FLINK_HOME, script)]
command += ['-c', 'org.apache.flink.api.python.PythonGatewayServer']
command += ['-c', 'org.apache.flink.client.python.PythonGatewayServer']

# Create a temporary directory where the gateway server should write the connection information.
conn_info_dir = tempfile.mkdtemp()
Expand Down
1 change: 1 addition & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ under the License.
<mockito.version>2.21.0</mockito.version>
<powermock.version>2.0.0-RC.4</powermock.version>
<hamcrest.version>1.3</hamcrest.version>
<py4j.version>0.10.8.1</py4j.version>
<japicmp.skip>false</japicmp.skip>
<flink.convergence.phase>validate</flink.convergence.phase>
<!--
Expand Down

0 comments on commit d1542e9

Please sign in to comment.