roee88 commented on issue #345:
URL: https://github.com/apache/arrow-rs/issues/345#issuecomment-849055081


   I think that the issue description here is wrong and this is expected for 
test_time32second_invalid_neg.
   
   I did a quick memory usage check and identifier the following:
   1. bit_length_test_string  consumes on its own ~8g accumulated to a peak of 
~10g that are not released after the test is done
   2. bit_length_test_large_string has similar memory consumption but the 
memory is freed after the test is done
   3. match_single_group  consumes 2.7g accumulated to a peak of ~4.5G that are 
not released after the test is done
   
   I tested ignoring bit_length_test_string  and github actions checks pass (5 
re-runs). I think it might be worth to ignore all three though. Some questions:
   1. Is it possible that there is a memory leak in bit_length_test_string and 
match_single_group?
   2. Should we disable bit_length_test_large_string  just because it consumes 
a lot of RAM while running?
   3. Should a support ticket be opened in Miri?
   
   I will submit a PR based on the answers. 


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


Reply via email to