CrownChu commented on code in PR #10777:
URL: https://github.com/apache/paimon/pull/10777#discussion_r4376928838
##########
paimon-elasticsearch/pom.xml:
##########
@@ -28,8 +28,8 @@ under the License.
<version>2.2-SNAPSHOT</version>
</parent>
- <artifactId>paimon-eslib</artifactId>
- <name>Paimon : ESLib Index</name>
+ <artifactId>paimon-elasticsearch</artifactId>
Review Comment:
Addressed in aa86d32.
The implementation remains in `paimon-elasticsearch`. A deprecated
`paimon-eslib`
relocation POM under `paimon-elasticsearch/compat` redirects the old Maven
coordinate to
the new artifact at the same group/version. The old `-Ppaimon-eslib`
profile is also
retained as an alias, and the Java ES CI lane includes the compatibility
module.
Validated the new profile, the old profile, and both profiles together. A
separate
consumer using the old coordinate compiled and ran against the new JAR and
its transitive
Lucene dependency in a fresh local Maven repository.
The guide also clarifies that direct JAR-download scripts and
source-directory selectors
must use the new name; Maven relocation only covers dependency resolution.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
