[CELEBORN-1156][BUILD] SBT publish support

[cfmcgrady]

What changes were proposed in this pull request?

As title

Why are the changes needed?

As title

Does this PR introduce any user-facing change?

Yes, the user can publish shade clients via SBT

How was this patch tested?

docker run -d -p 8081:8081 sonatype/nexus3

export SONATYPE_SNAPSHOTS_URL=http://192.168.3.46:8081/repository/maven-snapshots/
export SONATYPE_RELEASES_URL=http://192.168.3.46:8081/repository/maven-releases/
export ASF_USERNAME=admin
export ASF_PASSWORD=123456

• Publish the shade client for Spark 3.5:

./build/sbt -Pspark-3.4 celeborn-client-spark-3-shaded/publish

• Publish the shade client for Flink 1.18:

$ ./build/sbt -Pflink-1.18 celeborn-client-flink-1_18-shaded/publish

• Publish the shade client for MapReduce:

$ ./build/sbt -Pmr celeborn-client-mr-shaded/publish

[codecov]

Codecov Report

Attention: 108 lines in your changes are missing coverage. Please review.

Comparison is base (2504b50) 46.56% compared to head (c4fc46b) 46.03%.
Report is 3 commits behind head on main.

Files	Patch %	Lines
...leborn/common/network/sasl/CelebornSaslClient.java	0.00%	44 Missing ⚠️
...eborn/common/network/sasl/SaslClientBootstrap.java	0.00%	33 Missing ⚠️
...apache/celeborn/common/network/sasl/SaslUtils.java	0.00%	10 Missing ⚠️
.../celeborn/common/network/sasl/SaslCredentials.java	0.00%	6 Missing ⚠️
...born/common/network/sasl/SaslTimeoutException.java	0.00%	6 Missing ⚠️
...eleborn/common/network/client/TransportClient.java	0.00%	5 Missing ⚠️
...cala/org/apache/celeborn/common/CelebornConf.scala	77.78%	2 Missing ⚠️
...born/common/network/protocol/TransportMessage.java	0.00%	1 Missing ⚠️
...he/celeborn/common/network/util/TransportConf.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2129      +/-   ##
==========================================
- Coverage   46.56%   46.03%   -0.52%     
==========================================
  Files         175      180       +5     
  Lines       11377    11492     +115     
  Branches     1019     1027       +8     
==========================================
- Hits         5296     5289       -7     
- Misses       5732     5852     +120     
- Partials      349      351       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pan3793]

Is SONATYPE_HOST redundant? I suppose SONATYPE_RELEASES_URL and SONATYPE_SNAPSHOTS_URL are sufficient.

For Apache release, the default URL should be https://repository.apache.org/service/local/staging/deploy/maven2 and https://repository.apache.org/content/repositories/snapshots

[pan3793]

we can eliminate SONATYPE_USERNAME, SONATYPE_PASSWORD and simply pick up ASF_USERNAME, ASF_PASSWORD, as they were used in source/binary release process

[cfmcgrady]

For Apache release, the default URL should be https://repository.apache.org/service/local/staging/deploy/maven2 and https://repository.apache.org/content/repositories/snapshots

do you mean the default values(SONATYPE_SNAPSHOTS_URL/SONATYPE_RELEASES_URL) should be Apache URLs?

[pan3793]

Yea, we did such thing in another apache projects

[cfmcgrady]

we can eliminate SONATYPE_USERNAME, SONATYPE_PASSWORD and simply pick up ASF_USERNAME, ASF_PASSWORD, as they were used in source/binary release process

done

[pan3793]

does the tailing slash matter?

[cfmcgrady]

I've confirmed in a local environment that the presence or absence of a trailing slash doesn't affect the outcome.

[cfmcgrady]

thanks, merging to main(v0.4.0).