NotificationsYou must be signed in to change notification settings
Fork47
Star200

Bind`NULL` to`NA` (logical)#156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

krlmlr wants to merge1 commit intomain

base:main

Choose a base branch

fromb-const-null

Draft

Bind`NULL` to`NA` (logical)#156

krlmlr wants to merge1 commit intomainfromb-const-null

+1 −1

Conversation

Copy link

Collaborator

krlmlr commentedMay 4, 2024•
edited
Loading

Closes#155.

We'd need to either upstream this, or patch here. Unclear about the consequences.

duckdb<- asNamespace("duckdb")drv<-duckdb::duckdb()con<-DBI::dbConnect(drv)experimental<-FALSEdf1<-tibble::tibble(a=1)rel1<-duckdb$rel_from_df(con,df1,experimental=experimental)rel2<-duckdb$rel_project(rel1,list(    {tmp_expr<-duckdb$expr_reference("a")duckdb$expr_set_alias(tmp_expr,"a")tmp_expr    },    {tmp_expr<-if ("experimental"%in% names(formals(duckdb$expr_constant))) {duckdb$expr_constant(NA,experimental=experimental)      }else {duckdb$expr_constant(NA)      }duckdb$expr_set_alias(tmp_expr,"b")tmp_expr    }  ))rel2#> DuckDB Relation:#> ---------------------#> --- Relation Tree ---#> ---------------------#> Projection [a as a, NULL as b]#>   r_dataframe_scan(0x138dfe8e8)#>#> ---------------------#> -- Result Columns  --#> ---------------------#> - a (DOUBLE)#> - b (BOOLEAN)dput(duckdb$rel_to_altrep(rel2))#> structure(list(a = 1, b = NA), row.names = c(NA, -1L), class = "data.frame")DBI::dbDisconnect(con)