Rename and replace 'client' in classes and comments

the-other-tim-brown commented 3 months ago

Important Read

Please ensure the GitHub issue is mentioned at the beginning of the PR

What is the purpose of the pull request

Addresses #406 Renames the SourceClient, TargetClient, and OneTableClient to ConversionSource, ConversionTarget, and ConversionController.

Brief change log

Changed client package to conversion
SourceClient and similar classes are now ConversionSource or a name derived from that
TargetClient and similar classes are now ConversionTarget or a name derived from that
OneTableClient is ConversionController
RunSync configs were updated to remove references to client
Verify this pull request

This pull request is a trivial rework / code cleanup without any test coverage.

the-other-tim-brown commented 3 months ago

Raising as a draft. I need to confirm the demo still works after this.

the-other-tim-brown commented 3 months ago

cc @lmccay

the-other-tim-brown commented 3 months ago

I have confirmed that the demo is still working after these updates

xtable-bot commented 3 months ago

CI report:

aeefb3bdfbf1717da2306e24888d98a01f71e05a Azure: SUCCESS

Bot commands

@xtable-bot supports the following commands: - `@xtable-bot run azure` re-run the last Azure build

the-other-tim-brown commented 3 months ago

Tested the manual conversion flow documented in the README as well to confirm all the values are properly renamed in the yamls, I will merge this now.

apache / incubator-xtable