Skip to content

Commit

Permalink
[KYUUBI #5576][Bug] Fix wrong code in test case of dir command
Browse files Browse the repository at this point in the history
### _Why are the changes needed?_
To close #5575
 Fix wrong code in test case of dir command

### _How was this patch tested?_
- [x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.readthedocs.io/en/master/contributing/code/testing.html#running-tests) locally before make a pull request

### _Was this patch authored or co-authored using generative AI tooling?_
No

Closes #5577 from AngersZhuuuu/KYUUBI-5576.

Closes #5576

60e2cb8 [Angerszhuuuu] [KYUUBI #5576][Bug] Fix wrong code in test case of dir command

Authored-by: Angerszhuuuu <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
AngersZhuuuu authored and pan3793 committed Oct 31, 2023
1 parent 9be2a9c commit 058d3ee
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1450,7 +1450,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite {
val directory = File(tableDirectory).createDirectory()
val plan = sql(
s"""
|INSERT OVERWRITE DIRECTORY '$directory.path'
|INSERT OVERWRITE DIRECTORY '${directory.path}'
|USING parquet
|SELECT * FROM $reusedPartTable""".stripMargin)
.queryExecution.analyzed
Expand Down Expand Up @@ -1574,7 +1574,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite {
val directory = File(tableDirectory).createDirectory()
val plan = sql(
s"""
|INSERT OVERWRITE DIRECTORY '$directory.path'
|INSERT OVERWRITE DIRECTORY '${directory.path}'
|USING parquet
|SELECT * FROM $reusedPartTable""".stripMargin)
.queryExecution.analyzed
Expand All @@ -1599,7 +1599,7 @@ class HiveCatalogPrivilegeBuilderSuite extends PrivilegesBuilderSuite {
val directory = File(tableDirectory).createDirectory()
val plan = sql(
s"""
|INSERT OVERWRITE DIRECTORY '$directory.path'
|INSERT OVERWRITE DIRECTORY '${directory.path}'
|ROW FORMAT DELIMITED FIELDS TERMINATED BY ','
|SELECT * FROM $reusedPartTable""".stripMargin)
.queryExecution.analyzed
Expand Down

0 comments on commit 058d3ee

Please sign in to comment.