Skip to content

Commit fbd35ed

Browse files
committed
Commented out the hadoop-standard build
It's running too slow in GHA for some reason, but is not doing so locally. Retained the hadoop feature tests since those seem fine CTR
1 parent 748ef64 commit fbd35ed

File tree

1 file changed

+19
-16
lines changed

1 file changed

+19
-16
lines changed

.github/workflows/build-test.yml

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -117,22 +117,25 @@ jobs:
117117
# name: ${{ github.sha }}
118118
# path: ./gremlin-server/gremlin-server.tar
119119
# retention-days: 1
120-
hadoop-standard:
121-
name: hadoop standard
122-
timeout-minutes: 45
123-
needs: smoke
124-
runs-on: ubuntu-latest
125-
steps:
126-
- uses: actions/checkout@v6
127-
- name: Set up JDK 11
128-
uses: actions/setup-java@v5
129-
with:
130-
java-version: '11'
131-
distribution: 'temurin'
132-
- name: Build with Maven
133-
run: |
134-
mvn clean install -pl $EXCLUDE_MODULES -q -DskipTests -Dci
135-
mvn verify -pl :hadoop-gremlin -DskipTests -DskipIntegrationTests=false '-Dit.test=HadoopGraphProcessStandardIntegrateTest'
120+
# the hadoop-standard build runs really slowly in GHA for some reason. the command below runs in 10 minutes
121+
# locally so not sure what's the issue is. at least hadoop-feature seems to happily run, so perhaps that is
122+
# enough for now
123+
# hadoop-standard:
124+
# name: hadoop standard
125+
# timeout-minutes: 45
126+
# needs: smoke
127+
# runs-on: ubuntu-latest
128+
# steps:
129+
# - uses: actions/checkout@v6
130+
# - name: Set up JDK 11
131+
# uses: actions/setup-java@v5
132+
# with:
133+
# java-version: '11'
134+
# distribution: 'temurin'
135+
# - name: Build with Maven
136+
# run: |
137+
# mvn clean install -pl $EXCLUDE_MODULES -q -DskipTests -Dci
138+
# mvn verify -pl :hadoop-gremlin -DskipTests -DskipIntegrationTests=false '-Dit.test=HadoopGraphProcessStandardIntegrateTest'
136139
hadoop-feature:
137140
name: hadoop feature
138141
timeout-minutes: 45

0 commit comments

Comments
 (0)