erikbocks commented on PR #13210:
URL: https://github.com/apache/cloudstack/pull/13210#issuecomment-4566356785

   @DaanHoogland @winterhazel 
   
   I validated that QemuImg's behavior using longs was kept by taking a 
snapshot with the timeout configured as 2147483 (the closest number to Java's 
integer overflow). During the tests, I tried to set my snapshot timeout to 
2200000 (to cause an Integer overflow), and observed that the command failed to 
be sent because they use integers internally. Not related to the PR, but 
pointing it out.


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