Skip to content
New issue

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

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

Already on GitHub? # to your account

enhance: Add json key inverted index in stats for optimization #38039

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JsDove
Copy link
Contributor

@JsDove JsDove commented Nov 27, 2024

Add json key inverted index in stats for optimization
#36995

@sre-ci-robot sre-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines. label Nov 27, 2024
@mergify mergify bot added the dco-passed DCO check passed. label Nov 27, 2024
Copy link
Contributor

mergify bot commented Nov 27, 2024

@JsDove

Invalid PR Title Format Detected

Your PR submission does not adhere to our required standards. To ensure clarity and consistency, please meet the following criteria:

  1. Title Format: The PR title must begin with one of these prefixes:
  • feat: for introducing a new feature.
  • fix: for bug fixes.
  • enhance: for improvements to existing functionality.
  • test: for add tests to existing functionality.
  • doc: for modifying documentation.
  • auto: for the pull request from bot.
  1. Description Requirement: The PR must include a non-empty description, detailing the changes and their impact.

Required Title Structure:

[Type]: [Description of the PR]

Where Type is one of feat, fix, enhance, test or doc.

Example:

enhance: improve search performance significantly 

Please review and update your PR to comply with these guidelines.

@JsDove JsDove changed the title Enhancement: Add json key inverted index in stats for optimization enhance: Add json key inverted index in stats for optimization Nov 27, 2024
@mergify mergify bot added kind/enhancement Issues or changes related to enhancement and removed do-not-merge/invalid-pr-format labels Nov 27, 2024
Copy link
Contributor

mergify bot commented Nov 27, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Nov 27, 2024

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

Copy link
Contributor

mergify bot commented Nov 27, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Nov 27, 2024

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 2, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 2, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

1 similar comment
Copy link
Contributor

mergify bot commented Dec 3, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 3, 2024

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 3, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 10, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 10, 2024

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 10, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 10, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 10, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 13, 2024

@JsDove cpp-unit-test check failed, comment rerun cpp-unit-test can trigger the job again.

Copy link
Contributor

mergify bot commented Dec 13, 2024

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Feb 14, 2025

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

sre-ci-robot pushed a commit that referenced this pull request Feb 16, 2025
…39876)

Add json key inverted index in stats for optimization
issue: #36995
pr: #38039

---------

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
Co-authored-by: luzhang <luzhang@zilliz.com>
@JsDove JsDove force-pushed the optimization_json branch 2 times, most recently from 0a41326 to 2c09b04 Compare February 17, 2025 08:06
Copy link
Contributor

mergify bot commented Feb 17, 2025

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Feb 18, 2025

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

@JsDove
Copy link
Contributor Author

JsDove commented Feb 18, 2025

rerun go-sdk

Copy link
Contributor

mergify bot commented Feb 18, 2025

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

Copy link
Contributor

mergify bot commented Feb 18, 2025

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

sre-ci-robot pushed a commit that referenced this pull request Feb 19, 2025
…9972)

fix: exists expr on nested dictionaries fail in JSONkeyIndex
issue: #39963
pr: #38039

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
sre-ci-robot pushed a commit that referenced this pull request Feb 19, 2025
enhance: compress jsonstatslog path only save filename
issue: #36995
pr: #38039

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
luzhang and others added 4 commits February 19, 2025 16:10
Signed-off-by: luzhang <luzhang@zilliz.com>
Co-authored-by: luzhang <luzhang@zilliz.com>

jsoncontainexpr unaryexpr binaryexpr json index optimization

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve jsonContainSExpr

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

add jsonindex test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

support load json index after loadsegment

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve statschecker unittest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

jsonindex expr code format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix go format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix controllerbasetest fail

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonindex memeroy leak

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonkey go format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonindex go codeformat

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve jsoninvert unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

delete unuse code

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

refine test_json_key_index

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix cpp unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

delete loginfo

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix codeformat

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix createindex again hang up

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix createindex hang

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix triggerstatstask go ut

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonindex filter error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonindex filter error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve jsonkey unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

remove sealsegment chunknum assert

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix codereview

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonindex parse string to int

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix go test error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

improve

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

adjust jsonkey tantivy buffer size

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix: nested jsonarray cause incorrect index

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix: nested jsonarray cause incorrect index

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

feat growingsegment jsonkey index

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

feat growingsegment jsonkey index

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonkeyindex array contain

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonkeyindex arrayindex filter error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonkeyindex arrayindex filter error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

adjust createjsonindex trigger time

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

Revert "adjust createjsonindex trigger time"

This reverts commit 84ea08a.

fix jsoncontain

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsoncontain

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

growing jsonkey index no support offsetinput

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix yaml format

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

add jsonkeyindex unitest

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix e2e jsoncontain

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

bug fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

bug fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix go ut

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix ut

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

growing json key fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

remova skip

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

move upper to for {} outside

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

crashfix: concurrentvector json use view_element

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

rename variable

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

add json key stats enabled in yaml

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix complie error

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

modiy

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix jsonkey index read-write confict

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

revert jobmanager

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

revert

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

add growingjson index switch

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix codecheck

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix code check

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix lock

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix growing bitset out of size

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

remove mutx

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix gosdk test

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

replace interface

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

modify json key tantivy memory size

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>

fix

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
Copy link
Contributor

mergify bot commented Feb 19, 2025

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

@JsDove
Copy link
Contributor Author

JsDove commented Feb 19, 2025

rerun go-sdk

Copy link
Contributor

mergify bot commented Feb 19, 2025

@JsDove E2e jenkins job failed, comment /run-cpu-e2e can trigger the job again.

Copy link
Contributor

mergify bot commented Feb 19, 2025

@JsDove go-sdk check failed, comment rerun go-sdk can trigger the job again.

czs007 added a commit that referenced this pull request Feb 19, 2025
pr: #38039

Signed-off-by: zhenshan.cao <zhenshan.cao@zilliz.com>
sre-ci-robot pushed a commit that referenced this pull request Feb 22, 2025
Replace the position of EnabledJSONKeyStats
issue: #36995
pr: #38039

---------

Signed-off-by: Xianhui.Lin <xianhui.lin@zilliz.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area/compilation area/internal-api area/test dco-passed DCO check passed. kind/enhancement Issues or changes related to enhancement sig/testing size/XXL Denotes a PR that changes 1000+ lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants