Corwin Joy
Corwin Joy
@rtyler @adamreeve
Not quite sure why this got marked as draft since I believe it is ready to review? Also, a couple of notes: 1. I believe the full file paths are...
> It's unlikely we are going to merge something that works only for local file system paths. This needs more work to work for all various object store impls, thus...
@hntd187 @rtyler I have a related draft PR to support full file paths. Although, @rtyler mentioned that full file paths may already be in progress as part of the kernel...
@hntd187 Here is my reasoning on the deprecation flag. 1. Many of these deprecation warnings are similar to ``` warning: use of deprecated method `kernel::snapshot::EagerSnapshot::file_views_by_partitions`: Use `files` with kernel predicate...
@adamreeve @abhiaagarwal
OK. With the help of copilot I have added comments listing the warning message and what was done for each of these.
Note that fully supporting Parquet encryption requires being able to get write and read properties per-file, which is why the existing ability to set `WriterProperties` isn't sufficient, and why `WriterPropertiesFactory::create_writer_properties`...
@rtyler @roeap @alamb Tagging you here per our previous discussion on adding encryption support to delta-rs.
> I have marked this pull request as draft. This does not compile as is, I can come back to it once it is able to compile and pass unit...