wombatu-kun commented on PR #16498: URL: https://github.com/apache/iceberg/pull/16498#issuecomment-4531110829
Thanks @sungwy. I'd push back on the need for a separate `[DISCUSS]` thread here. This is a straightforward bug fix, not an intentional behavior change — the `startsWith(location)` filter matched sibling prefixes only because it was missing the trailing path separator. More to the point, both the problem and the fix direction were already settled on the issue itself — it was raised by @rdblue and triaged by @RussellSpitzer as a normal bug worth fixing. With the project's most senior maintainers having already agreed on the rationale and the approach, and given how minimal the change is, a dev-list thread or survey would mostly duplicate that and ask the community to re-decide something that's effectively already decided. @rdblue @RussellSpitzer — does a separate `[DISCUSS]` email add anything here, or does #16493 already cover it? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
