Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Spider algorithm #304

Merged
merged 9 commits into from
Feb 24, 2024
Merged

Add Spider algorithm #304

merged 9 commits into from
Feb 24, 2024

Conversation

vs9h
Copy link
Collaborator

@vs9h vs9h commented Nov 28, 2023 •
edited
Loading

In this pull request was added:

  • a spider algorithm - an algorithm for mining unary inclusion dependencies
  • tests for inclusion algorithms

Note: the first two commits related to the refactoring of hash-based tests are placed in a separate pull request (#303) and must be merged before this pull request.

@vs9h vs9h requested a review from polyntsov November 28, 2023 11:10
@vs9h vs9h force-pushed the spider branch 4 times, most recently from 94b8f72 to 40b5359 Compare December 7, 2023 10:11
@vs9h vs9h requested a review from Mstrutov January 8, 2024 18:06
@vs9h vs9h requested a review from slesarev-hub February 22, 2024 15:07
@vs9h vs9h requested a review from slesarev-hub February 23, 2024 15:40
slesarev-hub
slesarev-hub reviewed Feb 24, 2024
View reviewed changes
src/core/model/table/block_data.h Outdated Show resolved Hide resolved
src/core/model/table/column_domain.cpp Show resolved Hide resolved
src/core/model/table/column_domain.cpp Show resolved Hide resolved
src/core/model/table/column_domain.cpp Outdated Show resolved Hide resolved
src/core/model/table/column_domain.cpp Outdated Show resolved Hide resolved
src/core/model/table/column_domain.h Outdated Show resolved Hide resolved
src/core/model/table/column_domain.h Outdated Show resolved Hide resolved
src/core/model/table/column_domain.h Outdated Show resolved Hide resolved
src/core/model/table/column_domain.h Outdated Show resolved Hide resolved
src/core/config/mem_limit/option.cpp Show resolved Hide resolved
@vs9h vs9h requested a review from slesarev-hub February 24, 2024 12:48
@vs9h vs9h merged commit 23d212d into Desbordante:main Feb 24, 2024
20 checks passed
@vs9h vs9h deleted the spider branch February 24, 2024 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@slesarev-hub slesarev-hub slesarev-hub approved these changes

@polyntsov polyntsov Awaiting requested review from polyntsov

@Mstrutov Mstrutov Awaiting requested review from Mstrutov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vs9h @slesarev-hub