Remove duplicate LICENSE and NOTICE files from benchmark JARs

未关闭 适合新手
#3,695 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
72/100
Issue 类型
缺陷
描述清晰度
基本清楚
活跃度
活跃
技术栈
java
领域
build-system

调研方向

检查 parquet-benchmarks 和 parquet-plugins/parquet-plugins-benchmarks 模块中的 shade 配置。运行 ./mvnw --batch-mode -Pvector-plugins -DskipTests clean package,然后使用提供的检查来检查最终的 benchmark JAR。完成的标准是每个最终 JAR 都恰好包含一个 META-INF/LICENSE 和一个 META-INF/NOTICE,同时保留必需的归属信息。

由索引模型根据 Issue 内容生成。

描述

Similar to https://github.com/apache/iceberg/issues/17406

These local JMH artifacts are not published, so releases are unaffected. Update their shade configuration to retain only the intended META-INF/LICENSE and META-INF/NOTICE, while preserving required attribution in those canonical files.

The shaded benchmark uber-JARs contain multiple LICENSE/NOTICE variants:

  • parquet-benchmarks.jar
  • parquet-plugins-benchmarks.jar

Reproduction

# Build all default and vector-plugin JARs without running tests.
./mvnw --batch-mode -Pvector-plugins -DskipTests clean package

# Check all final JARs; exclude only Maven Shade backup artifacts.
(
  failed=0
  while IFS= read -r -d '' jar; do
    licenses=$(jar tf "$jar" | grep -Eic '(^|/)LICENSE([._-].*)?$' || true)
    notices=$(jar tf "$jar" | grep -Eic '(^|/)NOTICE([._-].*)?$' || true)

    if [ "$licenses" -eq 1 ] && [ "$notices" -eq 1 ]; then
      echo "PASS: $jar"
    else
      echo "FAIL: $jar (LICENSE=$licenses, NOTICE=$notices)"
      jar tf "$jar" | grep -Ei '(^|/)(LICENSE|NOTICE)([._-].*)?$'
      failed=1
    fi
  done < <(
    # original-* JARs are temporary backups created by Maven Shade.
    find . -type f -path '*/target/*.jar' \
      ! -name 'original-*.jar' \
      -print0 | sort -z
  )
  exit "$failed"
)

Observed failures:

FAIL: ./parquet-benchmarks/target/parquet-benchmarks.jar (LICENSE=9, NOTICE=4)
FAIL: ./parquet-plugins/parquet-plugins-benchmarks/target/parquet-plugins-benchmarks.jar (LICENSE=4, NOTICE=3)

Expected: every final JAR contains exactly one LICENSE and one NOTICE file under META-INF/.

主要语言
Java
星标
3.1k
派生
1.6k
平均合并
3 天 23 小时
30 天内合并 PR
30

贡献指南

这个仓库没有索引到贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

apache/parquet-java 的其他 Issue

查看 apache/parquet-java 的全部 Issue

相似的 Issue

更多 Java Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。