dramaticlly commented on a change in pull request #4151:
URL: https://github.com/apache/iceberg/pull/4151#discussion_r809612250
##########
File path: docs/versioned/api/python-api-intro.md
##########
@@ -36,7 +36,8 @@ To create a catalog:
from iceberg.hive import HiveTables
# instantiate Hive Tables
-conf = {"hive.metastore.uris": 'thrift://{hms_host}:{hms_port}'}
+conf = {"hive.metastore.uris": 'thrift://{hms_host}:{hms_port}',
+ "hive.metastore.warehouse.dir": tmpdir}
Review comment:
yeah I guess the legacy python API asked to supply the valid hive
metastore warehouse directory and fails the code if default to None. But I
guess `tmpdir` is not resolved directly, if needs proper setup I guess need to
use the python module
https://docs.python.org/3/library/tempfile.html#module-tempfile.
--
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]
