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

Posted to commits@samza.apache.org by "shanthoosh (via GitHub)" <gi...@apache.org> on 2023/05/24 16:44:26 UTC

[GitHub] [samza] shanthoosh commented on pull request #1666: SAMZA-2780: Implement ZkSystemEnvironmentSetter

shanthoosh commented on PR #1666:
URL: https://github.com/apache/samza/pull/1666#issuecomment-1561571432

   1. Problem with using System properties is that it is shared by all libraries running within a jvm. Any library can either overwrite the changes made by the Samza engine or can be affected by the changes. So, let us prefix all the configurations with samza scope for config isolation. 
   
   2. Let us also log all the properties that are used by samza processor during startup, so that it will be helpful for debugging purposes. 


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

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