-
Notifications
You must be signed in to change notification settings - Fork 916
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
[KYUUBI #6449] Fix Gluten CI ClassNotFoundException: io.glutenproject.GlutenPlugin #6449
Conversation
@pan3793 How can I tigger Gluten CI in this PR ? |
You can modify kyuubi/.github/workflows/gluten.yml Lines 20 to 22 in 1fb1f85
|
@pan3793 Thanks. I was hoping there was a button on some web page. But it looks like there is no such button. |
technically, we can allow the job to be triggered manually, then there will be a button. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6449 +/- ##
============================================
- Coverage 58.34% 0.00% -58.35%
============================================
Files 656 677 +21
Lines 40264 41734 +1470
Branches 5498 5704 +206
============================================
- Hits 23493 0 -23493
- Misses 14272 41734 +27462
+ Partials 2499 0 -2499 ☔ View full report in Codecov by Sentry. |
@@ -20,6 +20,10 @@ name: Gluten CI | |||
on: | |||
schedule: | |||
- cron: 0 4 * * * | |||
pull_request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this before merge
<!-- provides os.detected.classifier (i.e. linux-x86_64, osx-x86_64) property --> | ||
<extension> | ||
<groupId>kr.motd.maven</groupId> | ||
<artifactId>os-maven-plugin</artifactId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we're likely to reuse this plugin for proto file gen, let's pullout it to the root pom
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
.github/workflows/gluten.yml
Outdated
VERSION=$(. /etc/os-release && echo ${VERSION_ID}) | ||
ARCH=`uname -m` | ||
mkdir -p $THIRDPARTY_LIB | ||
cp /usr/lib/${ARCH}-linux-gnu/{libre2.so.9,libdouble-conversion.so.3,libidn.so.12,\ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why should we do such copy? /usr/lib/${ARCH}-linux-gnu
should be included in ld.so.conf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build and test are run in different vm. Libraries only exists in build vm.
.github/workflows/gluten.yml
Outdated
@@ -64,6 +68,18 @@ jobs: | |||
git clone https://github.com/oap-project/gluten.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this repo should be updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
I don't have further questions, would like @ulysses-you to take a look. |
.github/workflows/gluten.yml
Outdated
libldap-2.5.so.0,liblber-2.5.so.0,libntlm.so.0,librtmp.so.1,libsasl2.so.2,libssh.so.4,\ | ||
libicudata.so.70,libthrift-0.16.0.so} $THIRDPARTY_LIB/ | ||
cp /usr/local/lib/{libprotobuf.so.32,libboost_context.so.1.84.0,\ | ||
libboost_regex.so.1.84.0} $THIRDPARTY_LIB/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we send a pr to gluten to make package.sh support collect libraries only ? So that we can use that script directly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Let me try.
@pan3793 @ulysses-you I have fixed the errors in gluten project build script But Job
It seems to be caused by Gluten's cpp code. I've uploaded the jvm error report files as artifacts. It may help the debugging.
|
@@ -61,9 +65,9 @@ jobs: | |||
- name: Build gluten project |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try remove libunwind-dev
at line46
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Let me try.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ulysses-you It failed at StepInstall dependencies
.
+ sudo --preserve-env apt install -y libc-ares-dev libcurl4-openssl-dev libssl-dev libicu-dev libdouble-conversion-dev libgoogle-glog-dev libbz2-dev libgflags-dev libevent-dev liblz4-dev libzstd-dev libre2-dev libsnappy-dev libsodium-dev libthrift-dev liblzo2-dev libelf-dev libdwarf-dev bison flex libfl-dev tzdata
WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
Reading package lists...
Building dependency tree...
Reading state information...
bison is already the newest version (2:3.8.2+dfsg-1build1).
flex is already the newest version (2.6.4-8build2).
libbz2-dev is already the newest version (1.0.8-5build1).
libbz2-dev set to manually installed.
libicu-dev is already the newest version (70.1-2).
libicu-dev set to manually installed.
liblz4-dev is already the newest version (1.9.3-2build2).
libre2-dev is already the newest version (20220201+dfsg-1).
libzstd-dev is already the newest version (1.4.8+dfsg-3build1).
libzstd-dev set to manually installed.
libthrift-dev is already the newest version (0.16.0-2).
libcurl4-openssl-dev is already the newest version (7.81.0-1ubuntu1.16).
libssl-dev is already the newest version (3.0.2-0ubuntu1.15).
tzdata is already the newest version (2024a-0ubuntu0.22.04).
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
libgoogle-glog-dev : Depends: libunwind-dev or
libunwind7-dev but it is not installable
E: Unable to correct problems, you have held broken packages.
``
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me try libunwind7-dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can remove non-required library, according to https://github.com/apache/incubator-gluten/blob/main/docs/developers/docker_ubuntu22.04.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libunwind7-dev
does not work either
Thanks for the PR! This PR is being closed due to inactivity. This isn't a judgement on the merit of the PR in any way. If this is still an issue with the latest version of Kyuubi, please reopen it and ask a committer to remove the Stale tag! Thank you for using Kyuubi! |
🔍 Description
Issue References 🔗
This pull request fixes Gluten CI Error
ClassNotFoundException: io.glutenproject.GlutenPlugin
Describe Your Solution 🔧
Change
spark.plugins
's property value fromio.glutenproject.GlutenPlugin
toorg.apache.gluten.GlutenPlugin
.Types of changes 🔖
Test Plan 🧪
Behavior Without This Pull Request ⚰️
Behavior With This Pull Request 🎉
Related Unit Tests
Checklist 📝
Be nice. Be informative.