-
Notifications
You must be signed in to change notification settings - Fork 390
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
Fix migrating between aliased and non-aliased computeds #6566
Conversation
This version is designed to be cherry-pickable to 4.x, so it doesn't change how any of our schemas are represented, it just tries to actually fully make the transition when we switch. I'm going to do a follow-up where I rip all of this out and the cost of schema changes.
(cur_base := self.scls.get_bases(schema).objects(schema)[0]) | ||
and (subj := cur_base.get_subject(schema)) | ||
and subj == self.scls.get_subject(schema) |
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.
This checks if my subject is also the subject of my first base.
So it checks that I'm haven't been rebased to a different subject?
Why is this here?
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.
This is detecting the case discussed in #6568, where an aliased computed is actually the subtype of the other pointer that it is aliasing
@elprans Could you take a look at this? |
This version is designed to be cherry-pickable to 4.x, so it doesn't change how any of our schemas are represented, it just tries to actually fully make the transition when we switch. I'm going to do a follow-up where I rip all of this out and the cost of schema changes.
This version is designed to be cherry-pickable to 4.x, so it doesn't change how any of our schemas are represented, it just tries to actually fully make the transition when we switch. I'm going to do a follow-up where I rip all of this out and the cost of schema changes.
This version is designed to be cherry-pickable to 4.x, so it doesn't change how any of our schemas are represented, it just tries to actually fully make the transition when we switch. I'm going to do a follow-up where I rip all of this out and the cost of schema changes.
This version is designed to be cherry-pickable to 4.x, so it doesn't change how any of our schemas are represented, it just tries to actually fully make the transition when we switch. I'm going to do a follow-up where I rip all of this out and the cost of schema changes.
This version is designed to be cherry-pickable to 4.x, so it doesn't
change how any of our schemas are represented, it just tries to
actually fully make the transition when we switch.
I'm going to do a follow-up where I rip all of this out and the cost
of schema changes.