surahman commented on pull request #3741:
URL: https://github.com/apache/incubator-heron/pull/3741#issuecomment-985788153


   > I still think we can default to smaller value with low risk, but agree the 
risk is not 0%. After this is merged, I'll do some analysis on workloads at 
work to see what could be a good default. @windhamwong provided numbers also 
give me confidence because they match what I've observed. But I'll do a more 
rigorous process to capture numbers.
   
   I agree. Once we are sure of the changes which need to be made we can go 
ahead and figure out how to affect this change in a future PR.
   
   I have one last set of typos corrections I am making to the documentation 
that I will merge in later today.


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