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

Fix index out of bounds panic #819

Merged
merged 5 commits into from
Jun 28, 2023
Merged

Fix index out of bounds panic #819

merged 5 commits into from
Jun 28, 2023

Conversation

yahoNanJing
Copy link
Contributor

@yahoNanJing yahoNanJing commented Jun 27, 2023 •
edited
Loading

Which issue does this PR close?

Closes #818.

Rationale for this change

What changes are included in this PR?

  • The output_partitioning of TaskDescription is removed
  • The output_partitioning of execution stages is removed
  • The input_partition_count of UnresolvedShuffleExec is removed
  • The input stage id is added to the ShuffleReaderExec
  • The behavior of output_partitioning() of ShuffleWriterExec is corrected

Are there any user-facing changes?

@yahoNanJing
Copy link
Contributor Author

Hi @thinkharderdev and @Dandandan, could you help review this PR?

@yahoNanJing yahoNanJing merged commit 553b9a7 into apache:main Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thinkharderdev thinkharderdev Awaiting requested review from thinkharderdev

@andygrove andygrove Awaiting requested review from andygrove

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix panics at 'index out of bounds: the len is 0 but the index is 0': ballista/scheduler/src/planner.rs
2 participants
@yahoNanJing @kyotoYaho