neilconway opened a new issue, #26388:
URL: https://github.com/apache/datafusion/issues/26388

   ### Describe the bug
   
   We should add `try_append_with` and `try_append_byte_map` to 
`StringViewArrayBuilder`. Also worth considering why we need `try_` and 
panicking versions of so many methods.
   
   ### To Reproduce
   
   In `datafusion-cli`, built at commit b5bb8f234 (#26377's head):
   
   ```sql
   CREATE TABLE t AS VALUES (repeat('a', 540000000));
   SELECT length(translate(column1, 'a', '😀')) FROM t;
   ```
   
   ```
   thread 'main' panicked at datafusion/functions/src/strings.rs:545:52:
   byte array offset overflow
   ```
   
   
   ### Expected behavior
   
   Should return an error instead.
   
   ### Additional context
   
   Follow-up to #26377 (see review discussion there).
   
   
   ### Willingness to contribute
   
   None


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