AnzhiZhang commented on PR #4006: URL: https://github.com/apache/texera/pull/4006#issuecomment-3455969629
> @AnzhiZhang Can you confirm if this PR is minimal? I asked @aicam and @Xiao-zhen-Liu to review it. Yes, this is minimal. The added dependency is required to use these annotations. And all other file changes are adding validation annotations to basic chart operators. In fact, many other operators need these validation annotations to have a non-blank requirement in the JSON schema. The issue only mentions charts, so I changed the basic charts only. Changing other operators can be a long-term plan. -- 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]
