bobbai00 commented on PR #3724: URL: https://github.com/apache/texera/pull/3724#issuecomment-3289701115
I think this refactor is not needed because: 1. LakeFS's `initRepo` already validates the repository name. If we extract the validation logic, it will introduce redundancy. Later, say if LakeFS has changed the validation rule, our ow validation function becomes out-of-sync 2. I understand that we need a way validate the user-given datasetName. In this case, we can simply add a function in #3715 at both frontend & backend to validate user-given datasetName. -- 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]
