Closed
Bug 285443
Opened 20 years ago
Closed 20 years ago
Schema::Pg should remove FULLTEXT attribute from indexes
Categories
(Bugzilla :: Bugzilla-General, defect, P2)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: mkanat, Assigned: edwardjsabol)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
In order to be consistent with what's actually on the disk in the database, the
Pg _adjust_schema, at this point, should probably remove the FULLTEXT option
from all of the FULLTEXT indexes. (This is important so that later we can detect
from the serialized schema that the indexes aren't FULLTEXT and upgrade them.)
Reporter | ||
Updated•20 years ago
|
Summary: Schema::Pg should change FULLTEXT attribute to 0 for indexes → Schema::Pg should remove FULLTEXT attribute from indexes
Reporter | ||
Comment 1•20 years ago
|
||
Even better would be to do it in Schema::Pg::_initialize, so that we do it to
the abstract schema, too.
It should be done in Mysql::_initialize() after the call to
SUPER::_initialize().
I don't think you should be changing the abstract schema personally, but feel
free to take this (untested!) patch and run with it as you see fit, Max.
Reporter | ||
Comment 3•20 years ago
|
||
Comment on attachment 176892 [details] [diff] [review]
V1
Thank you, Ed. :-)
Yeah, I know that we're starting to do weird things with the Schema, but I had
a feeling that would happen once we actually had it. :-) It's really quite a
useful tool. :-)
Attachment #176892 -
Flags: review+
Reporter | ||
Comment 4•20 years ago
|
||
By the way, thank you for that ULTRA-fast patch, Ed. :-) That was awesome.
Assignee: general → edwardjsabol
Flags: approval?
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.20
Reporter | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Updated•20 years ago
|
What about removing FULLTEXT from the abstract schema? I presume that's for serialization purposes,
so I guess you could implement that in bug 285113 if deemed necessary.
Reporter | ||
Comment 6•20 years ago
|
||
Comment on attachment 176892 [details] [diff] [review]
V1
Oh, we don't change the {abstract_schema}, though.
Attachment #176892 -
Flags: review+ → review-
(In reply to comment #6)
> Oh, we don't change the {abstract_schema}, though.
Yeah, I was hoping to discuss that. It's easily enough rectified, but
I'm still coming to grips with the need to make the abstract schema
database-specific. Kind of goes contrary philosophically to the
original purpose. But I suppose it's necessary for your serialization
stuff, so here's another patch...
Attachment #176892 -
Attachment is obsolete: true
Attachment #176903 -
Flags: review?(mkanat)
Oh, that one has tabs. (I can't seem to turn that off in the Emacs perl-mode
that I'm using.)
Attachment #176903 -
Attachment is obsolete: true
Attachment #176905 -
Flags: review?(mkanat)
Reporter | ||
Comment 9•20 years ago
|
||
Comment on attachment 176903 [details] [diff] [review]
V2
Yeah, it's because of the serialization stuff.
Thanks, this looks good. :-)
Attachment #176903 -
Attachment is obsolete: false
Attachment #176903 -
Flags: review?(mkanat) → review+
Reporter | ||
Comment 10•20 years ago
|
||
Comment on attachment 176905 [details] [diff] [review]
V3
Oh, thanks for catching that. :-) Otherwise I would have had to fix it right
after checkin when the Tinderbox complained. :-)
Attachment #176905 -
Flags: review?(mkanat) → review+
Reporter | ||
Updated•20 years ago
|
Attachment #176903 -
Attachment is obsolete: true
Updated•20 years ago
|
Flags: approval? → approval+
Reporter | ||
Comment 11•20 years ago
|
||
Checking in Bugzilla/DB/Schema/Pg.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema/Pg.pm,v <-- Pg.pm
new revision: 1.3; previous revision: 1.2
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•