spring-projects / spring-framework

Spring Framework
https://spring.io/projects/spring-framework
Apache License 2.0
55.66k stars 37.77k forks source link

JPetStore doesn't work correctly on Application Server Clustering. [SPR-1742] #6439

Closed spring-projects-issues closed 12 years ago

spring-projects-issues commented 18 years ago

Takayuki Kaneko opened SPR-1742 and commented

HttpSession#setAttribute have to be called when Cart object is updated. Almost application servers use this method to decide whether they replicate objects.

The patch is below.

*** UpdateCartQuantitiesController.java Sat Feb 25 23:14:49 2006 --- UpdateCartQuantitiesController.java.modify Sat Feb 25 23:16:53 2006


* 34,39 ** --- 34,40 ---- //ignore on purpose } } +

  WebUtils.setSessionAttribute(request, "sessionCart", cart);
        return new ModelAndView("Cart", "cart", cart);
}

No further details from SPR-1742

spring-projects-issues commented 18 years ago

Juergen Hoeller commented

Thanks for pointing this out! I've adapted UpdateCartQuantitiesController accordingly.

That's hardly a critical issue though: It's just a sample app, and was never really attempting to be fully clusterable out-of-the-box... Hence I've lowered the priority to "minor".

Juergen

spring-projects-issues commented 18 years ago

Takayuki Kaneko commented

Thanks for adapting! I have seen your fix in CVS repository. I also understood about the priority.

JPetStore is a good sample app. I'll try it to be on the cluster. If I find out another issue, I'll post it as minor priority.

Sincerely,

-Takayuki