neilramaswamy opened a new pull request, #45905:
URL: https://github.com/apache/spark/pull/45905

   ### What changes were proposed in this pull request?
   
   The RocksDBStateEncoder now implements range projection by reading a list of 
ordering ordinals, and using that to project certain columns, in big-endian, to 
the front of the `Array[Byte]` encoded rows returned by the encoder.
   
   ### Why are the changes needed?
   
   StateV2 implementations (and other state-related operators) project certain 
columns to the front of `UnsafeRow`s, and then rely on the RocksDBStateEncoder 
to range-encode those columns. We can avoid the initial projection by just 
passing the RocksDBStateEncoder the ordinals to encode at the front. This 
should avoid any GC or codegen overheads associated with projection.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   New UTs. All existing UTs should pass.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   Yes
   


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

Reply via email to