SEZ9 commented on issue #11007: URL: https://github.com/apache/seatunnel/issues/11007#issuecomment-5409359590
Hi @KaustAbhinand, thanks for following up! Since you've completed the investigation of the Google Sheets connector and documented your findings in [PR #11943](https://github.com/apache/seatunnel/pull/11943), yes — you can go ahead and claim connector-graphql. A few asks for the connector-graphql work: 1. Please start with the same kind of analysis you did for connector-google-sheets: check whether there is any imperative validation that needs to be migrated into a declarative `OptionRule`, or whether it is already compliant. 2. If migration is needed, keep the PR scoped to connector-graphql only, target `dev`, and include focused validation test coverage for both valid and invalid configurations. 3. Link the PR (or your findings, if no changes are needed) back here on the thread so we can track it against the table. I'll take a look at PR #11943 for the Google Sheets findings. Thanks for your continued contributions! <!-- streview-comment:545 --> -- 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]
