[
https://issues.apache.org/jira/browse/DRILL-8158?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17564963#comment-17564963
]
ASF GitHub Bot commented on DRILL-8158:
---------------------------------------
jnturton commented on PR #2590:
URL: https://github.com/apache/drill/pull/2590#issuecomment-1180318047
Something I noticed when looking for git.properties after building is that
it appeared in every jar files in jars/. It doesn't seem great for it to be
heavily duplicated like this...
```
/opt/apache-drill/jars/drill-common-2.0.0-SNAPSHOT.jar
git.properties
/opt/apache-drill/jars/drill-druid-storage-2.0.0-SNAPSHOT.jar
git.properties
/opt/apache-drill/jars/drill-format-esri-2.0.0-SNAPSHOT.jar
git.properties
/opt/apache-drill/jars/drill-format-excel-2.0.0-SNAPSHOT.jar
git.properties
/opt/apache-drill/jars/drill-format-hdf5-2.0.0-SNAPSHOT.jar
git.properties
/opt/apache-drill/jars/drill-format-httpd-2.0.0-SNAPSHOT.jar
...
```
> Remove non-reproducible build outputs
> -------------------------------------
>
> Key: DRILL-8158
> URL: https://issues.apache.org/jira/browse/DRILL-8158
> Project: Apache Drill
> Issue Type: Bug
> Affects Versions: 1.20.0
> Reporter: Herve Boutemy
> Assignee: James Turton
> Priority: Major
> Fix For: 1.20.2
>
>
> For context see [1] and [2]. The git-commit-id plugin includes information
> like build host, email and time which is not compatible with a reproducible
> build. Drill's built in sys.version table will return the build email and
> time if they are present in the build's git.properties file so these columns
> must be deprecated. Other useful Git-related information is retained.
> In accompanying commits, some Kerberos unit test fixes are applied, and the
> tests reenabled, and some updates to Release.md are included.
> [1] [https://maven.apache.org/guides/mini/guide-reproducible-builds.html]
> [2]
> [https://github.com/jvm-repo-rebuild/reproducible-central#org.apache.drill:drill-root]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
