You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2023/02/27 08:01:06 UTC
[kylin] 26/34: KYLIN-5449 fix diagnose tool issue
This is an automated email from the ASF dual-hosted git repository.
xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git
commit 823300b39c78f87adf80d41efc5709b07d5fd123
Author: qianhao.zhou <z....@gmail.com>
AuthorDate: Tue Jan 3 16:42:03 2023 +0800
KYLIN-5449 fix diagnose tool issue
Co-authored-by: qhzhou <qi...@kyligence.io>
---
.../main/java/org/apache/kylin/rest/controller/NBasicController.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/common-server/src/main/java/org/apache/kylin/rest/controller/NBasicController.java b/src/common-server/src/main/java/org/apache/kylin/rest/controller/NBasicController.java
index a403338984..9f7c1df5cd 100644
--- a/src/common-server/src/main/java/org/apache/kylin/rest/controller/NBasicController.java
+++ b/src/common-server/src/main/java/org/apache/kylin/rest/controller/NBasicController.java
@@ -542,7 +542,7 @@ public class NBasicController {
public void downloadFromRemoteHost(final HttpServletRequest request, String url,
HttpServletResponse servletResponse) throws IOException {
File temporaryZipFile = KylinConfigBase.getDiagFileName();
- Preconditions.checkState(temporaryZipFile.getParentFile().mkdirs(), "create temporary zip file folder failed");
+ temporaryZipFile.getParentFile().mkdirs();
Preconditions.checkState(temporaryZipFile.createNewFile(), "create temporary zip file failed");
RequestCallback requestCallback = x -> {
Collections.list(request.getHeaderNames())