Skip to content

Conversation

@jerqi
Copy link
Contributor

@jerqi jerqi commented Dec 27, 2024

What changes were proposed in this pull request?

Remove repeated notice information

Why are the changes needed?

Fix: #2306

Does this PR introduce any user-facing change?

No.

How was this patch tested?

No need.

@github-actions
Copy link

Test Results

 2 966 files  ±0   2 966 suites  ±0   6h 27m 26s ⏱️ ±0s
 1 097 tests ±0   1 095 ✅ ±0   2 💤 ±0  0 ❌ ±0 
13 750 runs  ±0  13 720 ✅ ±0  30 💤 ±0  0 ❌ ±0 

Results for commit 48938cb. ± Comparison against base commit 1c56e74.

@jerqi
Copy link
Contributor Author

jerqi commented Dec 30, 2024

@zhaobing001 @zuston @kaijchen @advancedxy Could you help me review this pull request?

@jerqi jerqi merged commit e9e5904 into apache:master Dec 30, 2024
43 checks passed
jerqi added a commit that referenced this pull request Dec 30, 2024
### What changes were proposed in this pull request?

Remove repeated notice information

### Why are the changes needed?

Fix: #2306


### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
No need.
jerqi added a commit that referenced this pull request Jan 1, 2025
### What changes were proposed in this pull request?

Remove repeated notice information

### Why are the changes needed?

Fix: #2306


### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
No need.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] NOTICE-BINARY has 2 copies of the Jackson notice

2 participants