Daniel Weeks

Results 62 comments of Daniel Weeks

> Not having View or MV support is pretty limiting. Is that a restriction from the backend, or can we implement them as follow up? We definitely want to add...

> Lett initial comments. Is it possible to add a test for `iceberg.rest.token` and `iceberg.rest.credential`? @ebyhr It may be possible to add tests to validate that the token and credential...

@alexjo2144 and @ebyhr I think we've addressed all the comments and (thanks to a few upstream updates), the tests updates as well. Please take a look when you have a...

Thanks @alexjo2144, if there aren't any more major comments, we'd really like to get this in now that it's a rather large PR. Happy to keep iterating with smaller PRs...

The way Hive resolves this is that the client never reads the footer (only hive metadata), but when the task processes the file it will read the footer and resolve...

@saucam it looks like in hive they push the filter expression to the task side as well and evaluate it there. However, I don't see that they address the case...

@rizaon and @rdblue I made a comment in one of the replies above but wanted to ask about potentially bad behaviors around reading metadata tables. Since in those cases the...

Hey @rizaon, some recent changes actually exposed FileIO properties, which I believe would simplify how we can cache manifest files. Where we [read manifest files](https://github.com/apache/iceberg/blob/master/core/src/main/java/org/apache/iceberg/ManifestFiles.java#L85), we now have the file...

@rizaon because we want to isolate the caching to metadata files only, I think MetadataFiles.java would be a good place to start. We might want some utilities so if we...

Hey @rizaon I think this is getting real close now (thanks for sticking with us on this). Now that we've gotten this far, I see there's a little more we...