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.

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

Apache Arrow integration with substrait consumer-testing job is failing and not maintained - Removal proposal #196

Open
raulcd opened this issue Jan 14, 2025 · 1 comment
Assignees

Comments

@raulcd
Copy link

raulcd commented Jan 14, 2025

There is an integration nightly job which uses the substrait consumer-testing on the Apache Arrow repository that has been failing for 5 weeks. See:

Previous maintainers of this repository added that job for integration but the job does not seem to be maintained anymore.

I am going to propose the removal of the Job to the Arrow community and I just wanted to validate if there is any concern from the substrait community or if someone wants to maintain it.

Thanks.

@raulcd
Copy link
Author

raulcd commented Feb 7, 2025

I've created a PR to remove the integration job from Arrow:
apache/arrow#45463

FYI @ingomueller-net @EpsilonPrime
As you were the ones that did the consumer-testing refactor.

raulcd added a commit to apache/arrow that referenced this issue Feb 13, 2025
…tion job (#45463)

### Rationale for this change

The job has been failing for the last two months due to upstream refactoring. I opened an issue upstream but we didn't got any response:
- substrait-io/consumer-testing#196

Based on the following substrait consumer-testing PR there was a big refactor which removed the files that we were invoking on our tests:
- substrait-io/consumer-testing#162

I asked on the corresponding issue ~3 weeks ago for feedback on the removal and as there's no opposition I think we should remove this unmaintained job.

### What changes are included in this PR?

Remove extremely coupled `substrait-io/consumer-testing` job.

### Are these changes tested?

No

### Are there any user-facing changes?

No
* GitHub Issue: #45159

Authored-by: Raúl Cumplido <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants