You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by "Brett Konold (Jira)" <ji...@apache.org> on 2020/08/06 23:09:00 UTC

[jira] [Created] (SAMZA-2576) Consolidate container.* config references into their own class

Brett Konold created SAMZA-2576:
-----------------------------------

             Summary: Consolidate container.* config references into their own class
                 Key: SAMZA-2576
                 URL: https://issues.apache.org/jira/browse/SAMZA-2576
             Project: Samza
          Issue Type: Improvement
            Reporter: Brett Konold


Many configs that match `container.*` are hardcoded in various places in samza. these would be better suited to be consolidated into a new or existing config accessor class.

Some examples:

[https://github.com/apache/samza/blob/master/samza-kv-rocksdb/src/main/java/org/apache/samza/storage/kv/descriptors/RocksDbTableDescriptor.java#L43]

[https://github.com/apache/samza/blob/master/samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala#L60]

[https://github.com/apache/samza/blob/master/samza-core/src/main/scala/org/apache/samza/container/SamzaContainer.scala#L630]

among others.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)