[VL] #6942 breaks shared lib loader

[Yohahaha]

Problem description

#6942 use System.mapLibraryName(libName) to get lib name, which will convert libboost_atomic.so.1.84.0 to libboost_atomic.so.1.84.0.so, it's wrong.

System information

n/a

CMake log

No response

[Yohahaha]

Caused by: java.io.FileNotFoundException: liblibboost_atomic.so.1.84.0.so
	at org.apache.gluten.jni.JniLibLoader.moveToWorkDir(JniLibLoader.java:206)
	at org.apache.gluten.jni.JniLibLoader.loadAndCreateLink(JniLibLoader.java:184)
	... 42 more

[Yohahaha]

@PHILO-HE @zhouyuan could we keep one CI for dynamic lib packing ?

[zhztheplayer]

Would someone like to fix this first ? Seems to be a minor one. cc @wForget

[Yohahaha]

Would someone like to fix this first ? Seems to be a minor one. cc @wForget

I will fix this issue later.

[wForget]

#6942 use System.mapLibraryName(libName) to get lib name

Is this the root cause? The previous behavior also seemed to use System.mapLibraryName

incubator-gluten/gluten-substrait/src/main/java/org/apache/gluten/vectorized/JniLibLoader.java

Line 195 in 3af5bac

File file = moveToWorkDir(workDir, System.mapLibraryName(libName));

[wForget]

Introduced in #6791?

[wForget]

System.mapLibraryName does result in an incorrect libname, we can remove it in SharedLibraryLoader

[wForget]

@PHILO-HE @zhouyuan could we keep one CI for dynamic lib packing ?

How about adding a run TPC-H / TPC-DS step to velox_weekly.yml?

[zhouyuan]

@Yohahaha @wForget
This is the annoying issue I mentioned to you before, we need to carefully maintain those codes, with different platforms.
Yes, the CentOS 8 based tests is still using dynamic packaging method, the gap is there's not TPCH/DS test on it. Using gluten-it should be able to add the tests quickly. Let me try to add one for this

thanks, -yuan