[jbosstools-issues] [JBoss JIRA] (JBIDE-17148) Improvements needed (regarding caching) in the URLTransport* API

[Fred Bricon (JIRA)]

     [ https://issues.jboss.org/browse/JBIDE-17148?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Fred Bricon updated JBIDE-17148:
--------------------------------

    Description: 
There are a few things I'd like to see improved in the URL Transport API, wrt caching.

Currently :
* all files are downloaded and cached under <workspace>/.metadata/.plugins/org.jboss.tools.foundation.core/ECF_REMOTE_CACHE, using a cryptic name
* Cache doesn't survive JVM restart

What I need/expect :
* ability to set custom cache folder 
* store files under <cache.folder>/requested.url. eg. : if I request https://foo.bar.org:8080/some/location/file.txt, it'll be stored under  <workspace>/.metadata/.plugins/my.plugin/cache/http_foo.bar.org_8080/some/location/file.txt
* restarting the IDE will be able to read the cached file

Currently URLTransportCache is a singleton. We could change that and add the cache location as a constructor argument, have a URLTransportCache.DEFAULT instance, used by URLTransportUtility and probably override 
{quote}
public File getCachedFileForURL(String url, String displayName, int lifespan, IProgressMonitor monitor) 
{quote}

with 
{quote}
public File getCachedFileForURL(String url, String displayName, URLTransportCache transportCache, IProgressMonitor monitor) 
{quote}

Or if you have any better idea, be my guest :-)



  was:
There are a few things I'd like to see improved in the URL Transport API, wrt caching.

Currently :
* all files are downloaded and cached under <workspace>/.metadata/.plugins/org.jboss.tools.foundation.core/ECF_REMOTE_CACHE, using a cryptic name
* Cache doesn't survive JVM restart

What I need/expect :
* ability to set custom cache folder 
* store files under <cache.folder>/requested.url. eg. : if I request https://foo.bar.org:8080/some/location/file.txt, it'll be stored under  <workspace>/.metadata/.plugins/org.jboss.tools.foundation.core/cache/http_foo.bar.org_8080/some/location/file.txt
* restarting the IDE will be able to read the cached file

Currently URLTransportCache is a singleton. We could change that and add the cache location as a constructor argument, have a URLTransportCache.DEFAULT instance, used by URLTransportUtility and probably override 
{quote}
public File getCachedFileForURL(String url, String displayName, int lifespan, IProgressMonitor monitor) 
{quote}

with 
{quote}
public File getCachedFileForURL(String url, String displayName, URLTransportCache transportCache, IProgressMonitor monitor) 
{quote}

Or if you have any better idea, be my guest :-)




    
> Improvements needed (regarding caching) in the URLTransport* API
> ----------------------------------------------------------------
>
>                 Key: JBIDE-17148
>                 URL: https://issues.jboss.org/browse/JBIDE-17148
>             Project: Tools (JBoss Tools)
>          Issue Type: Feature Request
>    Affects Versions: 4.2.0.Beta1
>            Reporter: Fred Bricon
>            Assignee: Rob Stryker
>             Fix For: 4.2.0.Beta2
>
>
> There are a few things I'd like to see improved in the URL Transport API, wrt caching.
> Currently :
> * all files are downloaded and cached under <workspace>/.metadata/.plugins/org.jboss.tools.foundation.core/ECF_REMOTE_CACHE, using a cryptic name
> * Cache doesn't survive JVM restart
> What I need/expect :
> * ability to set custom cache folder 
> * store files under <cache.folder>/requested.url. eg. : if I request https://foo.bar.org:8080/some/location/file.txt, it'll be stored under  <workspace>/.metadata/.plugins/my.plugin/cache/http_foo.bar.org_8080/some/location/file.txt
> * restarting the IDE will be able to read the cached file
> Currently URLTransportCache is a singleton. We could change that and add the cache location as a constructor argument, have a URLTransportCache.DEFAULT instance, used by URLTransportUtility and probably override 
> {quote}
> public File getCachedFileForURL(String url, String displayName, int lifespan, IProgressMonitor monitor) 
> {quote}
> with 
> {quote}
> public File getCachedFileForURL(String url, String displayName, URLTransportCache transportCache, IProgressMonitor monitor) 
> {quote}
> Or if you have any better idea, be my guest :-)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira