You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@james.apache.org by GitBox <gi...@apache.org> on 2021/04/09 11:24:52 UTC
[GitHub] [james-project] chibenwa opened a new pull request #380: JAMES-3559 POC: Allow to window push notifications
chibenwa opened a new pull request #380:
URL: https://github.com/apache/james-project/pull/380
## Why?
Today a JAMES action might trigger several events being dispatched on the event bus (eg moving 2 messages will end up firing 2 additions and 2 deletion) resulting in likely 4 StateChanges being pushed to the end user.
## How?
Using the reactor library, James can likely delay pushes for a given timewindow and merge the state changes together in order to supply the client with only one state change.
A likely time value might be 2 seconds.
- Pros: avoid event storm and will lower re-synchronization request count (good for perf?)
- Cons: delays of 2s for real time across devices.
We could of course make it configurable via a jmap.properties configuration.
```
# Optional. Omiting this property leads to windowing being disables. James will
# then directly forward all StateChanges to the end user without attempting to
# buffer and aggregating them.
# If specified, James will delay stateChanges for that given amout of time and will attempt
# to aggregate subsequent state changes together before returning them to the client.
# Units: ms, s, min, h, d
push.aggregation.window.duration=2s
```
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org
[GitHub] [james-project] chibenwa edited a comment on pull request #380: JAMES-3559 Allow to window push notifications
Posted by GitBox <gi...@apache.org>.
chibenwa edited a comment on pull request #380:
URL: https://github.com/apache/james-project/pull/380#issuecomment-828468659
> Can you answer my comment? https://issues.apache.org/jira/browse/JAMES-3559?focusedCommentId=17326375&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17326375
Sure. But a long term mitigation strategy would anyway not address my short term needs.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org
[GitHub] [james-project] rouazana commented on pull request #380: JAMES-3559 Allow to window push notifications
Posted by GitBox <gi...@apache.org>.
rouazana commented on pull request #380:
URL: https://github.com/apache/james-project/pull/380#issuecomment-828421982
> I would like to make progress integrating this.
Can you answer my comment? https://issues.apache.org/jira/browse/JAMES-3559?focusedCommentId=17326375&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17326375
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org
[GitHub] [james-project] chibenwa commented on pull request #380: JAMES-3559 Allow to window push notifications
Posted by GitBox <gi...@apache.org>.
chibenwa commented on pull request #380:
URL: https://github.com/apache/james-project/pull/380#issuecomment-828468659
> Can you answer my comment? https://issues.apache.org/jira/browse/JAMES-3559?focusedCommentId=17326375&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17326375
Sure. But a long term mitigation strategy would anyway not address my current needs.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org
[GitHub] [james-project] chibenwa commented on pull request #380: JAMES-3559 Allow to window push notifications
Posted by GitBox <gi...@apache.org>.
chibenwa commented on pull request #380:
URL: https://github.com/apache/james-project/pull/380#issuecomment-828380889
I really want it for my deployments.
I don't mid marking it as experimental.
That would be a shame that I need to fork JMAP implem to get it...
I would like to make progress integrating this.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org