Hi Gerd,

I have adapted my patch.

Have a look here, will that be ok with you?

https://github.com/chrber/dcache/tree/fix/rt8203_update

Thanks.
Christian


-- 
Christian Bernardt
DESY Hamburg
Research Associate / Developer

Notkestr. 85
Building 02b, Room 213 
22607 Hamburg

phone:+49-(0)40-8998-2362
mail: christian.bernardt@desy.de




On 23 Jan 2014, at 19:00, Gerd Behrmann <notifications@github.com> wrote:

> In modules/dcache/src/main/java/org/dcache/pool/repository/RepositoryInterpreter.java:
> 
> >              for (int i = 0; i < args.argc(); i++) {
> >                  PnfsId pnfsid = new PnfsId(args.argv(i));
> > -                sb.append(_repository.getEntry(pnfsid));
> > -                sb.append("\n");
> > +                try {
> > +                    sb.append(_repository.getEntry(pnfsid));
> > +                    sb.append("\n");
> > +                } catch (FileNotInCacheException fnice) {
> > +                    cacheExceptions.add(fnice);
> > +                }
> > +            }
> > +            for (FileNotInCacheException cacheException : cacheExceptions) {
> > +                sb.append(cacheException).append("\n");
> I suggest using a nice error message than just the CacheException#toString output. At the very least CacheException#getMessage, but better to generate a string like "File not found: " (you can do that in the loop above, saving you from storing the rather large exception objects).
> 
> —
> Reply to this email directly or view it on GitHub.
>