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

[close #679] Let ColumnRef case insensitive #678

Merged
merged 3 commits into from
Dec 20, 2022

Conversation

shiyuhang0
Copy link
Collaborator

@shiyuhang0 shiyuhang0 commented Dec 8, 2022

Signed-off-by: shiyuhang [email protected]

What problem does this PR solve?

Problem Description: close #679

ColumnRef should be case insensitive

What is changed and how does it work?

Change the hashcode method

Signed-off-by: shiyuhang <[email protected]>
@shiyuhang0 shiyuhang0 changed the title Let ColumnRef case insensitive [close #679] Let ColumnRef case insensitive Dec 8, 2022
@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Base: 35.01% // Head: 35.14% // Increases project coverage by +0.13% 🎉

Coverage data is based on head (bc50d53) compared to base (24ed9e2).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #678      +/-   ##
============================================
+ Coverage     35.01%   35.14%   +0.13%     
- Complexity     1446     1453       +7     
============================================
  Files           278      278              
  Lines         17382    17382              
  Branches       1975     1975              
============================================
+ Hits           6086     6109      +23     
+ Misses        10684    10662      -22     
+ Partials        612      611       -1     
Impacted Files Coverage Δ
...ain/java/org/tikv/common/expression/ColumnRef.java 0.00% <0.00%> (ø)
src/main/java/io/grpc/netty/WriteQueue.java 74.43% <0.00%> (-2.26%) ⬇️
src/main/java/io/grpc/internal/ClientCallImpl.java 56.34% <0.00%> (-0.74%) ⬇️
...rc/main/java/io/grpc/netty/NettyClientHandler.java 57.11% <0.00%> (-0.65%) ⬇️
src/main/java/org/tikv/common/PDClient.java 58.98% <0.00%> (+0.24%) ⬆️
.../main/java/org/tikv/common/region/RegionCache.java 50.92% <0.00%> (+1.85%) ⬆️
...va/org/tikv/common/region/StoreHealthyChecker.java 75.94% <0.00%> (+2.53%) ⬆️
.../main/java/org/tikv/common/policy/RetryPolicy.java 94.44% <0.00%> (+5.55%) ⬆️
src/main/java/org/tikv/common/region/TiStore.java 74.35% <0.00%> (+12.82%) ⬆️
...java/org/tikv/common/operation/PDErrorHandler.java 60.00% <0.00%> (+20.00%) ⬆️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ti-srebot
Copy link
Collaborator

@zhangyangyu, Thanks for your review. The bot only counts LGTMs from Reviewers and higher roles, but you're still welcome to leave your comments. You are not a reviewer or committer or co-leader or leader.

@sunxiaoguang
Copy link
Member

/merge

@ti-srebot
Copy link
Collaborator

Your auto merge job has been accepted, waiting for:

  • 664

@ti-srebot
Copy link
Collaborator

/run-all-tests

@sunxiaoguang
Copy link
Member

/merge

@ti-srebot
Copy link
Collaborator

Your auto merge job has been accepted, waiting for:

  • 681

@ti-srebot
Copy link
Collaborator

/run-all-tests

@zhangyangyu zhangyangyu merged commit 1554ae5 into tikv:master Dec 20, 2022
ti-srebot pushed a commit to ti-srebot/client-java that referenced this pull request Dec 20, 2022
@ti-srebot
Copy link
Collaborator

cherry pick to release-3.3 in PR #690

shiyuhang0 added a commit to shiyuhang0/client-java that referenced this pull request Dec 30, 2022
iosmanthus pushed a commit to iosmanthus/client-java that referenced this pull request Dec 30, 2022
Signed-off-by: shiyuhang <[email protected]>
Signed-off-by: iosmanthus <[email protected]>
shiyuhang0 pushed a commit to shiyuhang0/client-java that referenced this pull request Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Column is case sensitive
5 participants