atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Madhan Neethiraj <mad...@apache.org>
Subject Re: Review Request 59264: ATLAS-1787: Replace explicit type with diamond operator
Date Mon, 15 May 2017 21:20:31 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/59264/#review175027
-----------------------------------------------------------



@Jan Hentschel: I would recommend to avoid large cleanup changes like this one - especially
as this might add overhead in backporting of future fixes from master to earlier versions.
Instead of updating all sources in one sweep, I would suggest to have new source files to
be complaint and fix exising sources when they are updated for a bug or a feature.

- Madhan Neethiraj


On May 14, 2017, 4:58 p.m., Jan Hentschel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/59264/
> -----------------------------------------------------------
> 
> (Updated May 14, 2017, 4:58 p.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-1787
>     https://issues.apache.org/jira/browse/ATLAS-1787
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> Replaced the remaining places where explicit types are used.
> 
> 
> Diffs
> -----
> 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1Graph.java
6a61075 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1GraphIndex.java
5ec1180 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/Titan1Vertex.java
1670e92 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/graphson/AtlasElementPropertyConfig.java
df67a83 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/graphson/AtlasGraphSONUtility.java
26abe2e 
>   graphdb/titan1/src/main/java/org/apache/atlas/repository/graphdb/titan1/serializer/StringListSerializer.java
eb99fae 
>   graphdb/titan1/src/test/java/org/apache/atlas/repository/graphdb/titan1/GraphQueryTest.java
45fbbea 
>   graphdb/titan1/src/test/java/org/apache/atlas/repository/graphdb/titan1/Titan1DatabaseTest.java
ab2c8c9 
>   intg/src/test/java/org/apache/atlas/type/TestAtlasObjectIdType.java fb4a183 
>   repository/src/main/java/org/apache/atlas/gremlin/optimizer/ExpandOrsOptimization.java
a48a007 
>   repository/src/main/java/org/apache/atlas/repository/converters/TypeConverterUtil.java
7902100 
>   repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedMetadataRepository.java
d1e8cb7 
>   repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java ca7fad0

>   repository/src/main/java/org/apache/atlas/repository/typestore/GraphBackedTypeStore.java
ac13586 
>   repository/src/main/java/org/apache/atlas/repository/typestore/TypeVertexFinder.java
8b38152 
>   repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java a4a596a

>   repository/src/test/java/org/apache/atlas/discovery/GraphBackedDiscoveryServiceTest.java
18573fc 
>   repository/src/test/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1Test.java
1dd7276 
>   webapp/src/main/java/org/apache/atlas/web/security/AtlasAbstractAuthenticationProvider.java
e55d2cf 
> 
> 
> Diff: https://reviews.apache.org/r/59264/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jan Hentschel
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message