Skip to content

Commit

Permalink
Hive translations for window functions were no more working (#293)
Browse files Browse the repository at this point in the history
* hive translation needs to depend on base_odbc_win too

* add a NEWS bullet
  • Loading branch information
cderv authored and hadley committed May 3, 2019
1 parent eeb8949 commit 56afe40
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions NEWS.md
@@ -1,5 +1,7 @@
# dbplyr (development version)

* Window functions are now translated correctly for Hive (#293, @cderv).

# dbplyr 1.4.0

## Breaking changes
Expand Down
2 changes: 1 addition & 1 deletion R/backend-hive.R
Expand Up @@ -18,7 +18,7 @@ sql_translate_env.Hive <- function(con) {
quantile = sql_quantile("PERCENTILE"),
median = sql_median("PERCENTILE")
),
sql_translator(.parent = base_odbc_agg,
sql_translator(.parent = base_odbc_win,
var = win_aggregate("VARIANCE"),
quantile = sql_quantile("PERCENTILE", window = TRUE),
median = sql_median("PERCENTILE", window = TRUE)
Expand Down

0 comments on commit 56afe40

Please sign in to comment.