[KinoSearch] Re: Internal error: field_num 111 > max_field_num 6

[Aaron Crane]

Marvin Humphrey writes:
> That method is actually present in 0.15, in KinoSearch::Util::Class...
> 
>   sub CLONE {
>       my $package = shift;
>       die(      "CLONE invoked by package '$package', indicating that threads "
>               . "or Win32 fork were initiated, but KinoSearch is not thread-safe"
>       );
>   }
> 
> ... but I'd taken it out in the 0.20 branch.

Ah, I grepped through 0.20_03 for it, but didn't think to check 0.15.

> I was thinking that it was draconian to kill any threaded app that loaded
> any KinoSearch class, even if it was in some distant dependency somewhere
> and KS wasn't actually used.  Seemed like being a bad citizen, somehow.
> But I've been persuaded by this email thread that more pain results from
> the failure to throw an appropriate exception.

I know what you mean; it _is_ somewhat draconian.  But, as you point out,
it's better than the alternatives, given that KS really can't work in a
multi-threaded environment at the moment.

-- 
Aaron Crane