aglinxinyuan commented on PR #4138: URL: https://github.com/apache/texera/pull/4138#issuecomment-3695266315
> @aglinxinyuan, this content of this PR was suggested by me in a discussion with @carloea2 . I agree with your comments though. We could spllit this PR into two. The first just remove unnecessary user information, and the second one will include the refactoring in the current PR. What do you think? Sure. Introducing a `fields` parameter when the only supported value is `name` feels unnecessary to me. I don’t have the context from the follow-up discussion, but this would make more sense once there are multiple possible values. We can introduce the `fields` parameter later, when we actually need that flexibility. -- 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]
