You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2022/02/07 10:43:47 UTC
[GitHub] [incubator-doris] yangzhg removed a comment on pull request #7963: [fix] fix ltrim result may incorrect in some case
yangzhg removed a comment on pull request #7963:
URL: https://github.com/apache/incubator-doris/pull/7963#issuecomment-1031321816
> Similar fixes has been done in #7712 , which was caught by current unit tests. It's an indication that test coverage should be improved.
Fix in #7712 will add an offset of 128 when mask is 0, but offset is 32 when mask is 0 in this pr, I not sure 128 is a correct value @amosbird ?
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org