chore: remove used_coding as it is not used currently

[broccoliSpicy]

remove used_coding as it is not used currently, we can add it back when we needed.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.92%. Comparing base (f60283e) to head (41f4e93).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3081      +/-   ##
==========================================
+ Coverage   77.87%   77.92%   +0.05%     
==========================================
  Files         240      240              
  Lines       81630    81564      -66     
  Branches    81630    81564      -66     
==========================================
- Hits        63566    63559       -7     
+ Misses      14830    14810      -20     
+ Partials     3234     3195      -39     

Flag	Coverage Δ
unittests	77.92% <ø> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[westonpace]

Thanks for cleaning up.

Not a rush, but we might want to consider adding data block constructors (e.g. new) at some point so that something like...

FixedWidthDataBlock {
    bits_per_value: self.bytes_per_value * 8,
    data,
    num_values,
    block_info: BlockInfo::new(),
    used_encoding: UsedEncoding::new()
}

Can become...

FixedWidthDataBlock::new(self.bytes_per_value * 8, data, num_values)

Then we can hide any default / newly-initialized fields (e.g. block_info).