Skip to content

Commit 4b1ecca

Browse files
authored
GH-48690: [R] Make "Can read Parquet files from a URL" less flaky (#48693)
### Rationale for this change The /raw/ path redirects to raw.githubusercontent.com and is arguably reliable (or rather recommended by GitHub itself) than blob/...?raw=true Reference: https://developer.github.com/changes/24/ All other URL tests in this codebase already use /raw/ format (test-io.R, test-csv.R, etc.) Disclaimer: I found this flakiness specifically in this test (with the non-raw link) that happens few times while other tests did not face this with raw links. I could not find the official claim related to it so it's based on my observation. ### What changes are included in this PR? This PR fixes the link of downloading the parquet file being used in the test, from `arrow/blob` to `arrow/raw`. ### Are these changes tested? No, I did not test. ### Are there any user-facing changes? No, test-only. * GitHub Issue: #48690 Lead-authored-by: Hyukjin Kwon <[email protected]> Co-authored-by: Hyukjin Kwon <[email protected]> Signed-off-by: Nic Crane <[email protected]>
1 parent 83df5f9 commit 4b1ecca

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

r/tests/testthat/test-parquet.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -477,7 +477,7 @@ test_that("Can read Parquet files from a URL", {
477477
skip_if_offline()
478478
skip_on_cran()
479479
skip_if_not_available("snappy")
480-
parquet_url <- "https://github.com/apache/arrow/blob/64f2cc7986ce672dd1a8cb268d193617a80a1653/r/inst/v0.7.1.parquet?raw=true" # nolint
480+
parquet_url <- "https://github.com/apache/arrow/raw/64f2cc7986ce672dd1a8cb268d193617a80a1653/r/inst/v0.7.1.parquet" # nolint
481481
pu <- read_parquet(parquet_url)
482482
expect_true(tibble::is_tibble(pu))
483483
expect_identical(dim(pu), c(10L, 11L))

0 commit comments

Comments
 (0)