Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22161#discussion_r212168906
  
    --- Diff: R/pkg/tests/fulltests/test_sparkSQL.R ---
    @@ -3613,11 +3613,11 @@ test_that("Collect on DataFrame when NAs exists at 
the top of a timestamp column
     test_that("catalog APIs, currentDatabase, setCurrentDatabase, 
listDatabases", {
       expect_equal(currentDatabase(), "default")
       expect_error(setCurrentDatabase("default"), NA)
    -  expect_error(setCurrentDatabase("foo"),
    -               "Error in setCurrentDatabase : analysis error - Database 
'foo' does not exist")
    +  expect_error(setCurrentDatabase("zxwtyswklpf"),
    +        "Error in setCurrentDatabase : analysis error - Database 
'zxwtyswklpf' does not exist")
    --- End diff --
    
    Not a big deal but can we fix this style too?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to