Comments
Sort by recent activity
Thanks again for all the research on this, Alex - what you're saying makes sense! Please, pass along my appreciation to your dev team, both in particular for their help on this issue and in general for their work on this fantastic product. / comments
Thanks again for all the research on this, Alex - what you're saying makes sense! Please, pass along my appreciation to your dev team, both in particular for their help on this issue and in genera...
Thanks Alex! I certainly understand your perspective on this, and agree that SQL Compare shouldn't be modify the code unnecessarily.
There definitely is an opportunity for a fix here, though, since legitimately-functioning code results in a deployment error when sent through the tool. I'm not necessarily pushing for a change that strips out the unnecessary qualified reference, but there's at least a known limitation of the tool that means "good but weird" code can't get deployed under certain circumstances.
/ comments
Thanks Alex! I certainly understand your perspective on this, and agree that SQL Compare shouldn't be modify the code unnecessarily.
There definitely is an opportunity for a fix here, though, sin...
Alex, thanks for your support on this.
I'm guessing, given what you just laid out, that at some point in the development process someone wrote it as a SELECT statement, just to try out the logic of the expression, and then simply copied the expression into the CREATE or ALTER TABLE script.
At this point I'd say that it's more of a development thing and, unless SQL Compare can remove aliases during the table rebuild process, the best approach for my team going forward will be to find and remove those cases!
Thanks again for your help!
/ comments
Alex, thanks for your support on this.
I'm guessing, given what you just laid out, that at some point in the development process someone wrote it as a SELECT statement, just to try out the logic ...
Hey Alex - no particular reason specific to computed columns, we usually qualify object names (and use SQL Prompt to auto-format all code to use it) in general, which means it's probably getting introduced at some point in the development process and then just carried through to the table definition.
/ comments
Hey Alex - no particular reason specific to computed columns, we usually qualify object names (and use SQL Prompt to auto-format all code to use it) in general, which means it's probably getting in...
Since the code formatter appears not to have liked my line breaks, I'm pasting the relevant script here in an unformatted block:
CREATE TABLE dbo.RG_Recovery_1_FactChecklistCompletion ( ChecklistCompletion_SK INT NOT NULL IDENTITY(1, 1) ,ChecklistItem_SK INT NOT NULL ,Person_SK INT NOT NULL ,ChecklistItemCompletionDate DATETIME2(3) NULL ,ChecklistItemCompletionDateKey AS (CONVERT(INT, CONVERT(CHAR(8), FactChecklistCompletion.ChecklistItemCompletionDate, (112)), (0))) ) GO
/ comments
Since the code formatter appears not to have liked my line breaks, I'm pasting the relevant script here in an unformatted block:
CREATE TABLE dbo.RG_Recovery_1_FactChecklistCompletion ( ...
@Alex B , that's incredibly helpful - thanks for the code snip, it's going to potentially save us a lot of trouble.
Is that something you think will get integrated into the product itself at some point?
/ comments
@Alex B , that's incredibly helpful - thanks for the code snip, it's going to potentially save us a lot of trouble.
Is that something you think will get integrated into the product itself at some ...
@Alex B , thanks for the answer!
I'm running 10.1.0.13908 at the moment, and it's still displaying the behavior described. .
Probably not relevant, but oddly enough I don't see that version number on the page you linked to - it only goes up to 9.5.23
/ comments
@Alex B , thanks for the answer!
I'm running 10.1.0.13908 at the moment, and it's still displaying the behavior described. .
Probably not relevant, but oddly enough I don't see that version numbe...
DBTL said:
dufflepud said:
Is this still not on the road map? I would imagine this is becoming more and more of an issue for people.
Certainly is for me. Without support for MFA, I cannot condone the use of Redgate products to my clients anymore.
Echoing this sentiment, it's been great to see MFA added to several other Redgate products but this one is a really useful feature that's unfortunately no longer available to us. / comments
DBTL said:
dufflepud said:
Is this still not on the road map? I would imagine this is becoming more and more of an issue for people.
Certainly is for me.Without support for MFA, I c...
Clarifying note, the error happens when I try to format a script that contains both of the above queries, when both queries contain a HAVING and FOR SYSTEM_TIME. / comments
Clarifying note, the error happens when I try to format a script that contains both of the above queries, when both queries contain a HAVING and FOR SYSTEM_TIME.
@Tianjiao_Li, thanks for your help.
I am on version 9.4.3.7121. I do have a custom format style, but I get the same error when I use the Default format style: were you testing it with both statements?
[image]
/ comments
@Tianjiao_Li, thanks for your help.
I am on version 9.4.3.7121. I do have a custom format style, but I get the same error when I use the Default format style: were you testing it with both statem...