Zach

Results 52 comments of Zach

Good idea. We should try to implement this generically, so that we can add it to DeltaLakeTableDataObject, IcebergTableDataObject and even HiveTableDataObject?

Iceberg tables also support reading and writing branches of the table. On other technologies (DeltaLake, Snowflake) this requires creating a "zero-copy-clone" of a table. We could have a naming convention...

Hi @AdiWehrli, thanks for the suggestion. You are right, the apache common-pool2 does not support testing JDBC connection. Its also not specifically designed for DB Connection, but a generic pool....

Hi @AdiWehrli, i created a draft for a new configuration attribute JdbcTableConnection.connectionPoolClassName. To configure your own ConnectionPool, you could implement a class class MyJdbcConnectionPool(jdbcTableConnection: JdbcTableConnection) extends ConnectionPool[java.sql.Connection] { override def...

Note: Spark 3.5 needs additional add-opens parameter for Java11+: "--add-exports java.base/sun.nio.ch=ALL-UNNAMED --add-opens java.base/java.nio=ALL-UNNAMED"

Frontend-build fails with a strange error: ``` Error: error:0308010C:digital envelope routines::unsupported at new Hash (node:internal/crypto/hash:71:19) at Object.createHash (node:crypto:133:10) at BulkUpdateDecorator.hashFactory (/home/runner/work/polynote/polynote/polynote-frontend/node_modules/webpack/lib/util/createHash.js:138:18) at BulkUpdateDecorator.update (/home/runner/work/polynote/polynote/polynote-frontend/node_modules/webpack/lib/util/createHash.js:44:50) at OriginalSource.updateHash (/home/runner/work/polynote/polynote/polynote-frontend/node_modules/webpack-sources/lib/OriginalSource.js:104:8) at NormalModule._initBuildHash (/home/runner/work/polynote/polynote/polynote-frontend/node_modules/webpack/lib/NormalModule.js:737:17)...

I guess the error is because i'm missing some permissions on the project, can you please check?

@omidmogasemi: Build works, again. Great! Find attached a small video: https://user-images.githubusercontent.com/13715754/224515820-43f217fd-e8d6-4c1a-871c-74bbbc38f34a.mp4

> I see this when I try to run this locally - I believe it might be due to the removed padding issues? I just opened a regular version on...