Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
@jincheng sun Thanks for your kind sharing. Regards, Lei Rui On 1/3/2020 16:01,jincheng sun wrote: I see. Thanks for your feedback! Lei Rui & Justin! As I know, Both Apache Beam[1] and Apache Flink[2] carry website updated PRs when bring up the release votes. This approach benefits Apache Beam and Apache Flink as the update changes of Beam and Flink are a little more complicated compare with Apache IoTDB, this approach speeds up the entire publishing process. However, the current update of the IoTDB website is a little bit easy, So, this approach has not brought obvious benefits. So, I am fine with keep current process of IoTDB. [1] https://lists.apache.org/thread.html/2fa0dd0e8710e62deab2e25c6081ed94636448f123d6f91ad190aa21%40%3Cdev.beam.apache.org%3E [2] https://lists.apache.org/thread.html/716337faba0160e8382543c8eb10bfa34895efc2419e94a1f3cc8d19%40%3Cdev.flink.apache.org%3E Best, Jincheng Lei Rui 于2020年1月3日周五 下午2:06写道: Hi, No need +1. I don't think it's a good practice, because if the released file doesn't exist, how can people review a PR like [1] where there are download links in it waiting to be checked, e.g., https://www.apache.org/dyn/closer.cgi/incubator/iotdb/0.9.0-incubating/apache-iotdb-0.9.0-incubating-source-release.zip . Regards, Lei Rui [1] https://github.com/apache/incubator-iotdb-website/commit/dc4d0226985c976561944068fd012e5cdfab4339 On 1/3/2020 13:52,jincheng sun wrote: No. You should only update the website after the IPMC vote has passes, the file put into the Apache mirrors and you waited 24 hours or so for the release to propagate. I mean that prepare the PR when bring up the VOTE, then people can review the PR ASAP. we can merge it when vote has passed and 24 hours later after the release . Best, Jincheng Lei Rui 于2020年1月3日周五 下午12:22写道: Thank Justin for the quick reply. Regards, Lei Rui On 1/3/2020 11:43,Justin Mclean wrote: Hi, Do you think that when bring up a release VOTE, we need to add the PR of update the website in VOTE thread? The changes of the PR would be similar with No. You should only update the website after the IPMC vote has passes, the file put into the Apache mirrors and you waited 24 hours or so for the release to propagate. Thanks, Justin
Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
I see. Thanks for your feedback! Lei Rui & Justin! As I know, Both Apache Beam[1] and Apache Flink[2] carry website updated PRs when bring up the release votes. This approach benefits Apache Beam and Apache Flink as the update changes of Beam and Flink are a little more complicated compare with Apache IoTDB, this approach speeds up the entire publishing process. However, the current update of the IoTDB website is a little bit easy, So, this approach has not brought obvious benefits. So, I am fine with keep current process of IoTDB. [1] https://lists.apache.org/thread.html/2fa0dd0e8710e62deab2e25c6081ed94636448f123d6f91ad190aa21%40%3Cdev.beam.apache.org%3E [2] https://lists.apache.org/thread.html/716337faba0160e8382543c8eb10bfa34895efc2419e94a1f3cc8d19%40%3Cdev.flink.apache.org%3E Best, Jincheng Lei Rui 于2020年1月3日周五 下午2:06写道: > Hi, > > > No need +1. > > > I don't think it's a good practice, because if the released file doesn't > exist, how can people review a PR like [1] where > there are download links in it waiting to be checked, e.g., > > https://www.apache.org/dyn/closer.cgi/incubator/iotdb/0.9.0-incubating/apache-iotdb-0.9.0-incubating-source-release.zip > . > > > Regards, > Lei Rui > > > [1] > https://github.com/apache/incubator-iotdb-website/commit/dc4d0226985c976561944068fd012e5cdfab4339 > On 1/3/2020 13:52,jincheng sun wrote: > No. You should only update the website after the IPMC vote has passes, > the file put into the Apache mirrors and you waited 24 hours or so for the > release to propagate. > > I mean that prepare the PR when bring up the VOTE, then people can review > the PR ASAP. we can merge it when vote has passed and 24 hours later after > the release . > > Best, > Jincheng > > > > Lei Rui 于2020年1月3日周五 下午12:22写道: > > Thank Justin for the quick reply. > > > Regards, > Lei Rui > On 1/3/2020 11:43,Justin Mclean wrote: > Hi, > > Do you think that when bring up a release VOTE, we need to add the PR of > update the website in VOTE thread? The changes of the PR would be similar > with > > No. You should only update the website after the IPMC vote has passes, the > file put into the Apache mirrors and you waited 24 hours or so for the > release to propagate. > > Thanks, > Justin >
Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
Hi, No need +1. I don't think it's a good practice, because if the released file doesn't exist, how can people review a PR like [1] where there are download links in it waiting to be checked, e.g., https://www.apache.org/dyn/closer.cgi/incubator/iotdb/0.9.0-incubating/apache-iotdb-0.9.0-incubating-source-release.zip. Regards, Lei Rui [1] https://github.com/apache/incubator-iotdb-website/commit/dc4d0226985c976561944068fd012e5cdfab4339 On 1/3/2020 13:52,jincheng sun wrote: No. You should only update the website after the IPMC vote has passes, the file put into the Apache mirrors and you waited 24 hours or so for the release to propagate. I mean that prepare the PR when bring up the VOTE, then people can review the PR ASAP. we can merge it when vote has passed and 24 hours later after the release . Best, Jincheng Lei Rui 于2020年1月3日周五 下午12:22写道: Thank Justin for the quick reply. Regards, Lei Rui On 1/3/2020 11:43,Justin Mclean wrote: Hi, Do you think that when bring up a release VOTE, we need to add the PR of update the website in VOTE thread? The changes of the PR would be similar with No. You should only update the website after the IPMC vote has passes, the file put into the Apache mirrors and you waited 24 hours or so for the release to propagate. Thanks, Justin
Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
Hi, > I mean that prepare the PR when bring up the VOTE, then people can review > the PR ASAP. we can merge it when vote has passed and 24 hours later after > the release . You could but I don’t see any great need, webpages are easily fixed if there is an issue. Thanks, Justin
Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
> No. You should only update the website after the IPMC vote has passes, the file put into the Apache mirrors and you waited 24 hours or so for the release to propagate. I mean that prepare the PR when bring up the VOTE, then people can review the PR ASAP. we can merge it when vote has passed and 24 hours later after the release . Best, Jincheng Lei Rui 于2020年1月3日周五 下午12:22写道: > Thank Justin for the quick reply. > > > Regards, > Lei Rui > On 1/3/2020 11:43,Justin Mclean wrote: > Hi, > > Do you think that when bring up a release VOTE, we need to add the PR of > update the website in VOTE thread? The changes of the PR would be similar > with > > No. You should only update the website after the IPMC vote has passes, the > file put into the Apache mirrors and you waited 24 hours or so for the > release to propagate. > > Thanks, > Justin
Re: [DISCUSS] Apache IoTDB (incubating) 0.9.1 RC1
Good find. I like the idea of a ticket to keep improving these types of things. On 1/1/2020 10:01 PM, Xiangdong Huang wrote: > Hi, > > using `find **/src -type f -exec grep -i "copyright" {} \; -print | sort > -u`, many apache header format issues can be found.. > > e.g., > 1. showTTLPlan.java > 2. all files for QueryMetricUI function... > 3. All scala files are using /** so the blank space is two chars while it > is one char in all java files. > 4. there are two blank spaces between "#" and content in all Python files > while it is one blank space in all sh files. > > I do not vote -1 but at least we need to fix these issues in next release. > > Best, > --- > Xiangdong Huang > School of Software, Tsinghua University > > 黄向东 > 清华大学 软件学院 > -- Kevin A. McGrail kmcgr...@apache.org Member, Apache Software Foundation Chair Emeritus Apache SpamAssassin Project https://www.linkedin.com/in/kmcgrail - 703.798.0171