Martin Smith

My feedback

  1. 1 vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    This is explained here https://feedback.azure.com/forums/908035-sql-server/suggestions/34627627-support-parameter-embedding-optimisation-for-queri

    You are using a "catch all query" and relying on the parameter embedding optimisation to come up with a decent plan. This seems to be disabled for queries that assign to variables

  2. 1 vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  3. 5 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    So the execution plan that the following gives (with four seeks and four rows read) would be fine

    SELECT MAX(MaxX)
    FROM (SELECT MAX(X) AS MaxX
    FROM (SELECT TOP 1 X
    FROM T1
    WHERE X IS NULL
    UNION ALL
    SELECT TOP 1 X
    FROM T1
    WHERE X IS NOT NULL
    ORDER BY X DESC) T1
    UNION ALL
    SELECT MAX(X) AS MaxX
    FROM (SELECT TOP 1 X
    FROM T2
    WHERE X IS NULL
    UNION ALL
    SELECT TOP 1 X
    FROM T2
    WHERE X IS NOT NULL
    ORDER BY X DESC) T2) T0

    Martin Smith shared this idea  · 
  4. 1 vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    In the past when I have wanted something like that I just check CONTEXT_INFO in the trigger and exit immediately if set to some value which I have defined as meaning "bypass triggers"

  5. 1 vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    Probably you need to refresh the intellisense cache. Ctrl+Shift+R

  6. 2 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  7. 7 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  8. 3 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  9. 3 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  10. 15 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  11. 2 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  12. 4 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    I presume that it is shown as XML as a hacky way to work around the length limitations of non XML text. But the proper solution for that would be to implement a way of showing long non XML text and removing this annoying limitation.

    Martin Smith shared this idea  · 
  13. 3 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  14. 22 votes
    Vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)
    You have left! (?) (thinking…)
    2 comments  ·  Logic Apps » Designer  ·  Flag idea as inappropriate…  ·  Admin →
    An error occurred while saving the comment
    Martin Smith commented  · 

    It would also be nice if the editor gave better feedback for invalid expressions. Such as detecting errors like using double quotes as string delimiters instead of single quotes.

    Martin Smith supported this idea  · 
  15. 1 vote
    Vote
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)
    You have left! (?) (thinking…)
    0 comments  ·  Logic Apps » Connectors  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  16. 47 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    3 comments  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  17. 5 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    1 comment  ·  SQL Server » Bugs  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith supported this idea  · 
  18. 8 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →

    Upvotes: 62

    <=-=Nov 13 2006 2:23PM=-=>

    HI Tim,

    As posted by Tibor Karaszi, BOL document that “SQL Server only maintains default values for CLR objects in this catalog view; therefore, this
    column has a value of 0 for Transact-SQL objects. To view the default value of a parameter in a Transact-SQL object, query the definition column of the sys.sql_modules catalog view, or use the OBJECT_DEFINITION system function.”

    We dont store even the bit that indicating parameter is of default value in Yukon.

    Thanks for the feedback, we will look into addressing this more completely in future releases.

    Best Regards
    Andrew
    [MSFT]

    <=-=Nov 13 2006 2:56PM=-=>

    Hi Andrew, this is Aaron not Tim. If you are not gong to fix these omissions from sys.parameters, then could you at least share/publish the code that Management Studio uses to parse syscomments.text, since it is fully capable of showing…

    Martin Smith supported this idea  · 
  19. 3 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    under review  ·  0 comments  ·  SQL Server » Suggestions  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
  20. 2 votes
    Sign in
    (thinking…)
    Sign in with: Microsoft
    Signed in as (Sign out)

    We’ll send you updates on this idea

    0 comments  ·  SQL Server » Other  ·  Flag idea as inappropriate…  ·  Admin →
    Martin Smith shared this idea  · 
← Previous 1

Feedback and Knowledge Base