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
