Skip to content

Commit 7a3dff5

Browse files
committed
perf: avoid arrange() overhead in check_ukey_unique()
1 parent f94030a commit 7a3dff5

File tree

3 files changed

+3
-1
lines changed

3 files changed

+3
-1
lines changed

NAMESPACE

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,3 +245,4 @@ importFrom(vctrs,vec_data)
245245
importFrom(vctrs,vec_duplicate_any)
246246
importFrom(vctrs,vec_equal)
247247
importFrom(vctrs,vec_size)
248+
importFrom(vctrs,vec_sort)

R/epiprocess-package.R

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
#' @importFrom tools toTitleCase
2929
#' @importFrom vctrs vec_data
3030
#' @importFrom vctrs vec_equal
31+
#' @importFrom vctrs vec_sort
3132
## usethis namespace: end
3233
NULL
3334

R/utils.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1038,7 +1038,7 @@ check_ukey_unique <- function(x, ukey_names, end_cli_message = character()) {
10381038
TRUE
10391039
} else {
10401040
# Fast check, slow error message.
1041-
arranged_ukeys <- arrange(x[ukey_names], across(all_of(ukey_names)))
1041+
arranged_ukeys <- vec_sort(x[ukey_names])
10421042
if (!any(vec_equal(arranged_ukeys[-1L, ], arranged_ukeys[-nrow(arranged_ukeys), ]))) {
10431043
TRUE
10441044
} else {

0 commit comments

Comments
 (0)