Printemps cache @Cacheable méthode ignoré lorsqu'il est appelé à partir de l'intérieur de la même classe

Je suis en train d'appeler un @Cacheable méthode à partir de l'intérieur de la même classe:

@Cacheable(value = "defaultCache", key = "#id")
public Person findPerson(int id) {
   return getSession().getPerson(id);
} 

public List<Person> findPersons(int[] ids) {
   List<Person> list = new ArrayList<Person>();
   for (int id : ids) {
      list.add(findPerson(id));
   }
   return list;
}

et en espérant que les résultats de findPersons sont mis en cache, mais la @Cacheable annotation est ignoré, et findPerson méthode ont été exécutés à chaque fois.

Je fais quelque chose de mal ici, ou est-ce prévu?

InformationsquelleAutor David Zhao | 2012-08-24
  1. 40

    C'est à cause de la façon dont les procurations sont créés pour la manipulation de la mise en cache, de transaction liés à la fonctionnalité du Printemps. C'est une très bonne référence de comment le Printemps gère elle - Transactions, la mise en Cache et AOP: la compréhension de l'utilisation de proxy au Printemps

    En bref, une auto d'appel ignore la dynamique de proxy et tout transversales qui concernent comme la mise en cache, transaction, etc, qui fait partie de la dynamique de procurations de la logique est également ignorés.

    La solution est d'utiliser AspectJ moment de la compilation ou de temps de chargement de tissage.

    • Nous avons eu le même problème avec transactionnel annotations et a fait exactement ce que Biju posté. Pas de problèmes depuis.
    • Il suffit de lire le blog, pourriez-vous nous donner des précisions sur l'utilisation de AspectJ au moment de la compilation pour résoudre ce problème? thx!
    • J'ai un exemple d'utilisation de la mise en cache des annotations avec compilation aspectj tissage à - github.com/bijukunjummen/cache-sample.git , le comportement est similaire à la vôtre, une méthode d'auto de l'appel d'un @Cacheable méthode
    • Aussi, une mauvaise solution sera d'obtenir le proxy et de faire l'appel à l'aide que - ((PersonService)AopContext.currentProxy())).findPerson(id)
    • Nous n'avions que de l'inclure dans notre applicationContext.xml: <aop:aspectj-autoproxy proxy-cible class="true" /> qui a travaillé avec le même Transactionnelle annotations. N'essayez pas avec Cache méthodes bien
    • J'ai <aop:aspectj-autoproxy proxy-cible class="true" /> dans le contexte de configuration xml, mais encore l'appel de findPerson(id) de findPersons(ids) ne déclenchent pas la mise en cache de l'événement
    • Non David, même proxy-target-class=true ne fonctionnera pas, c'est juste les forces de la création de CGLIB en fonction des procurations, où la classe cible est sous-classé, mais encore s'enroule autour de l'objet traité par proxy - donc this dans le proxy de l'objet sera toujours résoudre le proxy de l'objet et de contourner le proxy. La seule solution est aspectj ou des appels via le proxy à travers ((PersonService)AopContext.currentProxy())).findPerson(id)
    • J'ai injecté une instance du service de l'applicationContext dans un post-construction de la méthode, et disant que, plutôt que de "ce" et qui a travaillé-
    • Mémoire cache interne de configuration peut être utilisé pour "proxify" les appels de méthode, voir l'exemple dans cette réponse: stackoverflow.com/a/48168762/907576

    InformationsquelleAutor Biju Kunjummen
  2. 18

    Voici ce que je fais pour les petits projets de marginal, et l'utilisation de la méthode des appels au sein de la même classe. Dans le code de documentation est fortement advidsed, comme il peut sembler strage à des collègues. Mais il est facile de tester, simple, rapide à réaliser et de pièces de rechange moi la pleine soufflé AspectJ instrumentation. Cependant, pour plus d'utilisation intensive, je serais d'avis de la AspectJ solution. 

    @Service
@Scope(proxyMode = ScopedProxyMode.TARGET_CLASS)
class PersonDao {

    private final PersonDao _personDao;

    @Autowired
    public PersonDao(PersonDao personDao) {
        _personDao = personDao;
    }

    @Cacheable(value = "defaultCache", key = "#id")
    public Person findPerson(int id) {
        return getSession().getPerson(id);
    }

    public List<Person> findPersons(int[] ids) {
        List<Person> list = new ArrayList<Person>();
        for (int id : ids) {
            list.add(_personDao.findPerson(id));
        }
        return list;
    }
}
    • C'est vraiment léger, et très utile!
    • Pourriez-vous veuillez expliquer pourquoi cela fonctionne après ajouter @Scope(proxyMode = ScopedProxyMode.TARGET_CLASS)? Ce qui est utilisé pour?
    • Parce que pour injecter PersonDao en PersonDao vous avez besoin d'avoir une instance de PersonDao avant, il est instancié 🙂 Pour contourner un proxy stub (voir la doc) est créé, injecté et rempli plus tard avec le même PersonDao exemple, il a obtenu injecté pour. Sooo, PersonDao peut contenir une instance de lui-même, enveloppé dans un proxy stub. c'est très Simple 🙂 Ne sais pas si c'est assez clair... 😀
    • Si j'étais la lecture de ce n'importe quel jour de la semaine, je pense que cela sonne comme dark, dark stuff que j'espère ne jamais avoir à l'utiliser. Puis je remarque que je suis en train de lire à propos de @Cacheable détails de mise en œuvre et les pièges... 🙂
    • Merci, ça marche toujours super! Note que j'ai eu à remplacer toutes les lignes avant de @Cacheable... find Person() par @Autowired private PersonDao _personDao;
    InformationsquelleAutor Mario Eis
  3. 2

    Pour toute personne utilisant la Graal Printemps Cache plugin, une solution de contournement décrite dans la documentation. J'ai eu ce problème sur un graal application, mais, malheureusement, l'on a accepté la réponse semble être inutilisable pour le Graal. La solution, c'est moche, à mon humble avis, mais il fonctionne.

    L'exemple de code montre bien:

    class ExampleService {
    def grailsApplication

    def nonCachedMethod() {
        grailsApplication.mainContext.exampleService.cachedMethod()
    }

    @Cacheable('cachedMethodCache')
    def cachedMethod() {
        //do some expensive stuff
    }
}

    Il suffit de remplacer exampleService.cachedMethod() avec votre propre service et de la méthode.

    InformationsquelleAutor Dónal Boyle