You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by "Bolke de Bruin (Jira)" <ji...@apache.org> on 2019/08/29 07:12:00 UTC

[jira] [Comment Edited] (ATLAS-3393) Fix inconsistency with attribute createtime in aws_s3_bucket

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

Bolke de Bruin edited comment on ATLAS-3393 at 8/29/19 7:11 AM:
----------------------------------------------------------------

I understand that backward compatibility is important. However, we encountered this bug due to integration issues with our tooling expecting "createTime" as it is used everywhere else[1]. I do think it will create more problem down the road if you leave this inconstency in. "createTime" is also used hardcoded in [2]. If it is considered to be an "internal" attribute what happens if you mix the two? Is there a way to alias one to the other?

 

So the trade off is that this inconsistency needs to be tested for in donwstream tooling and its not described in the documentation. So the question becomes how much tooling is already integrating with "createtime" and how much future tooling will assume "createTime". I think the latter will be bigger.

 

[1] [https://github.com/apache/atlas/search?q=createTime&unscoped_q=createTime]

[2] [https://github.com/apache/atlas/blob/7e4788ed00c6a0ef4e8c89f0486b8073a56409e4/intg/src/main/java/org/apache/atlas/model/instance/AtlasRelationship.java#L63]


was (Author: bolke):
I understand that backward compatibility is important. However, we encountered this bug due to integration issues with our tooling expecting "createTime" as it is used everywhere else[1]. I do think it will create more problem down the road if you leave this inconstency in. "createTime" is also used hardcoded in [2]. If it is considered to be an "internal" attribute what happens if you mix the two? Is there a way to alias one to the other?

 

[1] https://github.com/apache/atlas/search?q=createTime&unscoped_q=createTime


[2] https://github.com/apache/atlas/blob/7e4788ed00c6a0ef4e8c89f0486b8073a56409e4/intg/src/main/java/org/apache/atlas/model/instance/AtlasRelationship.java#L63

> Fix inconsistency with attribute createtime in aws_s3_bucket
> ------------------------------------------------------------
>
>                 Key: ATLAS-3393
>                 URL: https://issues.apache.org/jira/browse/ATLAS-3393
>             Project: Atlas
>          Issue Type: Bug
>    Affects Versions: 1.2.0, 2.0.0
>            Reporter: Bolke de Bruin
>            Priority: Critical
>
> attributes are in camelcase. in aws_s3_bukcket createtime isnt
>  
> (fix available in linked issue/pr)
>  
>  



--
This message was sent by Atlassian Jira
(v8.3.2#803003)