Skip to content

builtins: add strptime/strftime builtins without experimental prefix#85756

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
rafiss:remove-experimental-func
Aug 10, 2022
Merged

builtins: add strptime/strftime builtins without experimental prefix#85756
craig[bot] merged 1 commit intocockroachdb:masterfrom
rafiss:remove-experimental-func

Conversation

@rafiss
Copy link
Copy Markdown
Collaborator

@rafiss rafiss commented Aug 8, 2022

refs #52139

These are just an alias for the existing implementation.

Release note (sql change): The strptime and strftime builtin functions
were added as aliases for experimental_strptime and
experimental_strftime.

@rafiss rafiss requested review from a team and RichardJCai August 8, 2022 17:19
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@rafiss rafiss force-pushed the remove-experimental-func branch 2 times, most recently from 11f7dc9 to ca0763c Compare August 9, 2022 15:12
These are just an alias for the existing implementation.

Release note (sql change): The strptime and strftime builtin functions
were added as aliases for experimental_strptime and
experimental_strftime.
@rafiss rafiss force-pushed the remove-experimental-func branch from ca0763c to dfc6123 Compare August 9, 2022 15:40
@rafiss
Copy link
Copy Markdown
Collaborator Author

rafiss commented Aug 10, 2022

tftr!

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 10, 2022

Build failed (retrying...):

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 10, 2022

Build succeeded:

@craig craig bot merged commit 9be7cf5 into cockroachdb:master Aug 10, 2022
@rafiss rafiss deleted the remove-experimental-func branch August 10, 2022 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants