accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4115) Thrift generation always replaces files
Date Sun, 24 Jan 2016 14:42:40 GMT

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

ASF GitHub Bot commented on ACCUMULO-4115:
------------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/accumulo/pull/65


> 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