Skip to content

use antlr4's static library instead of dynamic library#830

Merged
wxyucs merged 1 commit into
mainfrom
make-thirdparty-antlr4-static
Jun 23, 2025
Merged

use antlr4's static library instead of dynamic library#830
wxyucs merged 1 commit into
mainfrom
make-thirdparty-antlr4-static

Conversation

@wxyucs

@wxyucs wxyucs commented Jun 23, 2025

Copy link
Copy Markdown
Collaborator
  • compile static library of the thirdparty antlr4
  • compile the gen codes with static library

- compile static library of the thirdparty antlr4
- compile the gen codes with static library

Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
@wxyucs
wxyucs requested a review from LHT129 June 23, 2025 03:40
@wxyucs wxyucs self-assigned this Jun 23, 2025
@wxyucs wxyucs added the kind/improvement Optimizations, UX polish, or minor improvements 性能优化、体验打磨或细节改良 label Jun 23, 2025
@wxyucs
wxyucs requested a review from jiaweizone as a code owner June 23, 2025 03:40
@wxyucs wxyucs added the version/0.15 1. Add Attribute Filter 2. Quantization and transformation 1. 新增 Attribute Filter 2. 量化与变换 label Jun 23, 2025

@jiaweizone jiaweizone left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@codecov

codecov Bot commented Jun 23, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

@@           Coverage Diff           @@
##             main     #830   +/-   ##
=======================================
  Coverage   91.18%   91.18%           
=======================================
  Files         225      225           
  Lines       14279    14279           
=======================================
  Hits        13021    13021           
  Misses       1258     1258           
Flag Coverage Δ
cpp 91.18% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
common 93.96% <ø> (ø)
datacell 91.57% <ø> (-0.03%) ⬇️
index 89.53% <ø> (-0.07%) ⬇️
simd 100.00% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dfff280...de13a2f. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LHT129 LHT129 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@wxyucs
wxyucs merged commit b3f0ada into main Jun 23, 2025
28 of 30 checks passed
@wxyucs
wxyucs deleted the make-thirdparty-antlr4-static branch June 23, 2025 06:22
LHT129 pushed a commit to LHT129/vsag that referenced this pull request Apr 16, 2026
- compile static library of the thirdparty antlr4
- compile the gen codes with static library

Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
LHT129 pushed a commit that referenced this pull request May 11, 2026
- compile static library of the thirdparty antlr4
- compile the gen codes with static library

Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
Signed-off-by: LHT129 <tianlan.lht@antgroup.com>
Sia-Sheerland pushed a commit to Sia-Sheerland/vsag that referenced this pull request Jun 26, 2026
- compile static library of the thirdparty antlr4
- compile the gen codes with static library

Signed-off-by: Xiangyu Wang <wxy407827@antgroup.com>
Signed-off-by: Sia Sheerland <x1075956441x@163.com>

Signed-off-by: Sia Sheerland <x1075956441x@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/improvement Optimizations, UX polish, or minor improvements 性能优化、体验打磨或细节改良 size/S version/0.15 1. Add Attribute Filter 2. Quantization and transformation 1. 新增 Attribute Filter 2. 量化与变换

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants