Yicong-Huang commented on PR #4225:
URL: https://github.com/apache/texera/pull/4225#issuecomment-3937263711

   > We’re not reducing the test environments at this time. This PR was opened 
before the team discussion as a concrete example of what would change; the 
topic is still under discussion, so this shouldn’t be merged until we align as 
a team.
   > 
   These kind of discussions should have been public. You will even need to 
cast a vote as it is quite important decision. I don't see any linked issues or 
discussions from this PR, where are the discussions happening?
   
   > Reason for PR: The project wiki currently supports only Python 3.12, and 
some operators under development are encountering compatibility issues with the 
Python 3.10 test case. Given that we don’t currently state a requirement for 
backward compatibility (or support for older Python versions), keeping the same 
OS while varying only the Python version doesn’t seem to provide much signal 
right now.
   For all the benefits, it is better to put the reason in the PR description 
(and the related issues, if any) so reviewers can have the context. For this 
specific topic, Python CI is intended to test Python UDF engine support, not 
python-native operators. If some operators require old Python version, that 
should be discussed separately (and/or to be tested separately). It should not 
affect our ability to support python UDF in general. 
   


-- 
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]

Reply via email to