[jira] [Commented] (ATLAS-856) Lazy-load type cache provider

2016-06-21 Thread ATLAS QA (JIRA)

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

ATLAS QA commented on ATLAS-856:


{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment
  http://issues.apache.org/jira/secure/attachment/12812243/ATLAS-856.patch
  against master revision 7666ae9.

{color:red}-1 patch{color}.  The patch command could not apply the patch.

Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/348//console

This message is automatically generated.

> Lazy-load type cache provider
> -
>
> Key: ATLAS-856
> URL: https://issues.apache.org/jira/browse/ATLAS-856
> Project: Atlas
>  Issue Type: New Feature
>Reporter: David Kantor
>Assignee: David Kantor
> Attachments: ATLAS-856.patch, rb48565.patch
>
>
> A modified version of the default local cache provider which will run a 
> lookup on the backend storage if a requested type is not found in the cache, 
> and load it in the cache and return it if it's found. This lazy-loading cache 
> would cover the case where new types are registered, not the case where 
> existing types are modified, while avoiding the deployment/maintenance and 
> performance impact of a distributed cache in an HA environment.



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


[jira] [Commented] (ATLAS-856) Lazy-load type cache provider

2016-06-21 Thread David Kantor (JIRA)

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

David Kantor commented on ATLAS-856:


[~shwethags] Yes your changes make sense, good idea to use Configuration and 
mutate the property setting needed for the tests.  I noticed your patch does a 
delete and add of classes that I renamed.  Rather than lose the history of 
those renamed modules, I will apply your changes in my source, and upload a new 
set of changes to review board and attach an updated.  That way the renames 
will be recorded in git correctly.  Thanks very much for your help on this.

> Lazy-load type cache provider
> -
>
> Key: ATLAS-856
> URL: https://issues.apache.org/jira/browse/ATLAS-856
> Project: Atlas
>  Issue Type: New Feature
>Reporter: David Kantor
>Assignee: David Kantor
> Attachments: ATLAS-856.patch, rb48565.patch
>
>
> A modified version of the default local cache provider which will run a 
> lookup on the backend storage if a requested type is not found in the cache, 
> and load it in the cache and return it if it's found. This lazy-loading cache 
> would cover the case where new types are registered, not the case where 
> existing types are modified, while avoiding the deployment/maintenance and 
> performance impact of a distributed cache in an HA environment.



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


[jira] [Commented] (ATLAS-856) Lazy-load type cache provider

2016-06-21 Thread Shwetha G S (JIRA)

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

Shwetha G S commented on ATLAS-856:
---

[~dkantor], the attached patch has some changes to not create 
atlas-application.properties in respository/test. See if the changes make sense 
to you

> Lazy-load type cache provider
> -
>
> Key: ATLAS-856
> URL: https://issues.apache.org/jira/browse/ATLAS-856
> Project: Atlas
>  Issue Type: New Feature
>Reporter: David Kantor
>Assignee: David Kantor
> Attachments: ATLAS-856.patch, rb48565.patch
>
>
> A modified version of the default local cache provider which will run a 
> lookup on the backend storage if a requested type is not found in the cache, 
> and load it in the cache and return it if it's found. This lazy-loading cache 
> would cover the case where new types are registered, not the case where 
> existing types are modified, while avoiding the deployment/maintenance and 
> performance impact of a distributed cache in an HA environment.



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


[jira] [Commented] (ATLAS-856) Lazy-load type cache provider

2016-06-15 Thread ATLAS QA (JIRA)

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

ATLAS QA commented on ATLAS-856:


{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment
  http://issues.apache.org/jira/secure/attachment/12810859/rb48565.patch
  against master revision 39eb9f1.

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 2 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

+1 checkstyle.  The patch generated 0 code style errors.

{color:red}-1 findbugs{color}.  The patch appears to introduce 380 new 
Findbugs (version 2.0.3) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:red}-1 core tests{color}.  The patch failed these unit tests in :
   org.apache.atlas.discovery.DataSetLineageServiceTest
  org.apache.atlas.service.DefaultMetadataServiceTest
  org.apache.atlas.discovery.GraphBackedDiscoveryServiceTest
  
org.apache.atlas.repository.typestore.StoreBackedTypeCacheProviderTest
./repository/target/surefire-reports/junitreports/TEST-org.apache.atlas.discovery.DataSetLineageServiceTest
./repository/target/surefire-reports/junitreports/TEST-org.apache.atlas.service.DefaultMetadataServiceTest
./repository/target/surefire-reports/junitreports/TEST-org.apache.atlas.discovery.GraphBackedDiscoveryServiceTest
  org.apache.atlas.repository.typestore.GraphBackedTypeStoreTest

Test results: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningswebapp.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsauthorization.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningscommon.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningssqoop-bridge.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningshdfs-model.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsstorm-bridge.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsfalcon-bridge.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningshive-bridge.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsrepository.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningstypesystem.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningscatalog.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsclient.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningsnotification.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-ATLAS-Build/301//artifact/patchprocess/newPatchFindbugsWarningstitan.html
Console output: https://builds.apache.org/job/PreCommit-ATLAS-Build/301//console

This message is automatically generated.

> Lazy-load type cache provider
> -
>
> Key: ATLAS-856
> URL: https://issues.apache.org/jira/browse/ATLAS-856
> Project: Atlas
>  Issue Type: New Feature
>Reporter: David Kantor
>Assignee: David Kantor
> Attachments: rb48565.patch
>
>
> A modified version of the default local cache provider which will run a 
> lookup on the backend storage if a requested type is not found in the cache, 
> and load it in the cache and return it if it's found. This lazy-loading cache 
> would cover the case where new types are registered, not the case where 
> existing types are modified, while avoiding the deployment/maintenance and 
> performance impact of a distributed cache in an HA environment.



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