atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hemanth Yamijala <yhema...@gmail.com>
Subject Re: Review Request 47225: ATLAS-713 Atlas UI is still showing the deleted column of a hive table (Entity response of hive table does not have it). - with browser cache disabled.
Date Fri, 13 May 2016 03:41:38 GMT


> On May 12, 2016, 9:12 p.m., Suma Shivaprasad wrote:
> > repository/src/main/java/org/apache/atlas/discovery/HiveLineageService.java, lines
69-70
> > <https://reviews.apache.org/r/47225/diff/1/?file=1379386#file1379386line69>
> >
> >     while we are anyways doing this, should we rename HiveLineageService to DataSetLineageService

Change the class name and also the api? I know changing the API will be a breaking change.
But doing it now is better than later, no? The UI will be the most impacted from an internal
Atlas perspective. Other than that we will need to intimate Atlas dev list. We could pre-announce
on dev list *before* the patch gets in, IMO.


- Hemanth


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


On May 11, 2016, 9:21 a.m., Shwetha GS wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47225/
> -----------------------------------------------------------
> 
> (Updated May 11, 2016, 9:21 a.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-713
>     https://issues.apache.org/jira/browse/ATLAS-713
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> Current lineage APIs work on the dataset name. With soft delete, there can be more than
one entities for the given dataset name. 
> 1. Added new lineage APIs(input graph, output graph, schema) which work on entity id
> 2. Modified current lineage APIs to work on the active entity for the given dataset name
> 3. Modified UI to use the new APIs
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/atlas/AtlasClient.java b3ec95c 
>   dashboardv2/public/js/models/VLineage.js e33488a 
>   dashboardv2/public/js/models/VSchema.js 1f8e0bb 
>   dashboardv2/public/js/views/detail_page/DetailPageLayoutView.js 87adec0 
>   dashboardv2/public/js/views/graph/LineageLayoutView.js 973d091 
>   dashboardv2/public/js/views/schema/SchemaLayoutView.js 35487a5 
>   distro/src/conf/atlas-application.properties 119865d 
>   repository/src/main/java/org/apache/atlas/discovery/HiveLineageService.java 00905d7

>   repository/src/main/scala/org/apache/atlas/query/ClosureQuery.scala 05dc6a4 
>   repository/src/test/java/org/apache/atlas/BaseHiveRepositoryTest.java 66e1365 
>   repository/src/test/java/org/apache/atlas/discovery/HiveLineageServiceTest.java 6d5a15a

>   repository/src/test/scala/org/apache/atlas/query/GremlinTest2.scala 2fd8bb9 
>   server-api/src/main/java/org/apache/atlas/discovery/LineageService.java 8dc36cd 
>   typesystem/src/main/java/org/apache/atlas/typesystem/types/AttributeInfo.java 330d1cb

>   typesystem/src/main/resources/atlas-application.properties a343a20 
>   webapp/src/main/java/org/apache/atlas/web/resources/HiveLineageResource.java 9b3fbc9

>   webapp/src/main/java/org/apache/atlas/web/resources/LineageResource.java PRE-CREATION

>   webapp/src/test/java/org/apache/atlas/web/resources/HiveLineageJerseyResourceIT.java
bc02f90 
> 
> Diff: https://reviews.apache.org/r/47225/diff/
> 
> 
> Testing
> -------
> 
> UTs, ITs and end to end test with hive cli
> 
> 
> Thanks,
> 
> Shwetha GS
> 
>


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