Michiel De Smet

> The borrowed code is in `IcebergUtil`  > > * common methods `toIdentifier` and `fromIdentifier` > * and common regex I had also raised this in the PR. This would...

> @mdesmet https://github.com/trinodb/trino/pull/12227/commits seems to the commit only `toIdentifier` and `fromIdentifier` methods in `IcebergUtil` is removed, as I was trying to cherry-pick that. Please see, if you have time to...

The CI issues can be simulated by following code. Seems that although the result has returned the Trino API still provides one or more next_uri's to fetch in a minority...

> @hashhar would you agree that to address the original issue we should add a fetch call after `scalar()` to drain the cursor? We might also want to document this...

> However this is also a breaking change (even though it improves experience) so I'm approving it but not merging until we release a version from current `master` so that...

> Newer Trino versions will include [trinodb/trino#14122](https://github.com/trinodb/trino/pull/14122) which can mean CI can be green without this change. > > I think we should add one more entry to matrix with...

@hashhar: Squashed those commits. Are we good to go?

Note in our docs: https://trinodb.slack.com/archives/CFPVDCDHV/p1657883633509519 A callback to handle the redirect url can be provided via param `redirect_auth_url_handler` of the `trino.auth.OAuth2Authentication` class. By default, it will try to launch a...

True. By default it launches a webbrowser and it is up to the user to authenticate itself on the OAuth provider. Now if you want to automate that you would...

Are you saying that it doesn't launch your webbrowser? Can you try following code: ``` import webbrowser webbrowser.open_new("https://trino.io/") ``` If not working, could you tell us your operation system and...