On Mon, 4 May 2020 20:43:42 +0300 "Yordan Karadzhov (VMware)" <y.karadz@xxxxxxxxx> wrote: > So far the button was showing up only when we do a single-threaded > search in the data. Stopping the multi-threaded searches became > possible after the modification of the parallel search introduced > in the previous commit. When the multi-threaded search is restarted > (after being stopped) the label showing the number of matches found > is reset to show nothing. It will show the count again when the > search finishes or is stopped. > I'm pulling in this series. But more of an FYI, it's best to avoid statements like "previous commit" because when developing with git, it may not be a previous commit, when it goes into a tree. Especially if a commit gets cherry-picked. Instead, use phrases like "With the parallel search changes that were made, it is now possible to stop the multi-threaded searches and still be at the correct percentage of the search that is displayed to the user". It is frowned upon to mention "this commit" or "last commit" in change logs. I catch myself doing it once in a while, and try to conform as well ;-) -- Steve
|