Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] [BUILD] CI passes without checking for operatorsScore.csv, supportedExprs.csv update #11896

Open
mythrocks opened this issue Dec 19, 2024 · 0 comments
Open

[BUG] [BUILD] CI passes without checking for operatorsScore.csv, supportedExprs.csv update #11896

mythrocks opened this issue Dec 19, 2024 · 0 comments
Labels
? - Needs Triage Need team to review and classify bug Something isn't working build Related to CI / CD or cleanly building

Comments

@mythrocks
Copy link
Collaborator

mythrocks commented Dec 19, 2024 •
edited
Loading

Pre-merge CI seems to be passing without checking for checking for operatorsScore.csv and supportedExprs.csv being updated appropriately.

Case in point: #11833 introduced TruncDate and TruncTimestamp support. The PR updated operatorsScore.csv and supportedExprs.csv for most Spark versions, but not all. (For instance, 3**cdh, 35*, etc. were left out.)

It would be best if the pre-merge simply didn't pass for the cases where these files weren't updated for every supported version.

(#11890 will patch the symptoms for the moment.)
@mythrocks mythrocks added ? - Needs Triage Need team to review and classify bug Something isn't working build Related to CI / CD or cleanly building labels Dec 19, 2024
@mythrocks mythrocks mentioned this issue Dec 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
? - Needs Triage Need team to review and classify bug Something isn't working build Related to CI / CD or cleanly building
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mythrocks