Re: Removing unused parameter in SnapBuildGetOrBuildSnapshot

2022-09-25 Thread Amit Kapila
On Fri, Sep 23, 2022 at 8:58 AM Amit Kapila wrote: > > On Wed, Sep 21, 2022 at 8:11 PM Zhang Mingli wrote: > > > > On Sep 21, 2022, 22:22 +0800, Melih Mutlu , wrote: > > > > Hi hackers, > > > > Sharing a small patch to remove an unused parameter in > > SnapBuildGetOrBuildSnapshot function in sna

Re: Removing unused parameter in SnapBuildGetOrBuildSnapshot

2022-09-22 Thread Amit Kapila
On Wed, Sep 21, 2022 at 8:11 PM Zhang Mingli wrote: > > On Sep 21, 2022, 22:22 +0800, Melih Mutlu , wrote: > > Hi hackers, > > Sharing a small patch to remove an unused parameter in > SnapBuildGetOrBuildSnapshot function in snapbuild.c > > With commit 6c2003f8a1bbc7c192a2e83ec51581c018aa162f, Sna

Re: Removing unused parameter in SnapBuildGetOrBuildSnapshot

2022-09-21 Thread Zhang Mingli
On Sep 21, 2022, 22:22 +0800, Melih Mutlu , wrote: > Hi hackers, > > Sharing a small patch to remove an unused parameter in  > SnapBuildGetOrBuildSnapshot function in snapbuild.c > > With commit 6c2003f8a1bbc7c192a2e83ec51581c018aa162f, SnapBuildBuildSnapshot > no longer needs transaction id. This