Re: [HACKERS] Moving src/backend/utils/misc/rbtree.c to src/backend/lib

2014-12-22 Thread Robert Haas
On Mon, Dec 22, 2014 at 5:16 AM, Heikki Linnakangas wrote: > Peter Geoghegan suggested [1] moving rbtree.c to src/backend/lib, which I > think makes a lot of sense. Now that we have several other general purpose > data structures in src/backend/lib (linked lists, a binary heap, and a > pairing hea

Re: [HACKERS] Moving src/backend/utils/misc/rbtree.c to src/backend/lib

2014-12-22 Thread Heikki Linnakangas
On 12/22/2014 05:19 PM, Tom Lane wrote: However, wasn't there some speculation about removing rbtree entirely? Not that I recall. It's still used for GIN bulk loading. There might be better ways to do that, but there hasn't been any serious discussion on that. There was some discussion on r

Re: [HACKERS] Moving src/backend/utils/misc/rbtree.c to src/backend/lib

2014-12-22 Thread Tom Lane
Heikki Linnakangas writes: > Peter Geoghegan suggested [1] moving rbtree.c to src/backend/lib, which > I think makes a lot of sense. Now that we have several other general > purpose data structures in src/backend/lib (linked lists, a binary heap, > and a pairing heap), rbtree.c would definitely

[HACKERS] Moving src/backend/utils/misc/rbtree.c to src/backend/lib

2014-12-22 Thread Heikki Linnakangas
Peter Geoghegan suggested [1] moving rbtree.c to src/backend/lib, which I think makes a lot of sense. Now that we have several other general purpose data structures in src/backend/lib (linked lists, a binary heap, and a pairing heap), rbtree.c would definitely be better placed in src/backend/li