Support INSERT in Google Sheets connector

[sbernauer]

Description

Fixes #3866. Extracted parts out of #5011

This PR add support for INSERT statements in the Google Sheets connector

Non-technical explanation

Release notes

( ) This is not user-visible or 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:

# Google Sheets connector
* Add support for the `INSERT` statement ({issue}`15026`)

[ebyhr]

Please separate a commit into INSERT and type mapping.

[sbernauer]

@ebyhr thanks for your fast and extensive review! All feedback should be adressed

[sbernauer]

@ebyhr many thanks again! Feedback should be addressed

[sbernauer]

Rebased and fixed all the introduced conflicts. I'm going to look into creating a dedicated spreadsheet for every test invocation.

[sbernauer]

I refactored all the Google sheet tests to:

1. Create a dedicated temporary spreadsheet for every test run
2. Add Sheets (tabs) containing test data to the Spreadsheet
3. We can now happily insert or modify data as we won't effect any other running test
4. Afterwards the temporary spreadsheet gets deleted. We have to use Drive API to delete the spreadsheet :/ Currently this fails with Access Not Configured. Drive API has not been used in project 1006290340336 before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/drive.googleapis.com/overview?project=1006290340336 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry. @ebyhr do you have the permission to enable this API or do know someone who can do this? Currently the cleanup code is commented out.

[sbernauer]

Rebased to master. @ebyhr it would be really great if you could review this please

[ebyhr]

Can we extract a PR for the 2nd commit? I'm not convinced the internal format of a new type mapping.

[ebyhr]

Please rebase on master to resolve conflicts.

[glarrain-cdd]

@sbernauer Congrats on this PR.

Do you think it is going to be merged anytime soon?

Sorry to bother; it's just that it is critical for us. We'd give a hand with the code but we are Python, not Java, experts, and we aren't familiar enough with Trino internals.

[ebyhr]

it's just that it is critical for us

@glarrain-cdd Which feature (INSERT or type mapping) is critical for you?

[glarrain-cdd]

INSERT!

Thank you very much

[sbernauer]

Hi @glarrain-cdd and @ebyhr,

sorry for the slow response time. The customer I started implementing this dropped the requirement for this feature. Nevertheless I would like to get it in, just for the community purpose.

I did rework my entire PR (there were some changes regarding the TableHandle in the meantime). Additionally I dropped the whole type mapping thing, so that we can focus entirely on the INSERTs.
Would be happy about any feedback!

One open point is enabling Google Drive API access for the Google serviceuser the Trino project uses, so that we can clean up the Spreadsheets we create in the test run. I can't do that, some maintainer needs to do that.

All the best,
Sebastian

[sbernauer]

Rebased and fixed conflicts.
@ebyhr gentle reminder, a review would be great!

[sbernauer]

@mosabua thanks for the docs review! Addressed your comments

[sbernauer]

@kokosing thanks for your review! Feedback should be addressed

[mosabua]

Docs look good now

[kokosing]

% comments

[sbernauer]

@kokosing thanks again, feedback adressed

[kokosing]

Thank you. Merged. I am sorry for any delays you have experienced. This is really nice contribution.

[sbernauer]

Thanks for the quick review after posting on Slack!