Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BugFix] Fix http server direct buffer leak for request body (backport #49395) #49628

Merged
merged 2 commits into from
Aug 9, 2024

resolve

0fa2db4
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

[BugFix] Fix http server direct buffer leak for request body (backport #49395) #49628

resolve
0fa2db4
Select commit
Loading
Failed to load commit list.
GitHub Actions / checkstyle succeeded Aug 9, 2024 in 0s

reviewdog [checkstyle] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (15)

fe/fe-common/src/main/java/com/starrocks/common/jmockit/ThrowOfCheckedException.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ConstructorReflection.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ConstructorReflection.java|116| Line is longer than 130 characters (found 158).
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ConstructorReflection.java|117| Line is longer than 130 characters (found 137).
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ParameterReflection.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/MethodReflection.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/MethodReflection.java|99| Line is longer than 130 characters (found 147).
fe/fe-common/src/main/java/com/starrocks/common/jmockit/MethodReflection.java|128| Line is longer than 130 characters (found 140).
fe/fe-common/src/main/java/com/starrocks/common/jmockit/Deencapsulation.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/GeneratedClasses.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/FieldReflection.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/AutoType.java|1| Line does not match expected header line of '(^// Copyright 2021-present StarRocks, Inc. All rights reserved.)|(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// This file is licensed under the Elastic License 2.0. Copyright 2021-present, StarRocks Inc.$)|(^// This file is made available under Elastic License 2.0.$)'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/HllTest.java|36 col 50| Should use uppercase 'L'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/HllTest.java|37 col 50| Should use uppercase 'L'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/HllTest.java|41 col 17| Method name 'HllBasicTest' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9_]*$'.