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