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.

Sign up for GitHub

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

Jump to bottom

[BugFix] Fix wrong plan in max(count(distinct)) when enable lowcardinality (backport #53403) #53416

Open
wants to merge 1 commit into
base: branch-3.3
Choose a base branch
from
+8 −2

[BugFix] Fix wrong plan in max(count(distinct)) when enable lowcardin…

f3e7c32
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

[BugFix] Fix wrong plan in max(count(distinct)) when enable lowcardinality (backport #53403) #53416

[BugFix] Fix wrong plan in max(count(distinct)) when enable lowcardin…
f3e7c32
Select commit
Loading
Failed to load commit list.
GitHub Actions / checkstyle succeeded Dec 2, 2024 in 0s

reviewdog [checkstyle] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (18)

fe/fe-common/src/main/java/com/starrocks/common/jmockit/MethodReflection.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
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/AutoType.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ThrowOfCheckedException.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/ConstructorReflection.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
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 '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/GeneratedClasses.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/Deencapsulation.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/fe-common/src/main/java/com/starrocks/common/jmockit/FieldReflection.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
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_]*$'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/StarRocksListPartitionerTest.java|1| Line does not match expected header line of '(^// Licensed to the Apache Software Foundation (ASF) under one$)|(^// Copyright 2021-present StarRocks, Inc. All rights reserved.$)'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/StarRocksListPartitionerTest.java|29 col 18| Local variable name 'arr3_1' must match pattern '^a-z?$'.
fe/spark-dpp/src/test/java/com/starrocks/load/loadv2/dpp/StarRocksListPartitionerTest.java|31 col 18| Local variable name 'arr3_2' must match pattern '^a-z?$'.

View more details on GitHub Actions