MINOR: [C++] Make the utilities used to implement run_end_encode available to other compute kernels #35002
Conversation
felipecrv
force-pushed
the
ree_compute_util
branch
from
6cf6c26 to
c3db551
Compare
…e_util_internal.cc/h
…ove it out of header This helps reduce binary bloat as well.
felipecrv
force-pushed
the
ree_compute_util
branch
from
c3db551 to
3a3d1cc
Compare
github-actions
bot
added
awaiting merge
|
Benchmark runs are scheduled for baseline = 913c345 and contender = ff94702. ff94702 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
|
['Python', 'R'] benchmarks have high level of regressions. |
…lable to other compute kernels (apache#35002) ### Rationale for this change Make it easier to implement correct kernels dealing with run-end encoded data. ### What changes are included in this PR? Extraction of code to a header so kernels like REE filter kernels (see apache#35001) can use them. ### Are these changes tested? Indirectly via the `run_end_encode` and `run_end_decode` tests. ### Are there any user-facing changes? No, these are internal compute headers. Authored-by: Felipe Oliveira Carvalho <felipekde@gmail.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
…lable to other compute kernels (apache#35002) ### Rationale for this change Make it easier to implement correct kernels dealing with run-end encoded data. ### What changes are included in this PR? Extraction of code to a header so kernels like REE filter kernels (see apache#35001) can use them. ### Are these changes tested? Indirectly via the `run_end_encode` and `run_end_decode` tests. ### Are there any user-facing changes? No, these are internal compute headers. Authored-by: Felipe Oliveira Carvalho <felipekde@gmail.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
…lable to other compute kernels (apache#35002) ### Rationale for this change Make it easier to implement correct kernels dealing with run-end encoded data. ### What changes are included in this PR? Extraction of code to a header so kernels like REE filter kernels (see apache#35001) can use them. ### Are these changes tested? Indirectly via the `run_end_encode` and `run_end_decode` tests. ### Are there any user-facing changes? No, these are internal compute headers. Authored-by: Felipe Oliveira Carvalho <felipekde@gmail.com> Signed-off-by: Matt Topol <zotthewizard@gmail.com>
Rationale for this change
Make it easier to implement correct kernels dealing with run-end encoded data.
What changes are included in this PR?
Extraction of code to a header so kernels like REE filter kernels (see #35001) can use them.
Are these changes tested?
Indirectly via the
run_end_encodeandrun_end_decodetests.Are there any user-facing changes?
No, these are internal compute headers.