You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@eventmesh.apache.org by GitBox <gi...@apache.org> on 2022/05/25 07:06:42 UTC

[GitHub] [incubator-eventmesh] mroccyen opened a new issue, #890: [Feature] Add file listener for ConfigurationWrapper class by using WatchService

mroccyen opened a new issue, #890:
URL: https://github.com/apache/incubator-eventmesh/issues/890

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/eventmesh/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### Feature Request
   
   
   ![image](https://user-images.githubusercontent.com/22514893/170201176-3811aa8f-6b17-4645-893b-d14839275c45.png)
   
   ConfigurationWrapper internally reads files at regular intervals, regardless of whether the files change or not, and the timer introduces a delay that causes configuration information to not be updated immediately after the change. I think the configuration information can be updated in real time by using WatchService to listen for changes in the configuration file.
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org


[GitHub] [incubator-eventmesh] qqeasonchen commented on issue #890: [Feature] Add file listener for ConfigurationWrapper class

Posted by GitBox <gi...@apache.org>.
qqeasonchen commented on issue #890:
URL: https://github.com/apache/incubator-eventmesh/issues/890#issuecomment-1136922815

   @mroccyen nice~


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org


[GitHub] [incubator-eventmesh] qqeasonchen commented on issue #890: [Feature] Add file listener for ConfigurationWrapper class

Posted by GitBox <gi...@apache.org>.
qqeasonchen commented on issue #890:
URL: https://github.com/apache/incubator-eventmesh/issues/890#issuecomment-1136935208

   good  idea~
   
   On Wed, May 25, 2022 at 3:06 PM mroccyen ***@***.***> wrote:
   
   > Search before asking
   >
   >    - I had searched in the issues
   >    <https://github.com/apache/eventmesh/issues?q=is%3Aissue> and found no
   >    similar issues.
   >
   > Feature Request
   >
   > [image: image]
   > <https://user-images.githubusercontent.com/22514893/170201176-3811aa8f-6b17-4645-893b-d14839275c45.png>
   >
   > ConfigurationWrapper internally reads files at regular intervals,
   > regardless of whether the files change or not, and the timer introduces a
   > delay that causes configuration information to not be updated immediately
   > after the change. I think the configuration information can be updated in
   > real time by using WatchService to listen for changes in the configuration
   > file.
   > Are you willing to submit PR?
   >
   >    - Yes I am willing to submit a PR!
   >
   > —
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/incubator-eventmesh/issues/890>, or unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/AGH3J5I6HFW2VT6MTJPJCATVLXGQ3ANCNFSM5W4B7EUQ>
   > .
   > You are receiving this because you are subscribed to this thread.Message
   > ID: ***@***.***>
   >
   


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org


[GitHub] [incubator-eventmesh] qqeasonchen closed issue #890: [Feature] Add file listener for ConfigurationWrapper class

Posted by GitBox <gi...@apache.org>.
qqeasonchen closed issue #890: [Feature] Add file listener for ConfigurationWrapper class
URL: https://github.com/apache/incubator-eventmesh/issues/890


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: dev-help@eventmesh.apache.org