eric-maynard commented on PR #1532: URL: https://github.com/apache/polaris/pull/1532#issuecomment-2868619084
> INITIALIZE_DEFAULT_CATALOG_FILEIO_FOR_TEST was already present, but I don’t think that justifies promoting it to a first-class config flag. +1, it was left out of the original `PolarisConfiguration` change for a reason and this is also what I am saying [here](https://github.com/apache/polaris/pull/1532#discussion_r2075949695) As for removing it, I don't really have an opinion. It's certainly not a change that seems necessary to fix any kind of bug. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
