juripetersen opened a new issue, #796: URL: https://github.com/apache/wayang/issues/796
## Problem In recent PRs #772, #773, #774 we added three new platforms to be supported by Apache Wayang, which is awesome! However, in doing so, we broke some of the conventions for structuring our code. We now have multiple ways of providing demo-like examples of how users could run code with specific platforms. I think this is problematic as it makes it ambiguous for contributors that want to add new code to our project. ## Files The following files were added in non-conventional manner: - [Presto-setup](https://github.com/apache/wayang/tree/main/presto-setup) - [BigQuery-setup](https://github.com/apache/wayang/tree/main/bigquery-setup) - [Trino-setup](https://github.com/apache/wayang/tree/main/trino-setup) - [Trino .sh demo](https://github.com/apache/wayang/blob/main/demo-trino.sh) - [BigQuery .sh demo](https://github.com/apache/wayang/blob/main/demo-bigquery.sh) - [OS specific .env_template](https://github.com/apache/wayang/blob/main/env_template_osx.sh) ## Previous convention Demos or examples were previously found only in [wayang-applications](https://github.com/apache/wayang/tree/main/wayang-applications/src/main/java/org/apache/wayang/applications) or in [wayang-benchmark](https://github.com/apache/wayang/tree/main/wayang-benchmark). ## Docker in Wayangs source Additionally, the demo examples added in #772, #773 and #774 provide docker-compose.yml files for each of the platforms to be set up. While I agree that this is convenient for anyone willing to just execute the demo all batteries included, I do not think that these files should be present in the source of Wayang. An example or demo should be configurable to connect to any third party platform via its API, just by a config like we have in many other examples. This allows us to not rely on too many third party tools and allows users to plug their environment in more seamlessly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
