guan404ming commented on PR #54445:
URL: https://github.com/apache/airflow/pull/54445#issuecomment-3191192998

   > Is this actually streaming and rendering logs as they stream in or just 
reading the first stream? I suspect it is the latter but I am a bit new to this 
format.
   
   Yes, this is not real streaming. I misunderstood and also confirmed more 
details with @jason810496. After offline discussion and test, we confirmed that 
the streaming implementation still has some issues that need to be addressed. 
But switching from `application/json` to `application/x-ndjson` would still 
help reduce the load on the api-server.  So I’ll update this PR for only 
changing the header for now, and will implement actual streaming once the 
issues are resolved.


-- 
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