abhiramaab opened a new pull request, #8344:
URL: https://github.com/apache/hop/pull/8344

   Fixes #8327
   
   ### Problem
   
   In Hop Web (running via Eclipse RAP in the browser), background UI updates 
triggered via `Display.asyncExec(...)` (such as log lines from 
`HopGuiLogBrowser`'s 1-second refresh timer) are buffered on the server and 
never delivered to the client browser unless an active `ServerPushSession` is 
running.
   
   Previously:
   
   - In `HopGuiPipelineGraph` and `HopGuiWorkflowGraph`, 
`ServerPushSessionFacade.stop()` was invoked prematurely immediately after 
kicking off the execution delegate, terminating server push while execution was 
actively running in the background.
   - If background notifications were disabled, no server push session was 
running at all during the web session.
   - As a result, the logging tab in the Results panel stalled and did not 
refresh until the user clicked, typed, or resized the browser window to trigger 
a client request.
   
   ### Solution
   
   1. In `HopWebEntryPoint`, invoke `ServerPushSessionFacade.start()` upon 
session startup so server push remains active throughout the Hop Web user 
session. Clean shutdown is already handled during session destruction 
(`beforeDestroy`).
   2. Remove premature `ServerPushSessionFacade.stop()` calls from 
`HopGuiPipelineGraph` and `HopGuiWorkflowGraph`.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to