apache / apache/datafusion-sqlparser-rs
Fix licensing issues in source release found during 0.63.0 RC1 vote (missing NOTICE file, flamegraph.svg with CDDL-licensed JavaScript)
- Dominant language
- Rust
- Stars
- 3.5k
- Forks
- 772
- Avg merge
- 4d 9h
- Merged PRs (30d)
- 17
Description
**Describe the bug**
During the vote on the Apache DataFusion sqlparser-rs `0.63.0` RC1 release, @Xuanwo identified two licensing-material issues in the source archive and voted -1 (non-binding): https://lists.apache.org/thread/3lnhgs0bwzl148yrdrcl5g005h9pgtmx
1. **Missing `NOTICE` file**: the source archive contains `LICENSE.TXT` but no `NOTICE` file, which ASF releases are expected to include.
2. **`sqlparser_bench/img/flamegraph.svg` embeds third-party licensed code**: the generated flamegraph embeds JavaScript from [Inferno](https://github.com/jonhoo/inferno), whose upstream license is CDDL 1.0, but the archive contains no corresponding license materials.
**To Reproduce**
Download and unpack the 0.63.0 RC1 source archive (or check the repository at `v0.63.0-rc1`):
```shell
$ ls NOTICE*
ls: NOTICE*: No such file or directory
$ grep -c -i inferno sqlparser_bench/img/flamegraph.svg
2
```
**Expected behavior**
The source release complies with ASF licensing policy:
1. Add a `NOTICE` file to the repository so it is included in source archives.
2. Exclude the generated `sqlparser_bench/img/flamegraph.svg` from the source release
**Additional context**
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at the repository root with the 0.63.0-rc1 source archive and inspect NOTICE, LICENSE.TXT, and sqlparser_bench/img/flamegraph.svg; run the issue's ls and grep checks to confirm the findings. Done means the source release includes the required NOTICE file and excludes the generated flamegraph.svg without retaining the CDDL-licensed JavaScript.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- release
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 72/100