sunchao commented on pull request #7037:
URL: https://github.com/apache/arrow/pull/7037#issuecomment-622272292


   This definitely looks great from # of code deduction :D , but yeah it will 
be better if we can keep the perf loss minimum. 
   
   > Also the future of packed_simd is unclear and it's one less dependency on 
nightly, now we just need specialization on stable...
   
   I'm not seeing a light from this tunnel yet - maybe we could try to replace 
it with `[min_specialization](https://github.com/rust-lang/rust/pull/68970)` 
which seems more promising in terms of stabilization. But I found very little 
documentation on this so far.
   
   Another way is to rewrite the code (I think it is mostly used in Parquet 
encoder/decoder and Arrow array builders?). It is tedious work but I think it 
should be do-able.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to