thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-10 01:25:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

add error check

Browse Source
This commit is contained in:
liuneng1994 2022-01-18 05:55:41 +00:00 committed by neng.liu
parent b4e30ad78e
commit c6e205b2ad
2 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Storages/HDFS/StorageHDFS.cpp
View File

@ -72,6 +72,9 @@ namespace
HDFSFileInfo ls;
ls.file_info = hdfsListDirectory(fs.get(), prefix_without_globs.data(), &ls.length);
if (ls.file_info == nullptr) {
throw Exception(ErrorCodes::ACCESS_DENIED, "Cannot list directory {}: {}", prefix_without_globs, String(hdfsGetLastError()));
}
Strings result;
for (int i = 0; i < ls.length; ++i)
{

4
tests/integration/test_storage_hdfs/test.py
View File

@ -361,6 +361,10 @@ def test_hdfsCluster(started_cluster):
assert actual == expected
fs.delete(dir, recursive=True)
def testHdfsDirectoryNotExist(started_cluster):
ddl ="create table HDFSStorageWithNotExistDir (id UInt32, name String, weight Float64) ENGINE = HDFS('hdfs://hdfs1:9000/dir1/empty', 'TSV')";
node1.query(ddl)
assert "Cannot list directory" in node1.query_and_get_error("select * from HDFSStorageWithNotExistDir")
if __name__ == '__main__':
cluster.start()