John Bodley
John Bodley
@gwittel I'm in the process of trying to upgrade Airpal to work with Presto 0.144 as we (Airbnb) are in the the process of upgrading our Presto version. I started...
@dungdm93 I think in the future when Presto and Trino further diverge it makes sense, however at the moment there is a significant amount of the code that is the...
@ktmud thanks for the review. I've actually had to update the logic since your approval per https://github.com/apache/superset/pull/20729#discussion_r939048444.
Closing in favor of https://github.com/apache/superset/pull/21066.
At Airbnb we merely scrape the Knowledge Repo database for relevant metadata which is added to the Neo4j graph and Elasticsearch cluster. The actual posts are merely iframed within our...
Thanks @hashhar and @hovaesco for the review. I’ve hopefully answered/addressed all your comments/questions. I opted to downgrade the version of Poetry (from a pre-release to the latest stable version)—albeit it...
@sh4nks regarding, > While I am generally for this PR I am not sure if this might break some applications? per [here](https://realpython.com/python-pickle-module/): > Note: Newer versions of the protocol offer...
@hashhar I've addressed your comments. I addition to adding an integration test for this specific change I added additional tests which should cover the gamut of conditions for the `TrinoDialect.get_table_names`...
@mdesmet thanks for the review. I've addressed your comments.
@mdesmet thanks for the review. I've addressed your comments.