XinShuoWang commented on PR #5951:
URL:
https://github.com/apache/incubator-gluten/pull/5951#issuecomment-2144509002
```
template <typename T>
arrow::Status splitFixedType(const uint8_t* srcAddr, const
std::vector<uint8_t*>& dstAddrs) {
for (auto& pid : partitionUsed_) {
auto dstPidBase = (T*)(dstAddrs[pid] + partitionBufferBase_[pid] *
sizeof(T));
// record dstPidBase
...
```
@FelixYBW I added a log at the above location and found that the interval
between `dstPidBase` is about 16 KB. I think this can also be optimized to make
the interval between `dstPidBase` very small, which will help reduce cache
misses.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
