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

[CI] Test substrait integration job has been failing for some time #39425

Closed
raulcd opened this issue Jan 2, 2024 · 0 comments · Fixed by #39426
Closed

[CI] Test substrait integration job has been failing for some time #39425

raulcd opened this issue Jan 2, 2024 · 0 comments · Fixed by #39426
Assignees
@raulcd
Labels
Component: Continuous Integration Type: bug
Milestone
15.0.0

Comments

@raulcd
Copy link
Member

raulcd commented Jan 2, 2024

Describe the bug, including details regarding any error messages, version, and platform.

The following nightly job:

has been failing with the following:

 Validating imports
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: cannot import name 'AceroConsumer' from 'substrait_consumer.consumers' (/opt/conda/envs/arrow/lib/python3.10/site-packages/substrait_consumer-0.0.1-py3.10.egg/substrait_consumer/consumers/__init__.py)
1

It seems there was a refactor on how consumers work on this commit: substrait-io/consumer-testing@4d650fc

Component(s)

Continuous Integration
raulcd added a commit to raulcd/arrow that referenced this issue Jan 2, 2024
@raulcd
apacheGH-39425: [CI] Fix import to match new substrait repo structure
1d0226a
@github-actions github-actions bot mentioned this issue Jan 2, 2024
Merged
@raulcd raulcd added this to the 15.0.0 milestone Jan 2, 2024
raulcd added a commit that referenced this issue Jan 3, 2024
@raulcd
GH-39425: [CI] Fix import to match new substrait repo structure (#39426)
fe38d0e 
### Rationale for this change

Upstream substrait repo did a small refactor. We have to update our imports to match the new structure.

### What changes are included in this PR?

Update import

### Are these changes tested?
Via archery

### Are there any user-facing changes?
No
* Closes: #39425

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
clayburn pushed a commit to clayburn/arrow that referenced this issue Jan 23, 2024
@raulcd @clayburn
apacheGH-39425: [CI] Fix import to match new substrait repo structure (…
a2bf56f 
…apache#39426)

### Rationale for this change

Upstream substrait repo did a small refactor. We have to update our imports to match the new structure.

### What changes are included in this PR?

Update import

### Are these changes tested?
Via archery

### Are there any user-facing changes?
No
* Closes: apache#39425

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
dgreiss pushed a commit to dgreiss/arrow that referenced this issue Feb 19, 2024
@raulcd @dgreiss
apacheGH-39425: [CI] Fix import to match new substrait repo structure (…
cb7c7ee 
…apache#39426)

### Rationale for this change

Upstream substrait repo did a small refactor. We have to update our imports to match the new structure.

### What changes are included in this PR?

Update import

### Are these changes tested?
Via archery

### Are there any user-facing changes?
No
* Closes: apache#39425

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
@raulcd @zanmato1984
apacheGH-39425: [CI] Fix import to match new substrait repo structure (…
94404f0 
…apache#39426)

### Rationale for this change

Upstream substrait repo did a small refactor. We have to update our imports to match the new structure.

### What changes are included in this PR?

Update import

### Are these changes tested?
Via archery

### Are there any user-facing changes?
No
* Closes: apache#39425

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raulcd raulcd

Labels
Component: Continuous Integration Type: bug
Projects
None yet
Milestone
15.0.0
Development

Successfully merging a pull request may close this issue.

GH-39425: [CI] Fix import to match new substrait repo structure raulcd/arrow
1 participant
@raulcd