illegal JDOM comment error when running mvn release:prepare

[unidevel]

Your Environment

• Presto version used: 0.291
• Storage (HDFS/S3/GCS..):
• Data source and connector used:
• Deployment (Cloud or On-prem):
• Pastebin link to the complete debug logs:

[INFO] Transforming 'presto-kafka'...
[INFO] Transforming 'presto-redis'...
[INFO] Transforming 'presto-accumulo'...
[INFO] Transforming 'presto-cassandra'...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] presto-root 0.291 .................................. FAILURE [  0.326 s]
[INFO] presto-testng-services 0.291 ....................... SKIPPED
[INFO] presto-common 0.291 ................................ SKIPPED
[INFO] presto-spi 0.291 ................................... SKIPPED
[INFO] presto-plugin-toolkit 0.291 ........................ SKIPPED
[INFO] presto-client 0.291 ................................ SKIPPED
[INFO] presto-parser 0.291 ................................ SKIPPED
[INFO] presto-geospatial-toolkit 0.291 .................... SKIPPED
[INFO] presto-analyzer 0.291 .............................. SKIPPED
[INFO] presto-bytecode 0.291 .............................. SKIPPED
[INFO] presto-matching 0.291 .............................. SKIPPED
[INFO] presto-memory-context 0.291 ........................ SKIPPED
[INFO] presto-expressions 0.291 ........................... SKIPPED
[INFO] presto-tpch 0.291 .................................. SKIPPED
[INFO] presto-thrift-api 0.291 ............................ SKIPPED
[INFO] presto-function-namespace-managers 0.291 ........... SKIPPED
[INFO] presto-ui 0.291 .................................... SKIPPED
[INFO] presto-main 0.291 .................................. SKIPPED
[INFO] presto-blackhole 0.291 ............................. SKIPPED
[INFO] presto-resource-group-managers 0.291 ............... SKIPPED
[INFO] presto-thrift-testing-udf-server 0.291 ............. SKIPPED
[INFO] presto-tests 0.291 ................................. SKIPPED
[INFO] presto-atop 0.291 .................................. SKIPPED
[INFO] presto-hdfs-core 0.291 ............................. SKIPPED
[INFO] presto-cache 0.291 ................................. SKIPPED
[INFO] presto-jmx 0.291 ................................... SKIPPED
[INFO] presto-record-decoder 0.291 ........................ SKIPPED
[INFO] presto-kafka 0.291 ................................. SKIPPED
[INFO] presto-redis 0.291 ................................. SKIPPED
[INFO] presto-accumulo 0.291 .............................. SKIPPED
[INFO] presto-cassandra 0.291-SNAPSHOT .................... SKIPPED
[INFO] presto-bigquery 0.291-SNAPSHOT ..................... SKIPPED
[INFO] presto-memory 0.291-SNAPSHOT ....................... SKIPPED
[INFO] presto-orc 0.291-SNAPSHOT .......................... SKIPPED
[INFO] presto-benchmark 0.291-SNAPSHOT .................... SKIPPED
[INFO] presto-parquet 0.291-SNAPSHOT ...................... SKIPPED
[INFO] presto-rcfile 0.291-SNAPSHOT ....................... SKIPPED
[INFO] presto-hive-common 0.291-SNAPSHOT .................. SKIPPED
[INFO] presto-hive-metastore 0.291-SNAPSHOT ............... SKIPPED
[INFO] presto-tpcds 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-testing-docker 0.291-SNAPSHOT ............... SKIPPED
[INFO] presto-hive 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-hive-hadoop2 0.291-SNAPSHOT ................. SKIPPED
[INFO] presto-iceberg 0.291-SNAPSHOT ...................... SKIPPED
[INFO] presto-i18n-functions 0.291-SNAPSHOT ............... SKIPPED
[INFO] presto-teradata-functions 0.291-SNAPSHOT ........... SKIPPED
[INFO] presto-example-http 0.291-SNAPSHOT ................. SKIPPED
[INFO] presto-local-file 0.291-SNAPSHOT ................... SKIPPED
[INFO] presto-base-jdbc 0.291-SNAPSHOT .................... SKIPPED
[INFO] presto-mysql 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-oracle 0.291-SNAPSHOT ....................... SKIPPED
[INFO] presto-postgresql 0.291-SNAPSHOT ................... SKIPPED
[INFO] presto-prometheus 0.291-SNAPSHOT ................... SKIPPED
[INFO] presto-redshift 0.291-SNAPSHOT ..................... SKIPPED
[INFO] presto-sqlserver 0.291-SNAPSHOT .................... SKIPPED
[INFO] presto-mongodb 0.291-SNAPSHOT ...................... SKIPPED
[INFO] presto-ml 0.291-SNAPSHOT ........................... SKIPPED
[INFO] presto-jdbc 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-cli 0.291-SNAPSHOT .......................... SKIPPED
[INFO] presto-product-tests 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-pinot-toolkit 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-pinot 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-benchmark-driver 0.291-SNAPSHOT ............. SKIPPED
[INFO] presto-thrift-testing-server 0.291-SNAPSHOT ........ SKIPPED
[INFO] presto-thrift-connector 0.291-SNAPSHOT ............. SKIPPED
[INFO] presto-password-authenticators 0.291-SNAPSHOT ...... SKIPPED
[INFO] presto-hudi 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-hive-function-namespace 0.291-SNAPSHOT ...... SKIPPED
[INFO] presto-hana 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-druid 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-cluster-ttl-providers 0.291-SNAPSHOT ........ SKIPPED
[INFO] presto-singlestore 0.291-SNAPSHOT .................. SKIPPED
[INFO] presto-node-ttl-fetchers 0.291-SNAPSHOT ............ SKIPPED
[INFO] presto-session-property-managers 0.291-SNAPSHOT .... SKIPPED
[INFO] presto-kudu 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-elasticsearch 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-delta 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-clickhouse 0.291-SNAPSHOT ................... SKIPPED
[INFO] presto-server 0.291-SNAPSHOT ....................... SKIPPED
[INFO] presto-docs 0.291-SNAPSHOT ......................... SKIPPED
[INFO] presto-spark-common 0.291-SNAPSHOT ................. SKIPPED
[INFO] presto-verifier 0.291-SNAPSHOT ..................... SKIPPED
[INFO] presto-testing-server-launcher 0.291-SNAPSHOT ...... SKIPPED
[INFO] presto-benchto-benchmarks 0.291-SNAPSHOT ........... SKIPPED
[INFO] presto-proxy 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-benchmark-runner 0.291-SNAPSHOT ............. SKIPPED
[INFO] presto-spark-classloader-interface 0.291-SNAPSHOT .. SKIPPED
[INFO] presto-spark-base 0.291-SNAPSHOT ................... SKIPPED
[INFO] presto-spark 0.291-SNAPSHOT ........................ SKIPPED
[INFO] presto-spark-package 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-spark-launcher 0.291-SNAPSHOT ............... SKIPPED
[INFO] presto-spark-testing 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-thrift-spec 0.291-SNAPSHOT .................. SKIPPED
[INFO] presto-google-sheets 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-grpc-api 0.291-SNAPSHOT ..................... SKIPPED
[INFO] presto-grpc-testing-udf-server 0.291-SNAPSHOT ...... SKIPPED
[INFO] presto-lark-sheets 0.291-SNAPSHOT .................. SKIPPED
[INFO] presto-test-coverage 0.291-SNAPSHOT ................ SKIPPED
[INFO] presto-native-execution 0.291-SNAPSHOT ............. SKIPPED
[INFO] presto-router 0.291-SNAPSHOT ....................... SKIPPED
[INFO] presto-open-telemetry 0.291-SNAPSHOT ............... SKIPPED
[INFO] redis-hbo-provider 0.291-SNAPSHOT .................. SKIPPED
[INFO] presto-openapi 0.291-SNAPSHOT ...................... SKIPPED
[INFO] presto-native-sidecar-plugin 0.291-SNAPSHOT ........ SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  1.256 s (Wall Clock)
[INFO] Finished at: 2025-01-17T13:02:39Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-release-plugin:2.5:prepare (default-cli) on project presto-root: Execution default-cli of goal org.apache.maven.plugins:maven-release-plugin:2.5:prepare failed: The data "- dependencyManagement is to fix the Require upper bound dependencies error for slf4j  " is not legal for a JDOM comment: Comment data cannot start with a hyphen.. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginExecutionException

Expected Behavior

Current Behavior

Possible Solution

There is an extra - in file

presto/presto-cassandra/pom.xml

Line 188 in ae68ae4

<!--- dependencyManagement is to fix the Require upper bound dependencies error for slf4j -->

@nishithakbhaskaran

Steps to Reproduce

When prepare the new release, the script will failed when running the command below

./mvnw release:prepare --batch-mode \
                                                           -DskipTests \
                                                           -DautoVersionSubmodules \
                                                           -DdevelopmentVersion=0.291 \
                                                           -DreleaseVersion=0.291

Screenshots (if appropriate)

Context

[ZacBlanco]

@unidevel I see the PR is merged. Can you close this if the issue is fixed?