Skip to content

[improvement](test)Use show_nested_index_file to check inverted index - #38443

Merged
qidaye merged 3 commits into
apache:masterfrom
qidaye:opt_v2_cases
Aug 1, 2024
Merged

[improvement](test)Use show_nested_index_file to check inverted index#38443
qidaye merged 3 commits into
apache:masterfrom
qidaye:opt_v2_cases

Conversation

@qidaye

@qidayeqidaye commented Jul 29, 2024

Copy link
Copy Markdown
Contributor

Proposed changes

Use show_nested_index_file to check inverted index instead of calc_file_crc

@doris-robot

doris-robot commented Jul 29, 2024

Copy link
Copy Markdown

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@qidaye

qidaye commented Jul 29, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

@qidaye

qidaye commented Jul 29, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

airborne12
airborne12 previously approved these changes Jul 30, 2024
View reviewed changes

@airborne12airborne12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions

github-actionsBot commented Jul 30, 2024

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Jul 30, 2024
@github-actions

github-actionsBot commented Jul 30, 2024

Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@github-actionsgithub-actionsBot removed the approved Indicates a PR has been approved by one committer. label Jul 30, 2024
@qidaye

qidaye commented Jul 30, 2024

Copy link
Copy Markdown
ContributorAuthor

run buildall

airborne12
airborne12 approved these changes Aug 1, 2024
View reviewed changes

@airborne12airborne12 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actionsgithub-actionsBot added the approved Indicates a PR has been approved by one committer. label Aug 1, 2024
@github-actions

github-actionsBot commented Aug 1, 2024

Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

csun5285
csun5285 approved these changes Aug 1, 2024
View reviewed changes

@csun5285csun5285 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@qidaye
qidaye merged commit 004c5a3 into apache:masterAug 1, 2024
@qidaye
qidaye deleted the opt_v2_cases branch August 1, 2024 03:17
qidaye added a commit to qidaye/incubator-doris that referenced this pull request Aug 1, 2024
@qidaye
[improvement](test)Use show_nested_index_file to check inverted index (…
45a902b
…apache#38443)
Use show_nested_index_file to check inverted index instead of
calc_crc
qidaye added a commit to qidaye/incubator-doris that referenced this pull request Aug 1, 2024
@qidaye
[improvement](test)Use show_nested_index_file to check inverted index (…
6041938
…apache#38443)
Use show_nested_index_file to check inverted index instead of
calc_crc
qidaye added a commit to qidaye/incubator-doris that referenced this pull request Aug 1, 2024
@qidaye
[improvement](test)Use show_nested_index_file to check inverted index (…
e4a3497
…apache#38443)
Use show_nested_index_file to check inverted index instead of
calc_crc
@qidayeqidaye mentioned this pull request Aug 1, 2024
Merged
yiguolei pushed a commit that referenced this pull request Aug 2, 2024
@qidaye
[test](inverted index)Add cases for inverted index format v2 (#38132)(#…
f21d7e3
…38443) (#38222)
## Proposed changes
backport #38132#38443
feiniaofeiafei pushed a commit to feiniaofeiafei/doris that referenced this pull request Aug 9, 2024
@qidaye@feiniaofeiafei
[improvement](test)Use show_nested_index_file to check inverted index (…
3485850
…apache#38443)
## Proposed changes
Use show_nested_index_file to check inverted index instead of
calc_crc
qidaye added a commit to qidaye/incubator-doris that referenced this pull request Aug 12, 2024
@qidaye
[improvement](test)Use show_nested_index_file to check inverted index (…
3b00346
…apache#38443)
Use show_nested_index_file to check inverted index instead of
calc_crc
@qidayeqidaye mentioned this pull request Aug 12, 2024
Merged
xiaokang pushed a commit that referenced this pull request Aug 12, 2024
@qidaye
[improvement](test)Use show_nested_index_file to check inverted index (…
c0e6d1e
…#38443#39024#39077) (#39232)
dataroaring pushed a commit that referenced this pull request Aug 16, 2024
@qidaye@dataroaring
[improvement](test)Use show_nested_index_file to check inverted index (…
036bf34
…#38443#39024#39077) (#39232)
HappenLee pushed a commit to HappenLee/incubator-doris that referenced this pull request Apr 24, 2026
@qidaye@HappenLee
[improvement](test)Use show_nested_index_file to check inverted index (…
7b170c2
…apache#38443)
## Proposed changes
Use show_nested_index_file to check inverted index instead of
calc_crc
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xiaokangxiaokangxiaokang left review comments
@airborne12airborne12airborne12 approved these changes
@csun5285csun5285csun5285 approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.dev/2.1.6-mergeddev/3.0.2-mergedreviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@qidaye@doris-robot@xiaokang@airborne12@csun5285@yiguolei@dataroaring