aicam commented on PR #3724: URL: https://github.com/apache/texera/pull/3724#issuecomment-3289928252
> 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 [feat: allow datasets with the same name across different users #3715](https://github.com/apache/texera/pull/3715) at both frontend & backend to validate user-given datasetName. I agree, frontend can have a real-time validator for name for better user experience. -- 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]
