rdblue merged PR #178:
URL: https://github.com/apache/iceberg-python/pull/178
--
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: issues-unsubscr...@iceberg
Fokko commented on PR #178:
URL: https://github.com/apache/iceberg-python/pull/178#issuecomment-1841269395
Thanks for the suggestions @bitsondatadev 🙌
--
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 g
bitsondatadev commented on code in PR #178:
URL: https://github.com/apache/iceberg-python/pull/178#discussion_r1415369470
##
tests/catalog/test_sql.py:
##
@@ -21,6 +21,7 @@
import pytest
from pytest import TempPathFactory
+from pytest_lazyfixture import lazy_fixture
Review
Fokko commented on PR #178:
URL: https://github.com/apache/iceberg-python/pull/178#issuecomment-1839491704
@jayceslesar Thanks for chiming in here! If I'm not mistaken, PyIceberg
could serve as a backend behind the Ibis front end. We currently use sqlalchemy
as an ORM for different database
jayceslesar commented on PR #178:
URL: https://github.com/apache/iceberg-python/pull/178#issuecomment-1839457485
You might be able to support arbitrary SQL-like catalogs using
https://github.com/ibis-project/ibis
--
This is an automated message from the Apache Git Service.
To respond to t
Fokko opened a new pull request, #178:
URL: https://github.com/apache/iceberg-python/pull/178
--
This is an automated message from the Apache Git Service.
To respond to the message,
