You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "razeghi71 (via GitHub)" <gi...@apache.org> on 2023/12/07 23:31:33 UTC

[PR] Fix: Ensure Timestamp Parsing Rejects Characters After 'Z [arrow-rs]

razeghi71 opened a new pull request, #5189:
URL: https://github.com/apache/arrow-rs/pull/5189

   # Which issue does this PR close?
   
   Closes #5182
   
   Explained in the issue.
   
   # What changes are included in this PR?
   
   I have changed the `string_to_datetime` to make sure that the 'Z is always the last charachter.
   
   # Are there any user-facing changes?
   
   No


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] Fix: Ensure Timestamp Parsing Rejects Characters After 'Z [arrow-rs]

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold commented on PR #5189:
URL: https://github.com/apache/arrow-rs/pull/5189#issuecomment-1846268350

   Looks good to me, thank you


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] Fix: Ensure Timestamp Parsing Rejects Characters After 'Z [arrow-rs]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #5189:
URL: https://github.com/apache/arrow-rs/pull/5189#issuecomment-1847065069

   Thank you @razeghi71 


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] Fix: Ensure Timestamp Parsing Rejects Characters After 'Z [arrow-rs]

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold merged PR #5189:
URL: https://github.com/apache/arrow-rs/pull/5189


-- 
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-unsubscribe@arrow.apache.org

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