On 2018-08-21 16:55:47 +1000, Haribabu Kommi wrote:
> On Sun, Aug 5, 2018 at 7:48 PM Andres Freund <andres@anarazel.de> wrote:
> > I'm currently in the process of rebasing zheap onto the pluggable
> > storage work. The goal, which seems to work surprisingly well, is to
> > find issues that the current pluggable storage patch doesn't yet deal
> > with. I plan to push a tree including a lot of fixes and improvements
> > soon.
> >
> That's good. Did you find any problems in porting zheap into pluggable
> storage? Does it needs any API changes or new API requirement?
A lot, yes. The big changes are:
- removal of HeapPageScanDesc
- introduction of explicit support functions for tablesample & bitmap scans
- introduction of callbacks for vacuum_rel, cluster
And quite a bit more along those lines.
OK. Those are quite a bit of changes.
> Does the new TupleTableSlot abstraction patches has fixed any of these
> issues in the recent thread [1]? so that I can look into the change of
> FDW API to return slot instead of tuple.
Yea, that'd be a good thing to start with.
I found out only the RefetchForeignRow API needs the change and done the same.
Along with that, I fixed all the issues of running make check-world. Attached patches
for the same.
Now I will look into the remaining FIXME's that don't conflict with your further changes.
Regards,
Haribabu Kommi
Fujitsu Australia