kevinjqliu opened a new issue, #2399:
URL: https://github.com/apache/iceberg-python/issues/2399

   ### Apache Iceberg version
   
   None
   
   ### Please describe the bug 🐞
   
   We implemented context manager for catalogs in #2390 and also included an 
implementation for sql catalog. Let's check and see if we can implement the 
`close` method for other catalog implementations
   
   - [ ] rest
   - [ ] bigquery
   - [ ] dynamodb
   - [ ] glue
   - [ ] hive
   - [ ] memory
   - [x] sql
   
   It could be that for some catalogs, nothing need to be done, i.e. memory. 
but its good to double check :) 
   
   ### Willingness to contribute
   
   - [ ] I can contribute a fix for this bug independently
   - [ ] I would be willing to contribute a fix for this bug with guidance from 
the Iceberg community
   - [ ] I cannot contribute a fix for this bug at this time


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to