Skip to content

Conversation

@Yohahaha
Copy link
Contributor

No description provided.

@Yohahaha Yohahaha marked this pull request as draft May 16, 2023 08:38
@github-actions
Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/oap-project/gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

@github-actions
Copy link

Run Gluten Clickhouse CI

@github-actions
Copy link

Run Gluten Clickhouse CI

@Yohahaha
Copy link
Contributor Author

Reproduce in velox-backend-ubuntu2004-test.

Correct result: {7, 7, 7}
Velox result: {7, 7, 7}
After convert RowVector to GlutenBatch: {null, 7, 7} or core

@Yohahaha
Copy link
Contributor Author

@rui-mo rui-mo self-assigned this May 17, 2023
@rui-mo
Copy link
Contributor

rui-mo commented May 17, 2023

@Yohahaha Thank you. I will take a look later.

@majian4work
Copy link
Contributor

This is caused by the last operator C2R, and toArray uses the memory freed by the iterator. @JkSelf has a fix maybe can resolve the problem.

@JkSelf
Copy link

JkSelf commented Jun 1, 2023

@Yohahaha This is a bug for spark 3.2 and 3.3. And we have merged PR#40914 into spark 3.3 to fix this issue. I will port this fix in Gluten later. Thanks.

@Yohahaha
Copy link
Contributor Author

Yohahaha commented Jun 1, 2023

Good to know this!

@Yohahaha Yohahaha closed this Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants