[TBD] - Implement Docker support with image creation and configuration updates#767
[TBD] - Implement Docker support with image creation and configuration updates#767novatechflow wants to merge 2 commits into
Conversation
|
Thanks for bringing the conversation here Alex. |
|
Yes, that's the whole release we publish, I changed the Wayang-submit part so we could add configs there. |
|
Great, that's fine with me. Another thing we could provide is a docker image for working with the python api which is more cumbersome as the Wayang REST server needs to be up and running. What do you think? |
|
Dug into that - needs a few changes in code, so I just saw that we hard-coded the python client is hard-coded! to use localhost, which of course makes a docker container impossible. I also found a bunch of other hard-coded values which made Wayang seems clunky. |
- Updated Dockerfile for Python API with necessary configurations. - Created entrypoint script for Python container to ensure API availability. - Added example word count script for demonstration. - Enhanced Java integration with dynamic API URL configuration. - Updated dependencies in setup.cfg for compatibility. - Modified GitHub Actions workflow to include Python image build and tests.
|
lot of changes:
Verified:
|
TBD - to be discussed
Summary
Add Docker image support for Apache Wayang without bundling external execution platforms.
Changes
SPARK_HOME,HADOOP_HOME, andFLINK_HOMEoptional inwayang-submitwayang-assembly/README.mdRationale
The image should distribute Wayang only. Spark, Hadoop, Flink, JDBC drivers, and other platform runtimes remain user supplied so the project does not take ownership of maintaining third-party platform distributions inside the image.
Verification
bash -n bin/wayang-submitJAVA_HOME=/Library/Java/JavaVirtualMachines/zulu-17.jdk/Contents/Home ./mvnw -pl :wayang-java,:wayang-assembly -am -Pdistribution -DskipTests packageJAVA_HOME=/Library/Java/JavaVirtualMachines/zulu-17.jdk/Contents/Home OTHER_FLAGS=-Duser.home=/private/tmp wayang-assembly/target/apache-wayang-assembly-1.1.2-SNAPSHOT-dist/wayang-1.1.2-SNAPSHOT/bin/wayang-submit org.apache.wayang.apps.pi.PiEstimation java 1Notes
Built on Mac, path are different on Linux / Windows.