You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2022/07/04 18:07:54 UTC

[GitHub] [tinkerpop] xiazcy commented on pull request #1745: gremlin-go: fix data race on localizer initialization

xiazcy commented on PR #1745:
URL: https://github.com/apache/tinkerpop/pull/1745#issuecomment-1174045502

   I suppose it's similar to the serializer/deserializer change, which also makes sense. Though I'm not sure if we want to keep lazy initialization here? I.e. using `sync.Once` instead? Asking because I'm not sure if loading the language configs have overhead and/or implications when more languages are added down the road, and pinging Simon who may know more about this localizer @simonz-bq 


-- 
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: commits-unsubscribe@tinkerpop.apache.org

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