Re: Ima::DBI patch for mod_perl compatibility
[prev]
[thread]
[next]
[Date index for 2005/06/10]
--Apple-Mail-1-536937777
Content-Transfer-Encoding: 7bit
Content-Type: text/plain;
charset=US-ASCII;
delsp=yes;
format=flowed
On Jun 10, 2005, at 1:39 PM, Tony Bowden wrote:
> On Fri, Jun 10, 2005 at 07:12:10PM +0100, William Ross wrote:
>
>> me too. not that this is a democracy, or anything.
>>
>
> You're quite correct, it's not :)
>
> Sheer force of numbers won't convince me. Dispelling my reservations
> will. And it'll only take one person to do that if the arguments are
> correct!
Anyone correct me if the argument i am posting is incorrect.
1) The suggested patch does not change existing behavior for current
applications that do not use mod_perl.
2) The performance is effectively unaffected by the patch.
3) This affects (wild guess) over half of the users of Class::DBI
and the community (not the "community" as you sarcastically referred
to earlier).
Not including this as a patch, but pushing this off to a plugin
doesn't help people because:
a) if they don't know that there is a workaround for a problem (and
this behavior is a problem), then the complex and frankly, icky,
workarounds on the wiki will continue and the impression will be that
C::DBI is problematic for mod_perl apps.
b) it fosters the opinion in the community that either class::DBI
is closed source, or not in active development, or more trouble than
it is worth, or simply not as cool as some of the other OO-RDBMS
mapping tools (not just perl).
That is the argument, from my POV.
My commentary on the whole thing is below.
Tony, you are the maintainer of the module, but not really the
owner. Patches coming in is a great thing. Users are not
complaining (always) when they send in a patch. They are trying to
make what they use better for everyone.
The "community" that uses it wants to make a great tool better, but
the way things look (from an observer like myself) is that you have
set in stone what C::DBI is, so we can all just deal.
-jason gessner
jason@xxxxxxxx.xxx
--Apple-Mail-1-536937777
Content-Transfer-Encoding: quoted-printable
Content-Type: text/html;
charset=ISO-8859-1
<HTML><BODY style=3D"word-wrap: break-word; -khtml-nbsp-mode: space; =
-khtml-line-break: after-white-space; "><BR><DIV><DIV>On Jun 10, 2005, =
at 1:39 PM, Tony Bowden wrote:</DIV><BR =
class=3D"Apple-interchange-newline"><BLOCKQUOTE type=3D"cite"><DIV =
style=3D"margin-top: 0px; margin-right: 0px; margin-bottom: 0px; =
margin-left: 0px; ">On Fri, Jun 10, 2005 at 07:12:10PM +0100, William =
Ross wrote:</DIV> <BR><BLOCKQUOTE type=3D"cite"><DIV style=3D"margin-top: =
0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">me too. =
not that this is a democracy, or anything.<SPAN =
class=3D"Apple-converted-space">=A0</SPAN></DIV> <BR></BLOCKQUOTE><DIV =
style=3D"margin-top: 0px; margin-right: 0px; margin-bottom: 0px; =
margin-left: 0px; min-height: 14px; "><BR></DIV><DIV style=3D"margin-top: =
0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">You're =
quite correct, it's not :)</DIV><DIV style=3D"margin-top: 0px; =
margin-right: 0px; margin-bottom: 0px; margin-left: 0px; min-height: =
14px; "><BR></DIV><DIV style=3D"margin-top: 0px; margin-right: 0px; =
margin-bottom: 0px; margin-left: 0px; ">Sheer force of numbers won't =
convince me. Dispelling my reservations</DIV><DIV style=3D"margin-top: =
0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">will. =
And it'll only take one person to do that if the arguments are</DIV><DIV =
style=3D"margin-top: 0px; margin-right: 0px; margin-bottom: 0px; =
margin-left: 0px; ">correct!</DIV></BLOCKQUOTE></DIV><BR><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">Anyone correct me if the =
argument i am posting is incorrect.</FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">1)=A0 The suggested patch =
does not change existing behavior for current applications that do not =
use mod_perl.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">2)=A0 The performance is effectively unaffected by the =
patch.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">3)=A0 =A0This affects (wild guess) over half of the =
users of Class::DBI and the community (not the "community" as you =
sarcastically referred to earlier).</FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">Not including this as a =
patch, but pushing this off to a plugin doesn't help people =
because:</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">a)=A0 =A0if they don't know that there is a workaround =
for a problem (and this behavior is a problem), then the complex and =
frankly, icky, workarounds on the wiki will continue and the impression =
will be that C::DBI is problematic for mod_perl =
apps.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">b) =A0 =A0it fosters the opinion in the community that =
either class::DBI is closed source, or not in active development, or =
more trouble than it is worth, or simply not as cool as some of the =
other OO-RDBMS mapping tools (not just perl).</FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">That is the argument, from =
my POV.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">My commentary on the whole =
thing is below.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">Tony, you are the =
maintainer of the module, but not really the owner.=A0 Patches coming in =
is a great thing.=A0 Users are not complaining (always) when they send =
in a patch.=A0 They are trying to make what they use better for =
everyone.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">The "community" that uses it wants to make a great =
tool better, but the way things look (from an observer like myself) is =
that you have set in stone what C::DBI is, so we can all just =
deal.</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD"><BR =
class=3D"khtml-block-placeholder"></FONT></DIV><DIV><FONT =
class=3D"Apple-style-span" color=3D"#0000DD">-jason =
gessner</FONT></DIV><DIV><FONT class=3D"Apple-style-span" =
color=3D"#0000DD">=A0=A0 =A0<A =
href=3D"mailto:jason@xxxxxxxx.xxx">jason@xxxxxxxx.xxx</A></FONT></DIV></BO=
DY></HTML>=
--Apple-Mail-1-536937777--