On Fri, 26 Nov 2021 13:10:08 GMT, yosbits <d...@openjdk.java.net> wrote:

>> Abhinay Agarwal has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Update ROW_COUNT to 700_000
>
> tests/manual/controls/SelectTableViewTest.java line 19:
> 
>> 17:     //  final int ROW_COUNT = 80_000;
>> 18:     //  final int ROW_COUNT = 50_000;
>> 19:     //  final int ROW_COUNT = 8_000;
> 
> The number is meaningful because it is the number of data I used to show the 
> improvement effect in the original PR.
> 
> Reference value of the number of data that can be handled within 3 seconds of 
> processing time (before-> after)
> 
> TableView
> 
> selectAll: 8_000-> 700_000
> selectRange: 7_000-> 50_000
> 
> 
> 
> 
> 
> ``` Java
> public class SelectTableViewTest extends Application {
> 
>       final int ROW_COUNT = 700_000;
> //    final int ROW_COUNT = 80_000;
> //    final int ROW_COUNT = 50_000;
> //    final int ROW_COUNT = 8_000;
>       final int COL_COUNT = 3;

I reduced ROW_COUNT from 700_000 to 70_000 as the tests were taking a few 
seconds to run on my machine. I have reverted these now. Nevertheless, time 
taken to run a test have a number of variables. Depending on the machine the 
tests are run, it may not necessarily always take 3 seconds :)

-------------

PR: https://git.openjdk.java.net/jfx/pull/673

Reply via email to