Justin Mclean

It may or may not be, just because they have the same or similar licenses is not enough to go on. By default, anything with a Category X license (which...

I've taken a look at the distribution artifact and it looks good to me. Due to the Category X license, this would not normally be OK, but it is in...

Yes, it would be best to only include the URL change. I'm not 100% sure that the new URL is correct as it has what seems to be extra characters...

If it is Apache 2.0 licensed, then their is no need to include the full license text again and yes it shoudl be listed in the LICENSE.bin file, see https://infra.apache.org/licensing-howto.html#alv2-dep....

Just to be clear, we need to know: 1. Where did these files come from? 2. How are they licensed? 3. If it a ALv2 compatible license, then it should...

@LauraXia123, Where exactly did the svg files come from? And how do you know they are licensed under the Apache license? Just because you copied it from an ASF site...

> Is that ok when we mention http://www.apache.org/licenses/LICENSE-2.0 in header of hive.svg and doris.svg file? @justinmclean No it is not OK. We can not add a license header to a...

The CI is failing as we use spotless to make sure the code has consistent formatting. You'll need to run `./gradlew :clients:cli:spotlessApply` and check in the changes to fix this.

@Brijeshthummar02, there are still some outstanding, unaddressed comments. You have also resolved several issues without fixing them. It's getting there, but I think it still needs a little work. My...

In the past, projects were encouraged to create their own bylaws, but this practice has largely fallen out of favour. Most now follow the default guidelines described in various places...