Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 357476 - Change caching default to ISOLATED for multitenant's using a shared EMF.
Summary: Change caching default to ISOLATED for multitenant's using a shared EMF.
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Guy Pelletier CLA
QA Contact:
URL: http://wiki.eclipse.org/EclipseLink/D...
Whiteboard:
Keywords:
Depends on:
Blocks: 357402
  Show dependency tree
 
Reported: 2011-09-13 08:15 EDT by Guy Pelletier CLA
Modified: 2022-06-09 10:09 EDT (History)
2 users (show)

See Also:


Attachments
Proposed changes (15.28 KB, patch)
2011-09-20 14:33 EDT, Guy Pelletier CLA
no flags Details | Diff
Updated patch (16.31 KB, patch)
2011-09-20 15:54 EDT, Guy Pelletier CLA
no flags Details | Diff
Patch for trunk (17.93 KB, patch)
2011-09-21 14:09 EDT, Guy Pelletier CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Guy Pelletier CLA 2011-09-13 08:15:04 EDT
Current default is PROTECTED. This should be changed to ISOLATED.
Comment 1 Gordon Yorke CLA 2011-09-13 08:45:15 EDT
From the original bug as entered by James:

"If we allow caching of tenant data as we currently are, then if I have a
BankAccount of Id 1234 at BMO, then call find() for BankAccount 1234 for TD, we
will get a cache hit for this, and return the TD user the BankAccount from BMO.
As well, a query that uses a CacheUsage of CheckCacheOnly, or
CheckCacheThenDatabase, will return data from other tenants.
Also, and update-all, or delete-all query, or flush-clear will invalidate data
from other tenants."
Comment 2 Guy Pelletier CLA 2011-09-20 14:33:40 EDT
Created attachment 203704 [details]
Proposed changes
Comment 3 Guy Pelletier CLA 2011-09-20 15:54:05 EDT
Created attachment 203710 [details]
Updated patch
Comment 4 Guy Pelletier CLA 2011-09-21 14:09:12 EDT
Created attachment 203795 [details]
Patch for trunk
Comment 5 Guy Pelletier CLA 2011-09-21 14:25:42 EDT
Changes have been submitted.

Verified by: Gordon Yorke, James Sutherland

Tests: N/A existing multitenant tests continue to pass.
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:09:10 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink