Stamatis,

I didn't know CALCITE-3129 before.
I was tracking CALCITE-5092 at that time, and I did some experiments
locally,
and reproduced the problem reported in CALCITE-5092, hence I thought that
this was the reason. I added that section in case that we will run into the
same
problem.

[1] https://issues.apache.org/jira/browse/CALCITE-5092

Stamatis Zampetakis <[email protected]> 于2022年9月13日周二 15:23写道:

> The new section about publishing-the-website [1] has been added rather
> recently.
>
> @Benchao Since you added this section can you explain the intention behind
> it?
> As Francis mentioned, after CALCITE-3129, the automatic workflow should
> take care of everything.
>
> Best,
> Stamatis
>
> [1]
>
> https://github.com/apache/calcite/blob/main/site/README.md#publishing-the-website
>
> On Sun, Sep 11, 2022 at 2:29 AM Francis Chuang <[email protected]>
> wrote:
>
> > It's fine to update the calcite.version and the news item in the same
> > commit as the workflow only looks at site files that were changed.
> >
> > The commits should always be pushed to main. The workflow automatically
> > cherry-picks the commit to site if it satisfies the following rules:
> >
> >
> https://github.com/apache/calcite/blob/main/.github/workflows/publish-non-release-website-updates.yml#L23
> >
> > Once cherry-picked to site, it automatically builds the site and
> > publishes it.
> >
> > In summary, commits should always be pushed to main and the workflow
> > will take care of the rest. Would you be able to update
> > https://github.com/apache/calcite/blob/main/site/README.md with this new
> > information? I think it would be very helpful for the next RM. I noticed
> > there's a section talking about copying the generated site to the
> > calcite-site repo
> > (
> >
> https://github.com/apache/calcite/blob/main/site/README.md#publishing-the-website
> )
> >
> > but I don't think we need to do that at all since the workflow /
> > automation takes care of it. Maybe it can be removed?
> >
> > Francis
> >
> >
> > On 11/09/2022 10:17 am, Julian Hyde wrote:
> > > Thanks for the information, Francis.
> > >
> > > I wrote the news item after the release, as part of the commit that
> > > advanced the version number, and pushed that commit to main. Should I
> > have
> > > split it into separate commits? Or pushed to site rather than main?
> > >
> > > FWIW, I was able to get the site to rebuild by making a trivial change
> to
> > > the news item and pushing to main. I then used a force-push to back it
> > out
> > > from both main and site branches. So everything is good now. The
> > artifacts,
> > > news item, history, and javadoc for 1.32 are all deployed.
> > >
> > > Julian
> > >
> > > On Sep 10, 2022, at 4:05 PM, Francis Chuang <[email protected]>
> > > wrote:
> > >
> > > The site and javadocs are automatically published when a release is
> > tagged
> > > using this workflow:
> > >
> >
> https://github.com/apache/calcite/blob/main/.github/workflows/publish-website-on-release.yml
> > >
> > > New news items should be automatically published via this workflow:
> > >
> >
> https://github.com/apache/calcite/blob/main/.github/workflows/publish-non-release-website-updates.yml
> > >
> > > I can see that the workflow for the news item failed due to the commit
> on
> > > site being on main already:
> > >
> https://github.com/apache/calcite/runs/8286036230?check_suite_focus=true
> > >
> > > Did you manually cherry-pick/rebase the commit to main? I think I'll
> need
> > > update the workflow to account for this edge case.
> > >
> > > Francis
> > >
> > >
> > > On 11/09/2022 2:57 am, Julian Hyde wrote:
> > >
> > > Does anyone (probably a previous release manager) know whether the
> > > site, javadoc, news are published automatically by release scripts?
> > > It seems that the site and javadoc are generated automatically on
> > release.
> > > But if I add a news item after the release and push it to main, does
> > > the site get regenerated? It doesn't seem so [1]. The instructions
> > > seem to indicate that pushing to the 'site' branch is sufficient [2].
> > > Julian
> > > [1] https://calcite.apache.org/news/
> > > [2] https://calcite.apache.org/docs/howto.html#publishing-a-release
> > >
> >
>


-- 

Best,
Benchao Li

Reply via email to