chunjun icon indicating copy to clipboard operation
chunjun copied to clipboard

[feat-1145][convert] split BigDecimalColumn

Open Paddy0523 opened this issue 3 years ago • 0 comments

Purpose of this pull request

Split BigDecimalColumn into ShortColumn, IntColumn, LongColumn, FloatColumn, and DoubleColumn

  • Corrected some incorrect type mappings
  • Optimized part of the serialization code

Which issue you fix

Fixes # (issue). #1145 float=3.1234123 result of old code image

result of my commit image

Checklist:

  • [ ] I have executed the 'mvn spotless:apply' command to format my code.
  • [ ] I have a meaningful commit message (including the issue id, the template of commit message is '[label-type-#issue-id][fixed-module] a meaningful commit message.')
  • [ ] I have performed a self-review of my own code.
  • [ ] I have commented my code, particularly in hard-to-understand areas.
  • [ ] I have made corresponding changes to the documentation.
  • [ ] I have added tests that prove my fix is effective or that my feature works.
  • [ ] New and existing unit tests pass locally with my changes.
  • [ ] I have checked my code and corrected any misspellings.
  • [ ] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

Paddy0523 avatar Aug 12 '22 08:08 Paddy0523