Eduard Tudenhoefner
Eduard Tudenhoefner
@lordk911 can you please re-try running the smoke test with the fix proposed in https://github.com/apache/iceberg/pull/5819?
@snazy I think it makes sense to have this for the post-1.0 milestone, wdyt?
@beinan yes that makes perfect sense to do that. I added some cleanup after each test that deletes all branches/tags and also improved the overall isolation in the tests so...
@beinan turns out this isn't as trivial as I thought, because the query runner with nessie is usually created at the class level, while we clean up and therefore modify...
> @beinan turns out this isn't as trivial as I thought, because the query runner with nessie is usually created at the class level, while we clean up and therefore...
@beinan could you re-review this one please?
@beinan @ChunxuTang is this something that we can merge?
@beinan any idea why a different and unrelated CI action fails randomly?
@ChunxuTang looks like CI is passing, is this good to be merged then?
> * Do we need a request object that adds another layer to the JSON? This is minor but it seems odd to have a record that just wraps another...