[mapserver-dev] Mapserver thread safety

[thomas bonfort]

+1 for merging

On 14 August 2013 09:46, Umberto Nicoletti <umberto.nicoletti at gmail.com> wrote:
> On Wed, Aug 14, 2013 at 9:39 AM, Andreas Albarello
> <andreas.albarello at tol.bz.it> wrote:
>>
>> Umberto,
>>
>> 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.
>>
>
> I have added that after I sent the first email, so I suppose it's good for
> merging.
>
> Umberto
>
>>
>> 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.
>>
>> Regards,
>>
>> Andreas
>>
>> On 14.08.2013 09:07,umberto.nicoletti at gmail.com wrote:
>> > I have added OCI_THREADED to the flags passed to OciEnvCreate as per
>> > Andreas suggestion (and fixed a few compile warnings while at it).
>> >
>> > You can checkout the changes:
>> >
>> > https://github.com/mapserver/mapserver/pull/4729
>> >
>> > I'd like to merge this into master today, so I'd really appreciate it if
>> > someone could test it right away. I'll be on #mapserver if you need
>> > assistance.
>>
>>
>> --
>> --------------------------------------------------------------------
>> Andreas Albarello
>> Analysis & SW Development
>>
>> Territorium Online srl/GmbH
>> Via Buozzi Str. 12
>> I 39100 Bolzano/Bozen
>>
>> Phone:  +39 0471 068611
>> Fax:    +39 0471 068619
>>
>> email: andreas.albarello at tol.bz.it
>> web:   http://www.tol.bz.it
>> --------------------------------------------------------------------
>>
>>
>
>
> _______________________________________________
> mapserver-dev mailing list
> mapserver-dev at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/mapserver-dev
>