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

Unit test case 'planner/core TestPushdownDistinctEnable' fail on real TiKV #35834

Closed
tiancaiamao opened this issue Jun 30, 2022 · 1 comment
Closed
Labels
severity/moderate sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@tiancaiamao
Copy link
Contributor

Bug Report

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

I'm testing paging protocol by running tidb UT with a real TiKV using this branch #35647

tiup playground nightly --mode tikv-slim
cd planner/core
go test -run ^TestPushdownDistinctEnable$ -with-tikv 127.0.0.1:2379

2. What did you expect to see? (Required)

Success

3. What did you see instead (Required)

--- FAIL: TestPushdownDistinctEnable (3.81s)
    result.go:49: 
        	Error Trace:	result.go:49
        	            				physical_plan_test.go:1143
        	            				physical_plan_test.go:1056
        	Error:      	Not equal: 
        	            	expected: "[1 2]\n"
        	            	actual  : "[2 2]\n"
        	            	
        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1,2 +1,2 @@
        	            	-[1 2]
        	            	+[2 2]
        	            	 
        	Test:       	TestPushdownDistinctEnable
        	Messages:   	sql:select /*+ HASH_AGG() */ a, count(distinct a) from t;, args:[]
FAIL
exit status 1
FAIL	github.com/pingcap/tidb/planner/core	3.856s

4. What is your TiDB version? (Required)

master

commit 47a4f3a017768dda0b9a826451486c5539bc8427 (HEAD -> master, origin/master, origin/HEAD)
Author: Weizhen Wang <wangweizhen@pingcap.com>
Date:   Wed Jun 29 20:16:40 2022 +0800

    *: enable unconvert (#35821)
    
    ref pingcap/tidb#35345
@tiancaiamao tiancaiamao added type/bug The issue is confirmed as a bug. severity/moderate labels Jun 30, 2022
@jebter jebter added the sig/sql-infra SIG: SQL Infra label Jul 27, 2023
@Defined2014
Copy link
Contributor

TestPushdownDistinctEnable already moved into integration test. It runs good on unistore and tikv env, so we could close this issue right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity/moderate sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Development

No branches or pull requests

3 participants