Devin Smith
Devin Smith
Looks like they infer the type if the `uri` property is provided. https://github.com/apache/iceberg-python/blob/pyiceberg-0.6.1/pyiceberg/catalog/__init__.py#L155C5-L182 ```python if uri.startswith("http"): return CatalogType.REST elif uri.startswith("thrift"): return CatalogType.HIVE elif uri.startswith(("sqlite", "postgresql")): return CatalogType.SQL ```
@dependabot rebase
The change to include `core.*` was originally made in #2074
pydoc on https://deephaven.io/core/pydoc/code/deephaven.table.html#deephaven.table.Table.columns seems lacking. IMO, if we were re-writing `.columns`, I think it would be better exposed as a dict. As it stands today, we might want a different...
https://github.com/deephaven/deephaven-core/pull/5892 is poc
From Colin: https://github.com/jlumbroso/free-disk-space https://github.com/apache/flink/blob/18c8fa2c92bf926a285e8a4889583533f0749270/.github/workflows/template.flink-ci.yml#L174-L175 as potential ways to clean up at a runner environment level if necessary.
The actual test image being created (`deephaven/test-c-python-client-ticking-3.9-ubi-minimal:local-build`) isn't _too_ big:  The entrypoint code doesn't seem to be that big either: ``` $ docker run -it --name test --entrypoint bash...
The bigger culprit may be the image builder image: ``` $ docker images REPOSITORY TAG IMAGE ID CREATED SIZE deephaven/test-c-python-client-ticking-3.9-ubi-minimal local-build af68b8e823ac 44 minutes ago 395MB deephaven/py-client-ticking-wheel-cp39-cp39 local-build 0bb83c4024f6 46...
It's possible that we've relied on >14GB of disk space, and just more recently, GH CI has reduced the amount of disk space: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories https://github.com/actions/runner-images/pull/9251