+1 (binding)
I verified:
- sigs, sums, licenses, tests
I tested:
- REST Catalog implementation
- creating/loading/renaming/dropping tables
- creating/removing namespace properties
- loading table to arrow dataframe via scan (and pandas via arrow)
- loading table to duckdb
I had one test failure locally (though it appears to pass in CI, so I
consider this a non-blocker unless it is affecting everyone)
test_missing_uri
def test_missing_uri():
runner = CliRunner()
result = runner.invoke(run, ["list"])
> assert result.exit_code == 1
E assert 0 == 1
E + where 0 = <Result okay>.exit_code
tests/cli/test_console.py:140: AssertionError
FAILED tests/cli/test_console.py::test_missing_uri - assert 0 == 1
A few things I noted:
- I cannot import 'load_catalog' without the pyarrow dependency. So that
seems like a hard dependency at this point and we should workaround it in
the future.
On Mon, Dec 5, 2022 at 9:05 AM Jun H. <junhe...@gmail.com> wrote:
> Hi Everyone,
>
> I propose that we release the following RC as the official PyIceberg 0.2.0
> release.
>
> The commit ID is 577867e88da86ab70f4efcb12ab993d01062712a
> * This corresponds to the tag: pyiceberg-0.2.0rc1
> (509a38cc1f08f399712c1e0a65e53f1ad7749153)
> * https://github.com/apache/iceberg/releases/tag/pyiceberg-0.2.0rc1
> *
> https://github.com/apache/iceberg/tree/577867e88da86ab70f4efcb12ab993d01062712a
>
> The release tarball, signature, and checksums are here:
> * https://dist.apache.org/repos/dist/dev/iceberg/pyiceberg-0.2.0rc1/
>
> You can find the KEYS file here:
> * https://dist.apache.org/repos/dist/dev/iceberg/KEYS
>
> Convenience binary artifacts are staged on pypi:
> https://pypi.org/project/pyiceberg/0.2.0rc1/
>
> And can be installed using: pip3 install pyiceberg==0.2.0rc1
>
> Please download, verify, and test.
>
> Please vote in the next 72 hours.
>
> [ ] +1 Release this as PyIceberg 0.2.0
> [ ] +0
> [ ] -1 Do not release this because...
>
>
