- Notifications
You must be signed in to change notification settings - Fork5
Commit5c80642
committed
Remove unnecessary int2vector-specific hash function and equality operator.
These functions were originally added in commitd8cedf6 to supportuse of int2vector columns as catcache lookup keys. However, there areno catcaches that use such columns. (Indeed I now think it must alwayshave been dead code: a catcache with such a key column would need anunderlying unique index on the column, but we've never had an int2vectorbtree opclass.)Getting rid of the int2vector-specific operator and function does notlose any functionality, because operations on int2vectors will now fallback to the generic anyarray support. This avoids a wart that a btreeindex on an int2vector column (made using anyarray_ops) would fail tomatch equality searches, because int2vectoreq wasn't a member of theopclass. We don't really care much about that, since int2vector is notmeant as a type for users to use, but it's silly to have extra code andless functionality.If we ever do want a catcache to be indexed by an int2vector column,we'd need to put back full btree and hash opclasses for int2vector,comparable to the support for oidvector. (The anyarray code can't beused at such a low level, because it needs to do catcache lookups.)But we'll deal with that if/when the need arises.Also worth noting is that removal of the hash int2vector_ops opclass willbreak any user-created hash indexes on int2vector columns. While hashanyarray_ops would serve the same purpose, it would probably not computethe same hash values and thus wouldn't be on-disk-compatible. Given thatint2vector isn't a user-facing type and we're planning other incompatiblechanges in hash indexes for v10 anyway, this doesn't seem like somethingto worry about, but it's probably worth mentioning here.Amit LangoteDiscussion: <d9bb74f8-b194-7307-9ebd-90645d377e45@lab.ntt.co.jp>1 parent8518583 commit5c80642
File tree
12 files changed
+1
-41
lines changed- src
- backend
- access/hash
- utils
- adt
- cache
- include
- access
- catalog
- utils
12 files changed
+1
-41
lines changedLines changed: 0 additions & 8 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
130 | 130 |
| |
131 | 131 |
| |
132 | 132 |
| |
133 |
| - | |
134 |
| - | |
135 |
| - | |
136 |
| - | |
137 |
| - | |
138 |
| - | |
139 |
| - | |
140 |
| - | |
141 | 133 |
| |
142 | 134 |
| |
143 | 135 |
| |
|
Lines changed: 0 additions & 15 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
254 | 254 |
| |
255 | 255 |
| |
256 | 256 |
| |
257 |
| - | |
258 |
| - | |
259 |
| - | |
260 |
| - | |
261 |
| - | |
262 |
| - | |
263 |
| - | |
264 |
| - | |
265 |
| - | |
266 |
| - | |
267 |
| - | |
268 |
| - | |
269 |
| - | |
270 |
| - | |
271 |
| - | |
272 | 257 |
| |
273 | 258 |
| |
274 | 259 |
| |
|
Lines changed: 0 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
126 | 126 |
| |
127 | 127 |
| |
128 | 128 |
| |
129 |
| - | |
130 |
| - | |
131 |
| - | |
132 |
| - | |
133 |
| - | |
134 | 129 |
| |
135 | 130 |
| |
136 | 131 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
283 | 283 |
| |
284 | 284 |
| |
285 | 285 |
| |
286 |
| - | |
287 | 286 |
| |
288 | 287 |
| |
289 | 288 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
53 | 53 |
| |
54 | 54 |
| |
55 | 55 |
| |
56 |
| - | |
| 56 | + | |
57 | 57 |
| |
58 | 58 |
|
Lines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
573 | 573 |
| |
574 | 574 |
| |
575 | 575 |
| |
576 |
| - | |
577 |
| - | |
578 | 576 |
| |
579 | 577 |
| |
580 | 578 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
173 | 173 |
| |
174 | 174 |
| |
175 | 175 |
| |
176 |
| - | |
177 | 176 |
| |
178 | 177 |
| |
179 | 178 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
168 | 168 |
| |
169 | 169 |
| |
170 | 170 |
| |
171 |
| - | |
172 | 171 |
| |
173 | 172 |
| |
174 | 173 |
| |
|
Lines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
156 | 156 |
| |
157 | 157 |
| |
158 | 158 |
| |
159 |
| - | |
160 |
| - | |
161 | 159 |
| |
162 | 160 |
| |
163 | 161 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
117 | 117 |
| |
118 | 118 |
| |
119 | 119 |
| |
120 |
| - | |
121 | 120 |
| |
122 | 121 |
| |
123 | 122 |
| |
|
Lines changed: 0 additions & 3 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
540 | 540 |
| |
541 | 541 |
| |
542 | 542 |
| |
543 |
| - | |
544 | 543 |
| |
545 | 544 |
| |
546 | 545 |
| |
| |||
687 | 686 |
| |
688 | 687 |
| |
689 | 688 |
| |
690 |
| - | |
691 |
| - | |
692 | 689 |
| |
693 | 690 |
| |
694 | 691 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
192 | 192 |
| |
193 | 193 |
| |
194 | 194 |
| |
195 |
| - | |
196 | 195 |
| |
197 | 196 |
| |
198 | 197 |
| |
|
0 commit comments
Comments
(0)