Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
2dot5
ClickHouse
提交
e3003add
C
ClickHouse
项目概览
2dot5
/
ClickHouse
通知
3
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
C
ClickHouse
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
e3003add
编写于
2月 15, 2021
作者:
M
Maksim Kita
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
HashTable fix bug during resize with nonstandard grower
上级
4f919261
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
50 addition
and
1 deletion
+50
-1
src/Common/HashTable/HashTable.h
src/Common/HashTable/HashTable.h
+2
-1
src/Common/tests/gtest_hash_table.cpp
src/Common/tests/gtest_hash_table.cpp
+48
-0
未找到文件。
src/Common/HashTable/HashTable.h
浏览文件 @
e3003add
...
...
@@ -539,7 +539,8 @@ protected:
* after transferring all the elements from the old halves you need to [ o x ]
* process tail from the collision resolution chain immediately after it [ o x ]
*/
for
(;
!
buf
[
i
].
isZero
(
*
this
);
++
i
)
size_t
new_size
=
grower
.
bufSize
();
for
(;
i
<
new_size
&&
!
buf
[
i
].
isZero
(
*
this
);
++
i
)
{
size_t
updated_place_value
=
reinsert
(
buf
[
i
],
buf
[
i
].
getHash
(
*
this
));
...
...
src/Common/tests/gtest_hash_table.cpp
浏览文件 @
e3003add
...
...
@@ -317,3 +317,51 @@ TEST(HashTable, SerializationDeserialization)
ASSERT_EQ
(
convertToSet
(
cont
),
convertToSet
(
deserialized
));
}
}
template
<
typename
T
>
struct
IdentityHash
{
size_t
operator
()(
T
x
)
const
{
return
x
;
}
};
struct
OneElementResizeGrower
{
/// If collision resolution chains are contiguous, we can implement erase operation by moving the elements.
static
constexpr
auto
performs_linear_probing_with_single_step
=
true
;
static
constexpr
size_t
initial_count
=
1
;
size_t
bufSize
()
const
{
return
buf_size
;
}
size_t
place
(
size_t
x
)
const
{
return
x
%
buf_size
;
}
size_t
next
(
size_t
pos
)
const
{
return
(
pos
+
1
)
%
buf_size
;
}
bool
overflow
(
size_t
elems
)
const
{
return
elems
>=
buf_size
;
}
void
increaseSize
()
{
++
buf_size
;
}
void
set
(
size_t
)
{
}
void
setBufSize
(
size_t
buf_size_
)
{
buf_size
=
buf_size_
;
}
size_t
buf_size
=
initial_count
;
};
TEST
(
HashTable
,
Resize
)
{
{
/// Test edge case if after resize all cells are resized in end of buf and will take half of
/// hash table place.
using
HashSet
=
HashSet
<
int
,
IdentityHash
<
int
>
,
OneElementResizeGrower
>
;
HashSet
cont
;
cont
.
insert
(
3
);
cont
.
insert
(
1
);
std
::
set
<
int
>
expected
=
{
1
,
3
};
std
::
set
<
int
>
actual
=
convertToSet
(
cont
);
ASSERT_EQ
(
actual
,
expected
);
}
}
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录