techascent / tmducken

tech.ml.dataset integration with duckdb
MIT License
58 stars 5 forks source link

`[duckdb-0.8.1]` Toward #2

Closed harold closed 1 year ago

harold commented 1 year ago
harold@freeside:~/src/tmducken [duckdb-0.8.1]$ ./scripts/run-tests 

Running tests in #{"test"}
13:42:22.513 [main] INFO  tmducken.duckdb - Attempting to load duckdb from "/home/harold/src/tmducken/binaries/libduckdb.so"

Testing tmducken.duckdb-test
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by tech.v3.datatype.UnsafeUtil (file:/home/harold/.m2/repository/cnuernber/dtype-next/10.000-beta-51/dtype-next-10.000-beta-51.jar) to constructor java.nio.DirectByteBuffer(long,int)
WARNING: Please consider reporting this to the maintainers of tech.v3.datatype.UnsafeUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

This got surprisingly far before these "illegal reflective access operation" warnings...

Then the tests fail, because the result set has no rows (afaict) ... not sure if that's because they failed to be inserted, or because they failed to be returned, or?

Perhaps we should also fork this into a techascent repo? Or perhaps add me as a contributor on your fork?

harold commented 1 year ago

I changed my mind - I'm going to do this in a TechAscent fork of this repo and add you as a contributor. (: