Skip to content

PR Fixes failing Test in Windows when attempting to locate files with reserved characters in the file path#2115

Merged
chernser merged 1 commit intoClickHouse:mainfrom
MaheshMadushan:Windows_failing_test_fix
Jan 25, 2025
Merged

PR Fixes failing Test in Windows when attempting to locate files with reserved characters in the file path#2115
chernser merged 1 commit intoClickHouse:mainfrom
MaheshMadushan:Windows_failing_test_fix

Conversation

@MaheshMadushan
Copy link
Contributor

@MaheshMadushan MaheshMadushan commented Jan 25, 2025

Summary

Throws an exception when try to find a file with reserved character in the file path on Windows OS
Test failure in Windows when attempting to locate files with reserved characters in the file path.
{C3380B60-8E88-4027-8F1E-E1B745387316}

Closes Test failure in Windows when attempting to locate files with reserved characters in the file path.

Checklist

Delete items not relevant to your PR:

@CLAassistant
Copy link

CLAassistant commented Jan 25, 2025

CLA assistant check
All committers have signed the CLA.

@MaheshMadushan MaheshMadushan force-pushed the Windows_failing_test_fix branch 3 times, most recently from 61f94c8 to cd4f22a Compare January 25, 2025 05:45
@MaheshMadushan MaheshMadushan force-pushed the Windows_failing_test_fix branch from cd4f22a to 8ce70aa Compare January 25, 2025 05:49
@MaheshMadushan MaheshMadushan changed the title Fix a failing test in windows because of reserved characters in file … Fix a failing Test in Windows when attempting to locate files with reserved characters in the file path Jan 25, 2025
@MaheshMadushan MaheshMadushan changed the title Fix a failing Test in Windows when attempting to locate files with reserved characters in the file path PR Fixes failing Test in Windows when attempting to locate files with reserved characters in the file path Jan 25, 2025
@chernser
Copy link
Contributor

Good day, @MaheshMadushan !
My respect for this contribution!

@MaheshMadushan
Copy link
Contributor Author

Hello @chernser , Are these CI failures anticipated? Shall i look into it?

@chernser
Copy link
Contributor

@MaheshMadushan no worries - just cloud tests.

@chernser chernser merged commit fdf6430 into ClickHouse:main Jan 25, 2025
24 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test failure in Windows when attempting to locate files with reserved characters in the file path.

3 participants