Hongze Zhang

Results 100 comments of 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...

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...

> > 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...