Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
Forever310
druid
提交
91eecfe3
D
druid
项目概览
Forever310
/
druid
与 Fork 源项目一致
从无法访问的项目Fork
通知
3
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
D
druid
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
前往新版Gitcode,体验更适合开发者的 AI 搜索 >>
提交
91eecfe3
编写于
9月 30, 2014
作者:
N
nishantmonu51
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
fix #730 - memory check throws exception if fails
fix exception handling, do not catch ProvisionException
上级
16c9a029
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
50 addition
and
5 deletion
+50
-5
common/src/main/java/io/druid/common/utils/VMUtils.java
common/src/main/java/io/druid/common/utils/VMUtils.java
+2
-2
server/src/main/java/io/druid/guice/DruidProcessingModule.java
...r/src/main/java/io/druid/guice/DruidProcessingModule.java
+0
-3
server/src/test/java/io/druid/guice/DruidProcessingModuleTest.java
...c/test/java/io/druid/guice/DruidProcessingModuleTest.java
+48
-0
未找到文件。
common/src/main/java/io/druid/common/utils/VMUtils.java
浏览文件 @
91eecfe3
...
...
@@ -42,10 +42,10 @@ public class VMUtils
throw
new
UnsupportedOperationException
(
"VM.maxDirectMemory doesn't exist, cannot do memory check."
,
e
);
}
catch
(
InvocationTargetException
e
)
{
throw
new
Runtime
Exception
(
"static method shouldn't throw this"
,
e
);
throw
new
UnsupportedOperation
Exception
(
"static method shouldn't throw this"
,
e
);
}
catch
(
IllegalAccessException
e
)
{
throw
new
Runtime
Exception
(
"public method, shouldn't throw this"
,
e
);
throw
new
UnsupportedOperation
Exception
(
"public method, shouldn't throw this"
,
e
);
}
}
}
server/src/main/java/io/druid/guice/DruidProcessingModule.java
浏览文件 @
91eecfe3
...
...
@@ -98,9 +98,6 @@ public class DruidProcessingModule implements Module
}
catch
(
UnsupportedOperationException
e
)
{
log
.
info
(
e
.
getMessage
());
}
catch
(
RuntimeException
e
)
{
log
.
warn
(
e
,
e
.
getMessage
());
}
return
new
IntermediateProcessingBufferPool
(
config
.
intermediateComputeSizeBytes
());
}
...
...
server/src/test/java/io/druid/guice/DruidProcessingModuleTest.java
0 → 100644
浏览文件 @
91eecfe3
/*
* Druid - a distributed column store.
* Copyright (C) 2012, 2013 Metamarkets Group Inc.
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*/
package
io.druid.guice
;
import
com.google.inject.ProvisionException
;
import
io.druid.query.DruidProcessingConfig
;
import
org.junit.Test
;
public
class
DruidProcessingModuleTest
{
@Test
(
expected
=
ProvisionException
.
class
)
public
void
testMemoryCheckThrowsException
()
{
DruidProcessingModule
module
=
new
DruidProcessingModule
();
module
.
getIntermediateResultsPool
(
new
DruidProcessingConfig
()
{
@Override
public
String
getFormatString
()
{
return
"test"
;
}
@Override
public
int
intermediateComputeSizeBytes
()
{
return
Integer
.
MAX_VALUE
;
}
});
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录