isc-psulin

pip install outside of IPM still has the same problems, if using the --only-binary=:all:. I should have been more clear in my initial post. By args to pip, I meant...

I'm not sure I understand your first note: - When IPM calls out to pip install, it doesn't know the python version associated with the pip executable. By default, pip...

I can certainly add tests, but that is really testing the Metadata.create_all() function in SQLAlchemy and not pyiceberg code. I think that CREATE TABLE IF NOT EXISTS requires same permissions...

It does not look like CREATE TABLE IF NOT EXISTS is what is used. It is specific to each dialect and how they implement the has_table method, but the ones...

My thought was to pass an additional argument into init that allowed you to turn off creation of tables. My concern is that I don't think you will want this...

I had not yet seen the change introducing load_catalog instead of using the SQLCatalog constructor, so just updated to v0.70 - I was away for couple weeks. My point for...

I'm not sure how much the Python API is trying to conform to the Java API, but this same feature for the same reason was requested and implemented for the...

I think you were correct about removing ensure_tables_exist. The question comes down to what is the best method to use for checking the existence of a table. Create_all() relies on...

The problem I see with moving into the direction of creating integration tests for specific DB's(outside of SQLite) is that you are opening up to an enormous amount of possible...

Sounds good. I added ensure_tables_exist back to the code along with the new flag and the tests in the branch with the PR.