Yicong-Huang commented on PR #4024: URL: https://github.com/apache/texera/pull/4024#issuecomment-3544711603
> Test cases should be part of the PR. @Yicong-Huang Do you like the idea of splitting it into 5 PRs? We don't need to split it into 5 PRs. those 5 methods are related. I think it makes sense to refactor them all in one PR. But all of them should be tested, so tests should be included in the same PR. just my two cents. -- 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]
