jerryshao commented on issue #4409: URL: https://github.com/apache/gravitino/issues/4409#issuecomment-2274806550
Let me clarify more on this. I think we deliberately chose switch..case instead of if..else here, because we will add more relation types here later on. Also, I don't see any readable problem here using switch..case, so I'm inclined not doing such refactoring unless we really find issues here. If you have different opinions on this, you can leave messages here, otherwise I will close this issue to avoid misleading to many new contributors. -- 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]
