Hongze Zhang
Hongze Zhang
I observed this too as early as before https://github.com/apache/incubator-gluten/pull/5658 was merged. Will check again if the PR change is related.
> I observed this too as early as before #5658 was merged. Will check again if the PR change is related. I did a check and think it should not...
> there are some sed -i command in our build-velox.sh. Let's take this opportunity to put them together as other commits. +1 > Let's switch to git submodule I heard...
Do we have any updates on this PR?
I think the if conditions are no longer needed after clean up https://github.com/apache/incubator-gluten/pull/6035? Let's merge to see if everything goes fine. cc @zzcclp
@gaoyangxiaozhu Would you like to rebase ? Thanks.
``` [100%] Linking CXX shared library ../releases/libvelox.so Checking ld result of libvelox.so ld: warning: cannot find entry symbol _start; not setting start address Checking ldd result of libvelox.so linux-vdso.so.1 =>...
> we want to get rid of the transactional loading? I think it doesn't make any actual difference with loading the libs one by one. Probably we can test and...
Run Gluten Clickhouse CI
> > Is it possible for you to share some test results about this change? Thanks! > > I ran it in our grayscale adhoc query environment yesterday, and executorCpuTime...