<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Oct 13, 2015 at 8:53 AM, Cantor, Scott <span dir="ltr"><<a href="mailto:cantor.2@osu.edu" target="_blank">cantor.2@osu.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On 10/13/15, 8:47 AM, "users on behalf of Cantor, Scott" <<a href="mailto:users-bounces@shibboleth.net">users-bounces@shibboleth.net</a> on behalf of <a href="mailto:cantor.2@osu.edu">cantor.2@osu.edu</a>> wrote:<br>
<br>
<br>
<br>
>On 10/13/15, 5:29 AM, "users on behalf of TISSOT Jacques" <<a href="mailto:users-bounces@shibboleth.net">users-bounces@shibboleth.net</a> on behalf of <a href="mailto:jacques.tissot@unifr.ch">jacques.tissot@unifr.ch</a>> wrote:<br>
><br>
</span><span class="">>>It should be a single delete statement, but it looks like it's doing this and then a delete per result record:<br>
><br>
>SELECT distinct r.context FROM JPAStorageRecord r<br>
<br>
</span>Actually it's worse, it's loading every record and then doing a check on the expiration, so it's doing a ton of selects, yes. That's definitely something we need to fix.<br>
<br>
I assume/hope we can bypass the JPA abstraction and just issue a single delete. That's all it should need to do to clean up the table.</blockquote><div><br></div><div>Marvin mentioned to me that the delete appeared inefficient when he ran his test. Filed this issue for 3.2: <a href="https://issues.shibboleth.net/jira/browse/OSJ-133">https://issues.shibboleth.net/jira/browse/OSJ-133</a></div><div><br></div><div>--Daniel Fisher</div><div><br></div></div></div></div>