Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update cast-and-convert-transact-sql.md#6774

Closed
jasonhorner wants to merge 1 commit intoMicrosoftDocs:livefrom
jasonhorner:patch-3
Closed

Update cast-and-convert-transact-sql.md#6774
jasonhorner wants to merge 1 commit intoMicrosoftDocs:livefrom
jasonhorner:patch-3

Conversation

@jasonhorner
Copy link
Copy Markdown
Contributor

added a section to the table to call out the rounding rules when casting a datetime to a time.. The rounding behavior seems to me to be undocumented or a bug. It definitely was unexpected see this post on SO for further discussion: https://stackoverflow.com/questions/69033081/t-sql-truncate-time-not-round-to-the-nearest-minute it's proably worth adding an example to the PR to demonstrate the behavior. If I can get confirmation that this behavior is by design and not a bug from the product team I will be glad to amend the PR.

added a section to the table to call out the rounding rules when casting a datetime to a time.. The rounding behavior seems to me to be undocumented or a bug. It definitely was unexpected see this post on SO for further discussion: https://stackoverflow.com/questions/69033081/t-sql-truncate-time-not-round-to-the-nearest-minute it's proably worth adding an example to the PR to demonstrate the behavior. If I can get confirmation that this behavior is by design and not a bug from the product team I will be glad to amend the PR.
@PRMerger20
Copy link
Copy Markdown
Contributor

@jasonhorner : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Sep 2, 2021
@cawrites
Copy link
Copy Markdown
Contributor

cawrites commented Sep 10, 2021

@jasonhorner I appreciate you taking the time to provide this insight. I will look into this further to determine if it is a expected action or a bug

@rothja rothja requested review from rothja and removed request for cawrites December 3, 2021 21:17
@rothja rothja assigned rothja and unassigned cawrites Dec 3, 2021
@rothja
Copy link
Copy Markdown
Collaborator

rothja commented Dec 3, 2021

@WilliamDAssafMSFT William, could you take a look at this? Thank you! #reassign:williamdassafmsft

@rothja rothja removed their request for review December 3, 2021 21:18
@WilliamDAssafMSFT
Copy link
Copy Markdown
Contributor

Hi @jasonhorner! Thanks for taking the time to make this PR and apologize for the delay in a resource properly addressing it. I don't think this is quite the intent of the table - the conversion of datetime to time, and other date/time types, isn't a matter of decimal places, but of scale and possible the style involved. That much is documented elsewhere in this same article, or should be made plain by the document capabilities of each data type.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants