I have gone over those deferred PRs that were good to go and committed them. Nothing blocker from my side.
Haisheng On 2020/05/24 20:03:00, Haisheng Yuan <[email protected]> wrote: > OK, let's keep master closed for now. I can confirm that those PRs tagged > with LGTM are ready to merge. I can go ahead and merge them, but for others > we need other reviewers to help confirm. > > On 2020/05/24 19:53:09, Julian Hyde <[email protected]> wrote: > > Yes, review and (if they are ready) commit them before opening master. > > > > On Sun, May 24, 2020 at 11:30 AM Haisheng Yuan <[email protected]> wrote: > > > > > > I went over the JIRAs, I marked those fix-for-1.23.0 (but didn't get > > > chance to get into 1.23.0) to 1.24.0. > > > By handling those deferred PRs, do you mean we first review and commit > > > these PRs? > > > > > > Thanks, > > > Haisheng > > > > > > On 2020/05/24 18:22:59, Julian Hyde <[email protected]> wrote: > > > > What about the plan to keep it closed until we handled those deferred > > > > PRs? > > > > > > > > Julian > > > > > > > > > On May 24, 2020, at 08:19, Haisheng Yuan <[email protected]> wrote: > > > > > > > > > > Hi all, > > > > > > > > > > Calcite master branch is open for commit now. > > > > > > > > > > Cheers, > > > > > Haisheng > > > > > > >
