Skip to content

Commit be3cca6

Browse files
authored
Separate out Lucene tests so they run separately in PRB (#3440)
I was looking through recent PRB runs, and it seemed like the time to run other-tests was now around twice the time to run the core-tests. Most of the time seemed to be coming from lucene-tests, so I wanted to experiment with separating those out. This hopefully should decrease our total PRB time
1 parent c323aa8 commit be3cca6

File tree

1 file changed

+36
-3
lines changed

1 file changed

+36
-3
lines changed

.github/workflows/pull_request.yml

Lines changed: 36 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,40 @@ jobs:
6464
include-hidden-files: true
6565
retention-days: 1
6666

67+
lucene-tests:
68+
runs-on: ubuntu-latest
69+
permissions:
70+
checks: write
71+
contents: read
72+
pull-requests: write
73+
steps:
74+
- name: Checkout sources
75+
uses: actions/[email protected]
76+
- name: Setup Base Environment
77+
uses: ./actions/setup-base-env
78+
- name: Setup FDB
79+
uses: ./actions/setup-fdb
80+
- name: Run Gradle Test
81+
uses: ./actions/gradle-test
82+
with:
83+
gradle_command: :fdb-record-layer-lucene:test :fdb-record-layer-lucene:destructiveTest
84+
gradle_args: -PreleaseBuild=false -PpublishBuild=false
85+
- name: Publish Test Reports
86+
if: always()
87+
uses: actions/[email protected]
88+
with:
89+
name: lucene-test-reports
90+
path: |
91+
test-reports/fdb-record-layer-lucene/
92+
- name: Publish Coverage Data
93+
uses: actions/[email protected]
94+
with:
95+
name: lucene-coverage-data
96+
path: |
97+
**/.out/jacoco/*.exec
98+
include-hidden-files: true
99+
retention-days: 1
100+
67101
other-tests:
68102
runs-on: ubuntu-latest
69103
permissions:
@@ -80,7 +114,7 @@ jobs:
80114
- name: Run Gradle Test
81115
uses: ./actions/gradle-test
82116
with:
83-
gradle_command: test -x :fdb-record-layer-core:test destructiveTest -x :fdb-record-layer-core:destructiveTest
117+
gradle_command: test -x :fdb-record-layer-core:test -x :fdb-record-layer-lucene:test destructiveTest -x :fdb-record-layer-core:destructiveTest -x :fdb-record-layer-lucene:destructiveTest
84118
gradle_args: -PreleaseBuild=false -PpublishBuild=false
85119
- name: Publish Test Reports
86120
if: always()
@@ -92,7 +126,6 @@ jobs:
92126
test-reports/fdb-extensions/
93127
test-reports/fdb-record-layer-icu/
94128
test-reports/fdb-record-layer-spatial/
95-
test-reports/fdb-record-layer-lucene/
96129
test-reports/fdb-record-layer-jmh/
97130
test-reports/examples/
98131
test-reports/fdb-relational-api/
@@ -113,7 +146,7 @@ jobs:
113146
retention-days: 1
114147

115148
coverage:
116-
needs: [core-tests, other-tests]
149+
needs: [core-tests, lucene-tests, other-tests]
117150
runs-on: ubuntu-latest
118151
permissions:
119152
checks: read

0 commit comments

Comments
 (0)