<div dir="ltr">On Wed, Aug 14, 2013 at 9:39 AM, Andreas Albarello <span dir="ltr"><<a href="mailto:andreas.albarello@tol.bz.it" target="_blank">andreas.albarello@tol.bz.it</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Umberto,<br>
<br>
thank you. I think this should be ok, and I agree with Thomas' comment to the original pull request that OCI_THREADED should require USE_THREADS to be defined.<br>
<br></blockquote><div><br></div><div>I have added that after I sent the first email, so I suppose it's good for merging.</div><div><br></div><div>Umberto</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Unfortunately, I lost my MapServer build environment in a HDD crash two weeks ago and haven't yet had time to put everything together again, so I can't help with testing this right now. I will say though that I have been compiling MapServer with OCI_THREAEDED in maporaclespatial.c for some time now and haven't noticed any adverse effects. As I said, I did find though that some weird crashes in conjunction with an 11g client (10g clients did seem to be a lot more stable in this regard) went away which I think is a good thing and what prompted my response in the first place.<br>

<br>
Regards,<br>
<br>
Andreas<br>
<div class="im HOEnZb"><br>
On 14.08.2013 09:07,<a href="mailto:umberto.nicoletti@gmail.com">umberto.nicoletti@gmail.com</a> wrote:<br>
> I have added OCI_THREADED to the flags passed to OciEnvCreate as per<br>
> Andreas suggestion (and fixed a few compile warnings while at it).<br>
><br>
> You can checkout the changes:<br>
><br>
> <a href="https://github.com/mapserver/mapserver/pull/4729" target="_blank">https://github.com/mapserver/mapserver/pull/4729</a><br>
><br>
> I'd like to merge this into master today, so I'd really appreciate it if<br>
> someone could test it right away. I'll be on #mapserver if you need<br>
> assistance.<br>
<br>
<br>
</div><div class="HOEnZb"><div class="h5">--<br>
--------------------------------------------------------------------<br>
Andreas Albarello<br>
Analysis & SW Development<br>
<br>
Territorium Online srl/GmbH<br>
Via Buozzi Str. 12<br>
I 39100 Bolzano/Bozen<br>
<br>
Phone:  <a href="tel:%2B39%200471%20068611" value="+390471068611">+39 0471 068611</a><br>
Fax:    <a href="tel:%2B39%200471%20068619" value="+390471068619">+39 0471 068619</a><br>
<br>
email: <a href="mailto:andreas.albarello@tol.bz.it">andreas.albarello@tol.bz.it</a><br>
web:   <a href="http://www.tol.bz.it" target="_blank">http://www.tol.bz.it</a><br>
--------------------------------------------------------------------<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>