Thank you! On Mon, Feb 24, 2020 at 10:47 AM Suneel Marthi <[email protected]> wrote:
> https://issues.apache.org/jira/browse/HUDI-580 > > On Mon, Feb 24, 2020 at 1:42 PM Vinoth Chandar <[email protected]> wrote: > > > Hi, > > > > Have a filed a JIRA for this, tagged with 0.5.2? > > > > Thanks > > Vinoth > > > > On Sat, Feb 22, 2020 at 6:58 AM lamberken <[email protected]> wrote: > > > > > > > > > > > Right, will do. > > > > > > > > > Thanks, > > > Lamber-Ken > > > > > > At 2020-02-22 22:35:13, "[email protected]" <[email protected]> > wrote: > > > > > > > >+1 on ensuring all scripts in Hudi codebase follow same convention for > > > licensing. > > > >Balaji.V On Saturday, February 22, 2020, 06:16:29 AM PST, Suneel > > > Marthi <[email protected]> wrote: > > > > > > > > Please go ahead and make the change @lamberken > > > > > > > >I was just looking at scripts from Hive and Kafka projects, see below. > > > > > > > >https://github.com/apache/hive/blob/master/bin/init-hive-dfs.sh > > > >https://github.com/apache/hive/blob/master/bin/hive-config.sh > > > > > > > >https://github.com/apache/kafka/blob/trunk/bin/connect-distributed.sh > > > > > https://github.com/apache/kafka/blob/trunk/bin/kafka-leader-election.sh > > > > > > > >I suggest to fix all the script files to be consistent with apache > > license > > > >guide. > > > > > > > > > > > > > > > >On Sat, Feb 22, 2020 at 8:53 AM lamberken <[email protected]> wrote: > > > > > > > >> > > > >> > > > >> Hi all, > > > >> > > > >> > > > >> During the voting process on rc1 0.5.1-incubating release, Justin > > > pointed > > > >> out > > > >> docker/hoodie/hadoop/base/entrypoint.sh has an incorrect license > > header, > > > >> But, many script files used the same license header like > > "entrypoint.sh" > > > >> has. > > > >> > > > >> > > > >> From apache license guide[2], it says "The text should be enclosed > in > > > the > > > >> appropriate comment syntax for the file format." > > > >> So, need to remove the repeated "#", like following changes? > > > >> > > > >> > > > >> > > > >> > > > > > > ################################################################################ > > > >> # Licensed to the Apache Software Foundation (ASF) under one > > > >> # or more contributor license agreements. See the NOTICE file > > > >> # distributed with this work for additional information > > > >> # regarding copyright ownership. The ASF licenses this file > > > >> # to you under the Apache License, Version 2.0 (the > > > >> # "License"); you may not use this file except in compliance > > > >> # with the License. You may obtain a copy of the License at > > > >> # > > > >> # http://www.apache.org/licenses/LICENSE-2.0 > > > >> # > > > >> # Unless required by applicable law or agreed to in writing, > software > > > >> # distributed under the License is distributed on an "AS IS" BASIS, > > > >> # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or > > > implied. > > > >> # See the License for the specific language governing permissions > and > > > >> # limitations under the License. > > > >> > > > >> > > > > > > ################################################################################ > > > >> > > > >> > > > >> # > > > >> # Licensed to the Apache Software Foundation (ASF) under one > > > >> # or more contributor license agreements. See the NOTICE file > > > >> # distributed with this work for additional information > > > >> # regarding copyright ownership. The ASF licenses this file > > > >> # to you under the Apache License, Version 2.0 (the > > > >> # "License"); you may not use this file except in compliance > > > >> # with the License. You may obtain a copy of the License at > > > >> # > > > >> # http://www.apache.org/licenses/LICENSE-2.0 > > > >> # > > > >> # Unless required by applicable law or agreed to in writing, > software > > > >> # distributed under the License is distributed on an "AS IS" BASIS, > > > >> # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or > > > implied. > > > >> # See the License for the specific language governing permissions > and > > > >> # limitations under the License. > > > >> # > > > >> > > > >> > > > >> Any thought are welcome, thanks. > > > >> > > > >> > > > >> Thanks, > > > >> Lamber-Ken > > > >> > > > >> > > > >> [1] > > > >> > > > > > > https://lists.apache.org/thread.html/rd3f4a72d82a4a5a81b2c6bd71e1417054daa38637ce8e07901f26f04%40%3Cgeneral.incubator.apache.org%3E > > > >> [2] https://www.apache.org/licenses/LICENSE-2.0 > > > >> > > > >> > > > > > > > > > >
