delta-rs icon indicating copy to clipboard operation
delta-rs copied to clipboard

fix: properly deserialize percent-encoded file paths of Remove actions, to make sure tombstone and file paths match

Open sigorbor opened this issue 2 years ago • 2 comments

Description

Percent-encoded file paths of Remove actions were not properly deserialized, and when compared to active file paths, the paths didn't match, which caused tombstones to be recognized as active files (be kept in the state)

sigorbor avatar Dec 13 '23 16:12 sigorbor

@roeap @rtyler @wjones127 - a test is failing likely because I'm not supposed to manually "fix" test data delta logs. Can simply push the fix without the change in the test data? The scenario is simple - Remove transaction doesn't match Add transaction if there are percent encoded paths.

sigorbor avatar Dec 13 '23 16:12 sigorbor

@rtyler @roeap @wjones127 - ping...

sigorbor avatar Dec 19 '23 17:12 sigorbor