-
Notifications
You must be signed in to change notification settings - Fork 312
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
Update version check logic on the worker #718
Conversation
// is a breaking change on the UDF contract. | ||
internal const string CurrentVersion = "0.9.0"; | ||
// is a breaking change (major version change) on the UDF contract. | ||
internal const string CurrentVersion = "1.0.0"; |
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.
Is this version going to be tightly coupled with the Microsoft.Spark assembly version? If so then can we just get the version of Microsoft.Spark loaded in the the current AppDomain
?
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.
Yes, good point. Now that we are going with semantic versioning, we don't need this. I will rework on this after #715 is in.
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.
LGTM. Thanks @imback82 !
From 1.0.0, we enforce semantic versioning, meaning the worker will be compatible only within the same major version.
This PR changes the behavior of the version checking logic on the worker, following the semantic versioning.