You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/07/11 14:04:16 UTC

[GitHub] [beam] dgdheeraj commented on pull request #22167: Allow BigQuery TableIds to have space in between

dgdheeraj commented on PR #22167:
URL: https://github.com/apache/beam/pull/22167#issuecomment-1180450298

   > Thanks, this looks good.
   > 
   > Technically \s is broader than Zs, but I don't see a good way to make that distinction with Python's re module, and false positives here are preferable to false negatives.
   
   Thanks for the review!
   Yeah, I couldn't find up with a better way to  include Zs apart from \s 


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

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