I already fix that Julian Hyde <[email protected]>于2020年3月11日 周三上午2:00写道：
> Danny, > > Can you fix the tag please? > > Julian > > > > On Mar 6, 2020, at 10:26 AM, Vladimir Sitnikov < > [email protected]> wrote: > > > >> The RexNode normalization and project names remove from digest did > change > > a lot of plan from the Apache Flink side > > > > Hey Danny, I see it is dissatisfying, however, it is really sad you have > > never revealed which plans required changes and why. > > > >> The java doc doesn’t distinguish main API and test API, they are just > > together [1] > > > > I do not see "test API" there. Can you please clarify? > > > >> • The calcite-core does not publish the test sources now, should we add > it > > back ? > > > > We should **drop** publishing test artifacts via classifiers. > > The problem with classifier-style approach is those artifacts can't > declare > > their dependencies. > > > > We have already discussed it, and there was an agreement that we could > > extract calcite-test-framework or something like that. > > It could come as a separate artifact with its own properly-declared > > dependencies, sources, etc, etc. > > > >> Do we need to put the credit (commit name) after each commit message ? > It > > seems many projects put the credits in the last part together of the > > release note > > > > I think we should stop adding (contributor name) in the commit message. > > Git does have a proper field for the author's name. Of course committers > > need to ensure they don't overwrite that field by accident. > > > > Now we have Gradle, so we could even automate certain tasks. > > For instance, it could process Git history and build a list of > > contributors, etc, etc. > > > > > >> The RC artifacts was removed when I do the release, which actually I > think > > should not do, one awkward thing is that my network sucks > > > > What do you mean by "RC artifacts removed"? > > Release plugin does svn **MOVE** from repos/dist/dev to > repos/dist/release/ > > This makes it clear that the release artifacts are the same as the ones > > used for vote. > > > > > > Vladimir > >
