List:Internals« Previous MessageNext Message »
From:Paul Cadach Date:March 6 2001 7:16pm
Subject:Re: transparent query direction
View as plain text  
Hi,


> > To try to left some ditry work from you and Michael I'll
try to back-trace
> > fault until innobase's table handler to see what's wrong
there. And, if I
> > could, I'll create little bug-fix for it and send one
into this list. Ok?
>
> This would be great.

As I can see (without going deep through innobase's sources
and ha_innobase.cc) the trouble was at rem0cmp.c
(cmp_whole_field()). Because I've uses koi8_ru charset as
default, innobase uses DATA_VARMYSQL for varchar fields (and
DATA_VARCHAR for fields with charset latin1) which is not
listed in case() switch (found DATA_MYSQL only). Looking in
innobase_mysql_cmp() I've see that it supports for both
fixed and variable-length strings, so I've just added
DATA_VARMYSQL together with DATA_MYYSQL to the case in
cmp_whole_field().

Patch file is attached.


PS: Innobase handler is too slow comparing with
ISAM/MyISAM...


WBR,
Paul.


Thread
transparent query directionHroi Sigurdsson14 Feb
  • Re: transparent query directionRussell E Glaue14 Feb
  • Re: transparent query directionSasha Pachev15 Feb
    • Re: transparent query directionHroi Sigurdsson15 Feb
      • Re: transparent query directionPaul Repacholi15 Feb
      • Re: transparent query directionSasha Pachev15 Feb
    • Re: transparent query directionJeremy D. Zawodny15 Feb
      • Re: transparent query directionSasha Pachev15 Feb
        • Re: transparent query directionPaul Repacholi15 Feb
          • Re: transparent query directionSasha Pachev15 Feb
            • Re: transparent query directionPaul Repacholi16 Feb
        • Re: transparent query directionJeremy D. Zawodny21 Feb
          • Re: transparent query directionSasha Pachev21 Feb
            • Re: transparent query directionHroi Sigurdsson21 Feb
              • Re: transparent query directionSasha Pachev21 Feb
                • Re: transparent query directionHroi Sigurdsson21 Feb
                  • Re: transparent query directionSasha Pachev22 Feb
                    • Re: transparent query directionMichael Widenius3 Mar
                      • Re: transparent query directionSasha Pachev5 Mar
  • Re: transparent query directionPaul Cadach5 Mar
    • Re: transparent query directionSasha Pachev5 Mar
  • Re: transparent query directionPaul Cadach5 Mar
    • Re: transparent query directionSasha Pachev5 Mar
  • Re: transparent query directionPaul Cadach6 Mar
  • Re: transparent query directionPaul Cadach7 Mar
  • Re: transparent query directionPaul Cadach7 Mar
  • Re: transparent query directionPaul Cadach7 Mar
Re: transparent query directionMichael Widenius5 Mar
Re: transparent query directionHeikki Tuuri7 Mar