Re: [PR] Change ManagedSchemaTransformProvider to take a Row config [beam]

2024-05-22 Thread via GitHub
ahmedabu98 closed pull request #30937: Change ManagedSchemaTransformProvider to take a Row config URL: https://github.com/apache/beam/pull/30937 -- 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

Re: [PR] Change ManagedSchemaTransformProvider to take a Row config [beam]

2024-04-16 Thread via GitHub
ahmedabu98 commented on PR #30937: URL: https://github.com/apache/beam/pull/30937#issuecomment-2059859123 Actually, I don't think this is possible because we don't know the underlying transform's configuration schema ahead of time. Even if we do via some mapping, this still won't work

Re: [PR] Change ManagedSchemaTransformProvider to take a Row config [beam]

2024-04-11 Thread via GitHub
github-actions[bot] commented on PR #30937: URL: https://github.com/apache/beam/pull/30937#issuecomment-2050805127 Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment `assign set of reviewers` -- This is an automated

Re: [PR] Change ManagedSchemaTransformProvider to take a Row config [beam]

2024-04-11 Thread via GitHub
ahmedabu98 commented on PR #30937: URL: https://github.com/apache/beam/pull/30937#issuecomment-2050773630 Java IO precommit failing due to #30941 -- 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

[PR] Change ManagedSchemaTransformProvider to take a Row config [beam]

2024-04-11 Thread via GitHub
ahmedabu98 opened a new pull request, #30937: URL: https://github.com/apache/beam/pull/30937 This is a small implementation detail change to make `ManagedSchemaTransformProvider` take a config represented as a Row instead of a YAML string. It makes more sense for xlang use-cases to send a