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

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/12/09 17:43:37 UTC

[GitHub] [druid] gianm commented on pull request #10659: IdUtils: Forbid characters that cannot be used in znodes.

gianm commented on pull request #10659:
URL: https://github.com/apache/druid/pull/10659#issuecomment-741938381


   > Curious but did you run across any related curator exceptions which prompted this fix?
   
   Not in production, but I was testing with a bunch of crazy characters to see what would happen and ran into this. #10645 and this patch both came from that effort. In a future patch I'd like to add all these crazy characters to the integration tests, but I didn't do it immediately, because I ran into problems with getting Maven to pass them along properly.


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

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org