Christopher Lambert

Results 30 comments of Christopher Lambert

in tox we only run linting on python 3.8 https://github.com/projectnessie/nessie/blob/2839f66071791153e134444a047f3c645e1e1cb2/python/tox.ini#L20-L26 but flake8 is part of the basedev requirements so is installed always: https://github.com/projectnessie/nessie/blob/2839f66071791153e134444a047f3c645e1e1cb2/python/requirements_basedev.txt#L23 if flake8 was in `requirements_lint.txt` we could...

could it make sense to add methods to the builder to generate common filter expressions? or at least override `QueryBuilder.filter` method to provide javadoc how common cel filters would look...

not "all things CEL" but the suggestion would be to only build context specific things like: `String buildFilterEntryKeyLength(int size)` `String buildFilterEntryKeyStartsWith(List keyElements)` but this assumes that there are "common filter"-usecases...

we're trying to upgrade from `3.21.9` to `3.21.12` and are now hitting the same problem as in the ticket description: ``` Caused By (java.lang.NoSuchMethodError) java.nio.ByteBuffer.position(I)Ljava/nio/ByteBuffer; com.google.protobuf.IterableByteBufferInputStream.read():143 com.google.protobuf.CodedInputStream$StreamDecoder.read():2080 com.google.protobuf.CodedInputStream$StreamDecoder.tryRefillBuffer():2831 com.google.protobuf.CodedInputStream$StreamDecoder.isAtEnd():2754 com.google.protobuf.CodedInputStream$StreamDecoder.readTag():2107...

@fowles @deannagarcia i apologize for the ping but isnt this a somewhat serious problem? it cant have been intentional to break java 8 compatibility between patch versions

@deannagarcia thanks for the quick response, before my ping I had tried 3.22.0 but i was still hitting the same problem in our build

`getProgramOptions` is used here: https://github.com/projectnessie/cel-java/blob/c966875913fbf9800a1ac5f7ca73031977cda520/core/src/main/java/org/projectnessie/cel/Library.java#L54-L66 which gets used here in `ScriptHost`: https://github.com/projectnessie/cel-java/blob/c966875913fbf9800a1ac5f7ca73031977cda520/tools/src/main/java/org/projectnessie/cel/tools/ScriptHost.java#L123-L125 also note that we have a test for `ScriptHost` with a custom library here: https://github.com/projectnessie/cel-java/blob/5749d573772fd34673c7f41456e4ebb59ff6a415/tools/src/test/java/org/projectnessie/cel/tools/ScriptHostTest.java#L116-L139 from that example...

running this with the mybinder.org links i had days were this was working fine and days when the kernel died quite quickly as reported by Tomasz.... did not notice a...

> Please tell me how to do it with pyiceberg generally speaking you use the REST catalog these docs may help: https://py.iceberg.apache.org/configuration/#rest-catalog https://kevinjqliu.substack.com/i/147257480/connect-to-the-rest-catalog running the nessie server: https://projectnessie.org/guides/iceberg-rest/

`RestCatalog` class seems to live in `pyiceberg.catalog.rest`: https://github.com/apache/iceberg-python/blob/c30e43adf94a82ec1a225d3a1bf69fface592cfd/pyiceberg/catalog/rest.py#L248 however according to https://py.iceberg.apache.org/api/ one is now supposed to use something like: ``` from pyiceberg.catalog import load_catalog catalog = load_catalog("rest", ) ```