accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4115) Thrift generation always replaces files
Date Mon, 25 Jan 2016 02:44:39 GMT

    [ https://issues.apache.org/jira/browse/ACCUMULO-4115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15114670#comment-15114670
] 

Josh Elser commented on ACCUMULO-4115:
--------------------------------------

Nah, not really.

> Thrift generation always replaces files
> ---------------------------------------
>
>                 Key: ACCUMULO-4115
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4115
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Christopher Tubbs
>            Assignee: Christopher Tubbs
>             Fix For: 1.8.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Since the switch to thrift 0.9.3 in ACCUMULO-4077, all thrift-generated files get a {{"date
= %Y-%m-%d"}} (I'm using `man date` syntax here) parameter to a {{@Generated}} annotation.
This causes all checked-in files to be replaced whenever the {{-Pthrift}} profile is used.
> This is going to cause problems for us when we run the release plugin, which activates
this profile, and will cause the resulting builds to differ from checked-in code. It will
also create a lot of noise in git when making small changes to thrift classes.
> Rather than copy over all changed files when running with the thrift profile, we should
only copy over files which differ by more than the date in this annotation. Alternatively,
we can suppress the dates using the java option for thrift {{generated_annotations=undated}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message