Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[yaml] Add Beam YAML Examples #32519

Closed
wants to merge 6 commits into from
Closed

[yaml] Add Beam YAML Examples #32519

wants to merge 6 commits into from

Conversation

eder001
Copy link

@eder001 eder001 commented Sep 20, 2024

1x Example: MapToFields and Combine using SQL
1x Example: MapToFields using SQL and the SQL type command
1x Example: Using JDBC connection to read (Mysql) and write (Postgre) in a database

1x Example: MapToFields and Combine using SQL
1x Example: MapToFields using SQL and the SQL type command
1x Example: JDBC IO for read (Mysql) and to write (Postgre) in a database
Copy link
Contributor

@Polber Polber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left some comments, thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would move this example to the io subdirectory

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Polber Done!

Copy link
Contributor

Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment assign set of reviewers

@damccorm
Copy link
Contributor

damccorm commented Oct 3, 2024

@Polber @eder001 what are next steps on this PR?

@claudevdm
Copy link
Contributor

@Polber @eder001 is this waiting on a next round of reviews?

@eder001 eder001 closed this Nov 18, 2024
@eder001
Copy link
Author

eder001 commented Nov 18, 2024

No, it is not... please just disregard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants