Hi, > There are a lot of Datum *values, bool *nulls argument pairs that should > really be const. The 0001 patch makes those changes. Some of these > hunks depend on each other. > > The 0002 patch, which I'm not proposing to commit at this time, makes > similar changes but in a way that breaks the table and index AM APIs. > So I'm just including that here in case anyone wonders, why didn't you > touch those. And also maybe if we ever change that API incompatibly we > could throw this one in then.
0001 looks good to me and passes the tests in several environments, not surprisingly. Let's merge it. -- Best regards, Aleksander Alekseev
