BiteTheDDDDt opened a new pull request, #43360:
URL: https://github.com/apache/doris/pull/43360

   …alized_bytes
   
   ### What problem does this PR solve?
   add LZ4_MAX_INPUT_SIZE check on 
DataTypeString::get_uncompressed_serialized_bytes
   Problem Summary:
   /*! LZ4_compressBound() :
       Provides the maximum size that LZ4 compression may output in a "worst 
case" scenario (input data not compressible)
       This function is primarily useful for memory allocation purposes 
(destination buffer size).
       Macro LZ4_COMPRESSBOUND() is also provided for compilation-time 
evaluation (stack memory allocation for example).
       Note that LZ4_compress_default() compresses faster when dstCapacity is 
>= LZ4_compressBound(srcSize)
           inputSize  : max supported value is LZ4_MAX_INPUT_SIZE
           return : maximum output size in a "worst case" scenario
                 or 0, if input size is incorrect (too large or negative)
   */
   
   - Test <!-- At least one of them must be included. -->
   
       - [ ] Regression test
       - [ ] Unit Test
       - [ ] Manual test (add detailed scripts or steps below)
       - [ ] No need to test or manual test. Explain why:
           - [ ] This is a refactor/code format and no logic has been changed.
           - [ ] Previous test can cover this change.
           - [ ] No colde files have been changed.
           - [ ] Other reason <!-- Add your reason?  -->
   
   - Behavior changed:
   
       - [ ] No.
       - [ ] Yes. <!-- Explain the behavior change -->
   
   - Does this need documentation?
   
       - [ ] No.
       - [ ] Yes. <!-- Add document PR link here. eg: 
https://github.com/apache/doris-website/pull/1214 -->
   
   - Release note
   
       <!-- bugfix, feat, behavior changed need a release note -->
       <!-- Add one line release note for this PR. -->
       None
   
   ### Check List (For Reviewer who merge this PR)
   
   - [ ] Confirm the release note
   - [ ] Confirm test cases
   - [ ] Confirm document
   - [ ] Add branch pick label <!-- Add branch pick label that this PR should 
merge into -->
   
   


-- 
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: commits-unsubscr...@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to