Skip to content

Commit

Permalink
ORC-1853: Rename class TesScanData to TestScanData
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

This PR aims to rename class `TesScanData` to `TestScanData`.

### Why are the changes needed?

This is a typo when we added this class.
- #1833

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #2132 from dongjoon-hyun/ORC-1853.

Authored-by: Dongjoon Hyun <dongjoon@apache.org>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
(cherry picked from commit 47ac473)
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
  • Loading branch information
dongjoon-hyun committed Feb 20, 2025
1 parent 84a32a9 commit da05d85
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@

import static org.junit.jupiter.api.Assertions.assertTrue;

public class TesScanData {
public class TestScanData {
private Path workDir = new Path(System.getProperty("test.tmp.dir"));
private Configuration conf;
private FileSystem fs;
Expand All @@ -47,7 +47,7 @@ public void openFileSystem() throws Exception {
conf = new Configuration();
fs = FileSystem.getLocal(conf);
fs.setWorkingDirectory(workDir);
testFilePath = new Path("TesScanData.testScan.orc");
testFilePath = new Path("TestScanData.testScan.orc");
fs.delete(testFilePath, false);
}

Expand Down Expand Up @@ -86,6 +86,6 @@ public void testScan() throws Exception {
assertTrue(output.contains("{\"category\": \"struct\", \"id\": 0, \"max\": 2, \"fields\": [\n" +
"{ \"x\": {\"category\": \"int\", \"id\": 1, \"max\": 1}},\n" +
"{ \"y\": {\"category\": \"string\", \"id\": 2, \"max\": 2}}]}"));
assertTrue(output.contains("File: TesScanData.testScan.orc, bad batches: 0, rows: 10000/10000"));
assertTrue(output.contains("File: TestScanData.testScan.orc, bad batches: 0, rows: 10000/10000"));
}
}

0 comments on commit da05d85

Please sign in to comment.