Skip to content

Commit

Permalink
Use io.lakefs:sdk coordinates
Browse files Browse the repository at this point in the history
Also revert *Esti* to use a published SDK from Maven central.  Previously it
was temporarily building its own SDK copy.

Fixes #6730.
  • Loading branch information
arielshaqed committed Oct 8, 2023
1 parent 63ee2c6 commit d73a977
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 11 deletions.
10 changes: 1 addition & 9 deletions .github/workflows/esti.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -296,11 +296,6 @@ jobs:
STORAGE_NAMESPACE: s3://test-bucket/data
REPOSITORY: lakefsfs-contract-test

- name: Build current lakeFS SDK
# TODO(ariels): Remove this in favour of a single lakeFS client version
working-directory: clients/java
run: mvn -P'!treeverse-signing' install

- name: Build and test hadoopfs (hadoop2)
working-directory: clients/hadoopfs
run: mvn clean --quiet --batch-mode --update-snapshots -P'!treeverse-signing',contract-tests-hadoop2 verify
Expand Down Expand Up @@ -387,10 +382,7 @@ jobs:
distribution: "adopt-hotspot"
java-version: "8"
cache: "sbt"
- name: Build current lakeFS SDK
# TODO(ariels): Remove this in favour of a single lakeFS client version
working-directory: clients/java
run: mvn -P'!treeverse-signing' install

- name: Build lakeFS HadoopFS
working-directory: clients/hadoopfs
run: mvn -Passembly -DfinalName=client --batch-mode --update-snapshots package -DskipTests
Expand Down
4 changes: 2 additions & 2 deletions clients/hadoopfs/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -277,8 +277,8 @@ To export to S3:
<dependency>
<groupId>io.lakefs</groupId>
<!-- TODO(ariels): Publish an actual sdk-client version and use *that*! -->
<artifactId>sdk-client</artifactId>
<version>0.111.2-RC.0</version>
<artifactId>sdk</artifactId>
<version>0.112.1</version>
</dependency>
<dependency>
<groupId>org.apache.commons</groupId>
Expand Down

0 comments on commit d73a977

Please sign in to comment.