Update TableFunctionProcessorProvider.getDataProcessor to include ConnectorSession parameter

[tbaeg]

Description

Fixes #19778

Update TableFunctionProcessorProvider.getDataProcessor to include ConnectorSession parameter to provide a consistent interface with TableFunctionProcessorProvider.getSplitProcessor.

Additional context and related issues

Slack discussion

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text:

# Section
* Update `TableFunctionProcessorProvider.getDataProcessor` to include `ConnectorSession` parameter. This is a breaking change to the interface. ({issue}`19778`)

[cla-bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

[tbaeg]

@findepi Not all too familiar with that trino-sqlserver error and didn't seem related to the changes I made at first glad. Any chance you or someone more close to the plugin would be able to assist?

[tbaeg]

Ah, nevermind. I just saw b060fff. Let me know if there is anything else I need to do.

[findepi]

can you please make sure you filled the CLA? For more information, see https://github.com/trinodb/cla.

[tbaeg]

can you please make sure you filled the CLA? For more information, see https://github.com/trinodb/cla.

Already signed and sent a week ago.

[cla-bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

[findepi]

can you please make sure you filled the CLA? For more information, see https://github.com/trinodb/cla.

Already signed and sent a week ago.

Thanks!

Let's wait until it's processed by @martint .

[findepi]

#19849 fixed an bug in Delta and Iceberg, but it had to introduce new implementations of the interfaces being changed here. I guess the implementation may require an update in this PR, since we're changing interface signatures. @tbaeg Can you please rebase?

[cla-bot]

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

[tbaeg]

#19849 fixed an bug in Delta and Iceberg, but it had to introduce new implementations of the interfaces being changed here. I guess the implementation may require an update in this PR, since we're changing interface signatures. @tbaeg Can you please rebase?

Rebased and also added ClassLoaderSafeTableFunctionDataProcessor to be in line with the commit mentioned above. There are cases I had my own implementation so I think it makes sense to include. Let me know if you'd like a separate PR for this.

[tbaeg]

can you please make sure you filled the CLA? For more information, see https://github.com/trinodb/cla.

Already signed and sent a week ago.

Thanks!

Let's wait until it's processed by @martint .

CLA processed. Thanks @martint!

[findepi]

@cla-bot check

[cla-bot]

The cla-bot has been summoned, and re-checked this pull request!

[findepi]

cc @homar