From jvanek at openjdk.org Thu Nov 14 16:49:03 2024 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Thu, 14 Nov 2024 16:49:03 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable Message-ID: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. From: `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 `COMBINATIONS_JSON` same as combinations, but a valid json `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed * TESTS are already implemented as ALL_MATCHING * COMBINATIONS as ALL_MATCHING_COMBINATIONS * ALL_TESTS as ALL COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. The _JSON variants are tobe decided, but ability to JQL it sounds nice. Suggestion to better names welcomed. ------------- Commit messages: - Refactored lsiting so it takes proper argument - refactored listTests() to its own class Changes: https://git.openjdk.org/jcstress/pull/153/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903774 Stats: 132 lines in 4 files changed: 101 ins; 21 del; 10 mod Patch: https://git.openjdk.org/jcstress/pull/153.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/153/head:pull/153 PR: https://git.openjdk.org/jcstress/pull/153 From jvanek at openjdk.org Fri Nov 15 17:50:16 2024 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Fri, 15 Nov 2024 17:50:16 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable [v2] In-Reply-To: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: <8faRANyM3pLjni3R3IWcFk9dNw0YauRtKvKzYLW8Wws=.ff7e9cd3-0655-497f-8c20-eeaed58346ec@github.com> > This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. > > It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. > > From: > > `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact > `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 > `COMBINATIONS_JSON` same as combinations, but a valid json > `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed > > > * TESTS are already implemented as ALL_MATCHING > * COMBINATIONS as ALL_MATCHING_COMBINATIONS > * ALL_TESTS as ALL > > COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. > The _JSON variants are tobe decided, but ability to JQL it sounds nice. > > Suggestion to better names welcomed. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: implemented MATCHING_(INVERTED)GROUPS(_COUNT) ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/153/files - new: https://git.openjdk.org/jcstress/pull/153/files/d8a3644c..0a2e6c7a Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=01 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=00-01 Stats: 90 lines in 2 files changed: 76 ins; 1 del; 13 mod Patch: https://git.openjdk.org/jcstress/pull/153.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/153/head:pull/153 PR: https://git.openjdk.org/jcstress/pull/153 From jvanek at openjdk.org Fri Nov 15 17:50:16 2024 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Fri, 15 Nov 2024 17:50:16 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable In-Reply-To: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: On Thu, 14 Nov 2024 16:45:30 GMT, Ji?? Van?k wrote: > This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. > > It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. > > From: > > `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact > `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 > `COMBINATIONS_JSON` same as combinations, but a valid json > `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed > > > * TESTS are already implemented as ALL_MATCHING > * COMBINATIONS as ALL_MATCHING_COMBINATIONS > * ALL_TESTS as ALL > > COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. > The _JSON variants are tobe decided, but ability to JQL it sounds nice. > > Suggestion to better names welcomed. Implemented COMBINATIONS_GROUPED and COMBINATIONS_GROUPS (as MATCHING_GROUPS and MATCHING_GROUPS_COUNT) also with its inverted variants The nice json will maybe need some refactorings. WIP on it, but any feedback appreciated ------------- PR Comment: https://git.openjdk.org/jcstress/pull/153#issuecomment-2479589599 From jvanek at openjdk.org Fri Nov 15 17:53:56 2024 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Fri, 15 Nov 2024 17:53:56 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable [v3] In-Reply-To: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: > This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. > > It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. > > From: > > `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact > `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 > `COMBINATIONS_JSON` same as combinations, but a valid json > `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed > > > * TESTS are already implemented as ALL_MATCHING > * COMBINATIONS as ALL_MATCHING_COMBINATIONS > * ALL_TESTS as ALL > > COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. > The _JSON variants are tobe decided, but ability to JQL it sounds nice. > > Suggestion to better names welcomed. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Removed redundnant hyphen ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/153/files - new: https://git.openjdk.org/jcstress/pull/153/files/0a2e6c7a..a58cd140 Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=02 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/153.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/153/head:pull/153 PR: https://git.openjdk.org/jcstress/pull/153 From jvanek at openjdk.org Fri Nov 15 18:45:24 2024 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Fri, 15 Nov 2024 18:45:24 GMT Subject: RFR: 7903774: make all tests combinations printing nicer and/or configurable [v4] In-Reply-To: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> References: <3wZeI91CX_iFefOZJdSxdmZFSyzjd8-w1PJXYufNxrM=.77abf5c8-c5ca-45b9-9b8b-e5c0a7e9b478@github.com> Message-ID: > This is still WIP, but already on row what [CODETOOLS-7903774](https://bugs.openjdk.org/browse/CODETOOLS-7903774) suggested. > > It will be finished once all desired output combinations are included, but already now is doing the usage of listing much more comfortable and code readable. > > From: > > `TESTS` - as introduced by https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS` - as introduced in https://github.com/openjdk/jcstress/pull/149 verbose mode (thus surpassing the `-v` impact > `ALL_TESTS` - all tests as it was before https://github.com/openjdk/jcstress/pull/149 > `COMBINATIONS_GROUPED ` - as suggested in https://github.com/openjdk/jcstress/pull/149#discussion_r1668678910 > `COMBINATIONS_JSON` same as combinations, but a valid json > `COMBINATIONS_GROUPS` same as COMBINATIONS_GROUPED only instead of enumeration in each group, only count of members will be printed > > > * TESTS are already implemented as ALL_MATCHING > * COMBINATIONS as ALL_MATCHING_COMBINATIONS > * ALL_TESTS as ALL > > COMBINATIONS_GROUPED, and COMBINATIONS_GROUPS are under development. > The _JSON variants are tobe decided, but ability to JQL it sounds nice. > > Suggestion to better names welcomed. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Fixed spacing in help ------------- Changes: - all: https://git.openjdk.org/jcstress/pull/153/files - new: https://git.openjdk.org/jcstress/pull/153/files/a58cd140..67b737ba Webrevs: - full: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=03 - incr: https://webrevs.openjdk.org/?repo=jcstress&pr=153&range=02-03 Stats: 5 lines in 1 file changed: 0 ins; 0 del; 5 mod Patch: https://git.openjdk.org/jcstress/pull/153.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/153/head:pull/153 PR: https://git.openjdk.org/jcstress/pull/153 From shade at openjdk.org Fri Nov 29 19:34:28 2024 From: shade at openjdk.org (Aleksey Shipilev) Date: Fri, 29 Nov 2024 19:34:28 GMT Subject: RFR: 7903891: jcstress: Switch to latest runner versions Message-ID: We always playing the catchup game with runners. Let's just default to latest runners for every OS flavor. ------------- Commit messages: - Fix Changes: https://git.openjdk.org/jcstress/pull/154/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=154&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903891 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jcstress/pull/154.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/154/head:pull/154 PR: https://git.openjdk.org/jcstress/pull/154 From duke at openjdk.org Fri Nov 29 19:43:42 2024 From: duke at openjdk.org (duke) Date: Fri, 29 Nov 2024 19:43:42 GMT Subject: git: openjdk/jcstress: 7903891: jcstress: Switch to latest runner versions Message-ID: <941ff3a5-6f97-4975-94e6-5f1e051f1f9f@openjdk.org> Changeset: 2b49c400 Branch: master Author: Aleksey Shipil?v Committer: GitHub Date: 2024-11-29 20:43:02 +0000 URL: https://git.openjdk.org/jcstress/commit/2b49c40055bf1fce0c631329ec71bf4c53d0b7c6 7903891: jcstress: Switch to latest runner versions ! .github/workflows/pre-integration.yml From shade at openjdk.org Fri Nov 29 19:45:51 2024 From: shade at openjdk.org (Aleksey Shipilev) Date: Fri, 29 Nov 2024 19:45:51 GMT Subject: Withdrawn: 7903891: jcstress: Switch to latest runner versions In-Reply-To: References: Message-ID: On Fri, 29 Nov 2024 19:29:26 GMT, Aleksey Shipilev wrote: > We always playing the catchup game with runners. Let's just default to latest runners for every OS flavor. This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jcstress/pull/154 From shade at openjdk.org Fri Nov 29 19:49:00 2024 From: shade at openjdk.org (Aleksey Shipilev) Date: Fri, 29 Nov 2024 19:49:00 GMT Subject: RFR: 7903890: jcstress: Rewrite/amend Singleton and Lazy primitives samples Message-ID: It is obvious we do not have a good narrative around these, so let's add some. ------------- Commit messages: - Lazy + Singleton examples Changes: https://git.openjdk.org/jcstress/pull/155/files Webrev: https://webrevs.openjdk.org/?repo=jcstress&pr=155&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903890 Stats: 3045 lines in 37 files changed: 2243 ins; 802 del; 0 mod Patch: https://git.openjdk.org/jcstress/pull/155.diff Fetch: git fetch https://git.openjdk.org/jcstress.git pull/155/head:pull/155 PR: https://git.openjdk.org/jcstress/pull/155