CookiePieWw commented on issue #3174:
URL: https://github.com/apache/datafusion/issues/3174#issuecomment-2380588637

   > I wonder if you could use a length check on the characters too (it 
wouldn't be perfect, but it might catch more)
   
   Thanks for your suggestion! But if we want to precisely match it, we have to 
iter through the prefix of Int64::MAX and then we got:
   ```
   922337203685477580[0-7]
   9223372036854775[0-7]\d{2}
   922337203685477[0-4]\d{3}
   92233720368547[0-6]\d{4}
   ```
   and so on...
   
   Perhaps we can use a function for match strings here instead of direct regex 
here, which seems more flexible for me. If you agree on that, I'll try to 
submit a pr to `arrow-rs` for further discussions :)


-- 
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: github-unsubscr...@datafusion.apache.org

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


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

Reply via email to