Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[NSE-1193] fix jni unload #1180

Merged
merged 1 commit into from
Dec 16, 2022
Merged

Conversation

zhouyuan
Copy link
Collaborator

What changes were proposed in this pull request?

unload native libs on spark context exit
Signed-off-by: Yuan Zhou yuan.zhou@intel.com

How was this patch tested?

pass jenkins

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
@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/native-sql-engine/issues

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

[NSE-${ISSUES_ID}] ${detailed message}

See also:

@zhouyuan zhouyuan marked this pull request as ready for review December 14, 2022 07:34
@zhouyuan zhouyuan changed the title fix jni unload [NSE-1193] fix jni unload Dec 16, 2022
@github-actions
Copy link

#1193

@zhouyuan zhouyuan merged commit 6d2fbd1 into oap-project:main Dec 16, 2022
zhouyuan added a commit to zhouyuan/native-sql-engine that referenced this pull request Dec 16, 2022
fix the seg fault on spark-shell exit

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
zhouyuan added a commit that referenced this pull request Dec 16, 2022
* fix jni unload (#1180)

fix the seg fault on spark-shell exit

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>

* [NSE-1191] fix AQE exchange reuse in Spark3.2 (#1192)

Fix AQE exchange reuse in Spark3.2

Note with this patch, the jars for Spark321 and Spark322 are separated.
- to build jar for Spark322 use -Pspark-3.2.2
- to build jar for Spark321 use -Pspark3.2.1

Signed-off-by: Yuan Zhou yuan.zhou@intel.com

Signed-off-by: Yuan Zhou <yuan.zhou@intel.com>
Signed-off-by: Yuan Zhou yuan.zhou@intel.com
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant