You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@inlong.apache.org by "gosonzhang (via GitHub)" <gi...@apache.org> on 2023/08/16 07:49:27 UTC

[GitHub] [inlong] gosonzhang opened a new issue, #8741: [Bug][DataProxy] Wrong constant reference in CommonConfigHolder class

gosonzhang opened a new issue, #8741:
URL: https://github.com/apache/inlong/issues/8741

   ### What happened
   
   In the CommonConfigHolder class, VAL_MIN_FILE_METRIC_MAX_CACHE_CNT and VAL_MIN_FILE_METRIC_STAT_INVL_SEC are placed in the wrong position. Although their values are equal, they belong to different definitions and need to be corrected
   
   
   ![image](https://github.com/apache/inlong/assets/14038849/441b663a-76e2-4669-88fa-883a568d0cb0)
   https://github.com/apache/inlong/blob/master/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/config/CommonConfigHolder.java#L466C29-L466C62
   
   ### What you expected to happen
   
   Different variables are defined using corresponding constants
   
   ### How to reproduce
   
   review codes
   
   ### Environment
   
   Linux
   
   ### InLong version
   
   master
   
   ### InLong Component
   
   InLong DataProxy
   
   ### Are you willing to submit PR?
   
   - [X] Yes, I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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: dev-unsubscribe@inlong.apache.org.apache.org

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

[GitHub] [inlong] gosonzhang closed issue #8741: [Bug][DataProxy] Wrong constant reference in CommonConfigHolder class

Posted by "gosonzhang (via GitHub)" <gi...@apache.org>.
gosonzhang closed issue #8741: [Bug][DataProxy] Wrong constant reference in CommonConfigHolder class
URL: https://github.com/apache/inlong/issues/8741


-- 
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: dev-unsubscribe@inlong.apache.org

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