tidyverse / dbplyr

Database (DBI) backend for dplyr
https://dbplyr.tidyverse.org
Other
474 stars 173 forks source link

add comments to date translation to clarify non-SQL origins (months(), days() on Postgres + Snowflake) #1491

Open fh-mthomson opened 5 months ago

fh-mthomson commented 5 months ago

I found myself confused while trying to trace back date translations in dbplyr to their non-SQL origins, specifically base vs. lubridate functions:

I've proposed minor comments to hopefully prevent this confusion for future folks referencing these backend-specific translations for days() and months() (only available for Postgres and Snowflake - the latter was mirrored from the former).

cc: @fh-afrachioni