-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
be conservative when reading from fresh created column #31436
Conversation
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
e6a67c1
to
f3668f2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 makes sense, but please fix the indents and make them consistent
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
the odd indents turned out to be a Microsoft Github presentation bug |
/backport to stable23 |
/backport to stable22 |
/backport to stable21 |
/backport to stable20 |
The backport to stable21 failed. Please do this backport manually. |
The backport to stable20 failed. Please do this backport manually. |
backports to 21 and 20 done manually ^ |
When a column was added into a table definition we may run into risk that when it is read it may happen on a DB node where the column is not synced to, yet. Then the where clause is also not necessary since all rows qualify.