Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

apache / arrow Public

Notifications You must be signed in to change notification settings
Fork 4k
Star 16.4k

Code
Issues 3.7k
Pull requests 349
Discussions
Actions
Projects 1
Security
Insights

Additional navigation options

Code
Issues
Pull requests
Discussions
Actions
Projects
Security
Insights

ARROW-14575: [R] Allow functions with `pkg::` prefixes#13160

Merged

nealrichardson merged 129 commits intoapache:masterapache/arrow:masterfrom

dragosmg:allow_namespacingdragosmg/arrow:allow_namespacingCopy head branch name to clipboard

Jul 15, 2022

Conversation Commits129 (129)Checks Files changed

Merged

ARROW-14575: [R] Allow functions with `pkg::` prefixes#13160
nealrichardson merged 129 commits intoapache:masterfrom
dragosmg:allow_namespacing

Commits

Commits on Jul 15, 2022

register 2 bindings (`"package_fun"` as a replacement for `package::fun` and `"fun"`)
dragosmg
committed
first try with `lubridate_as_datetime` and `lubridate_make_difftime`
dragosmg
committed
replace `::` in expressions with "_"
dragosmg
committed
remove redundant pipeline and add tests with namespacing
dragosmg
committed
no piping
dragosmg
committed
update `dplyr-funcs` unit test to reflect new naming convention
dragosmg
committed
updated un-registering logic in `register_binding()` + unit tests to reflect the new naming convention
dragosmg
committed
undo changes to `mutate` (no more syntax translation)
dragosmg
committed
update `register_binding` logic + register the utils bindings
dragosmg
committed
add the `::` binding
dragosmg
committed
replace `_` with `::` for `as_datetime` and `make_difftime`
dragosmg
committed
update `register_binding()` tests
dragosmg
committed
update comment
dragosmg
committed
moved `register_bindings_utils` in `dplyr-funcs.R` + deleted `dplyr-funcs-utils.R`
dragosmg
committed
qualifying `string()` with `arrow::` no longer needed and typos (`if64` instead of `i64`)
dragosmg
committed
replace `lubridate::ymd_hms()` with `as.POSIXct()` in the filtering tests since we do not have the `ymd_hms` binding yet
dragosmg
committed
batch1 of `nse_funcs` with `pkg::` prefix + unit tests
dragosmg
committed
batch 2 - date/time functionality + undo unary namespacing (it fails for `ceiling`)
dragosmg
committed
updated registrations and unit tests for math functions
dragosmg
committed
reverse changes in expressions and math tests
dragosmg
committed
test no longer needed since we suppor namespacing
dragosmg
committed
update names and tests for string bindings
dragosmg
committed
small datetime styling update
dragosmg
committed
renaming type bindings + unit test update
dragosmg
committed
undo before merge
dragosmg
committed
remove before merge
dragosmg
committed
clean dyplyr-funcs
dragosmg
committed
unit tests for maths functions
dragosmg
committed
`lubridate::date`
dragosmg
committed
added unit tests for `base::tolower()` and `base::toupper()`
dragosmg
committed
`lubridate::date`
dragosmg
committed
`pkg::` prefix for maths functions & `tolower` + `toupper`
dragosmg
committed
`abs` and `as.factor`
dragosmg
committed
undo `lubridate::date`
dragosmg
committed
`sign` and `stringi::stri_reverse`
dragosmg
committed
`ceiling` and `floor`
dragosmg
committed
`lubridate::date`
dragosmg
committed
remove `arrow:::` prefix
dragosmg
committed
point to the correct expression
dragosmg
committed
`base::strptime()` and `base::strftime()`
dragosmg
committed
undo
dragosmg
committed
register namespaced `fast_strptime()`
dragosmg
committed
math unary bindings
dragosmg
committed
math functions for ArrowDatum
dragosmg
committed
bump ci
dragosmg
committed
make `filter` fail
dragosmg
committed
remove unrelated change
dragosmg
committed
removed residual code
dragosmg
committed
add namepace qualified tests and remove `pkg::` prefixes from other tests
dragosmg
committed
fallback to `pkg::fun` when the binding does not exist
dragosmg
committed
use rlang's `%||%` operator
dragosmg
committed
add namespacing test with `group_by()`
dragosmg
committed
remove `tz = "UTC"` (unrelated to the scope of the PR)
dragosmg
committed
remove `pkg::` prefixes from test-dplyr-funcs-conditional.R
dragosmg
committed
remove `pkg::` prefixes from test-dplyr-funcs-string.R
dragosmg
committed
removed `pkg::` prefixes from test-dplyr-funcs-datetime.R
dragosmg
committed
remove `pkg::` prefixes from test-dplyr-funcs-type.R
dragosmg
committed
remove `pkg::` prefix from test-dplyr-mutate.R
dragosmg
committed
add namespacing support in `summarise()` + unit test
dragosmg
committed
lint + `if()` statement
dragosmg
committed
rename `name` to `unqualified_name` and `fun_name` to `qualified_name`
dragosmg
committed
change
dragosmg
committed
deal with nested function calls + unit tests for summarise with nested calls
dragosmg
committed
`all_funs()` unit tests
dragosmg
committed
lint
dragosmg
committed
cleaner test-dplyr-funcs-math.R
dragosmg
committed
clean-up `Math.ArrowDatum`
dragosmg
committed
style dplyr-funcs-datetime.R
dragosmg
committed
minor improvements
dragosmg
committed
`mutate` and `transmute` namespacing tests
dragosmg
committed
register all agg bindings with namespace
dragosmg
committed
merge conflict
dragosmg
committed
unit tests for `quantile()` with namespacing
dragosmg
committed
style
dragosmg
committed
bump ci
dragosmg
committed
solve `quantile` issue + test for `median()`
dragosmg
committed
split test and skip when RE2 not available
dragosmg
committed
minor change to test-dplyr-mutate.R
dragosmg
committed
tests for unary and binary string bindings
dragosmg
committed
removed duplicate tests
dragosmg
committed
added tests for namespaced versions of the datetime bindings
dragosmg
committed
test for `base::as.factor()`
dragosmg
committed
unit tests for conditional bindings
dragosmg
committed
unit tests for namespaced date/time bindings - part 1
dragosmg
committed
unit tests for namespaced date/time bindings - part 2
dragosmg
committed
register namespaced `sqrt()` and `exp()` bindings + add math unit tests
dragosmg
committed
unit test for namespaced `fast_strptime`
dragosmg
committed
bump ci
dragosmg
committed
bump ci
dragosmg
committed
unit tests for namespaced string bindings
dragosmg
committed
unit tests for type bindings
dragosmg
committed
style
dragosmg
committed
comment
dragosmg
committed
comment + remove duplicate test
dragosmg
committed
unit tests for namespaced aggregating bindings
dragosmg
committed
comments
dragosmg
committed
comments
dragosmg
committed
warn and test for duplicate bindings
dragosmg
committed
short paragraph in "Writing bindings"
dragosmg
committed
lint
dragosmg
committed
remove `arrow:::` from the `all_funs()` test
dragosmg
committed
typo
dragosmg
committed
simplify `stats::quantile` if statement
dragosmg
committed
keep a single `dplyr::case_when()` test
dragosmg
committed
simplify the registration of the `"::"` binding and the `register_binding()` function
dragosmg
committed
typo
dragosmg
committed
change to a `compare_dplyr_binding()` test for `dplyr::between()`
dragosmg
committed
shrink datetime unit tests
dragosmg
committed
shrink type unit tests
dragosmg
committed
shrink summarise unit tests
dragosmg
committed
lints
dragosmg
committed
shrink mutate unit tests
dragosmg
committed
shrink string unit tests
dragosmg
committed
shrink math unit tests
dragosmg
committed
removed empty line
dragosmg
committed
trim string test file
dragosmg
committed
trim conditional unit tests file
dragosmg
committed
improved comments
dragosmg
committed
remove the ability to remove a binding + typo
dragosmg
committed
removed duplicate test
dragosmg
committed
some more trimming
dragosmg
committed
simplify `wrap_hash_quantile()`
dragosmg
committed
remove duplicate helper data
dragosmg
committed
trimmed condition for binding registration
dragosmg
committed
simplify `register_binding()` further
dragosmg
committed
use `sub()` with `{+}`
dragosmg
committed
test `transmute()` defusing the dots with `stringr::str_squish()`
dragosmg
committed
lint
dragosmg
committed
add `tbl` back in
dragosmg
committed

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.