feat: Add C ABI for embedded seekdb (multi‑language SDKs)#153
Closed
dengfuping wants to merge 49 commits intomasterfrom
Closed
feat: Add C ABI for embedded seekdb (multi‑language SDKs)#153dengfuping wants to merge 49 commits intomasterfrom
dengfuping wants to merge 49 commits intomasterfrom
Conversation
…st scripts for darwin
…. [1.1, 2.2, 3.3])
…dout to log file before set_file_name
…ix MTL_SWITCH namespace; remove §7/doc refs - C ABI: row_nulls for NULL vs empty; read_lob_data for TEXT/JSON LOB; MTL_SWITCH(OB_SYS_TENANT_ID) for ObLobManager; namespace share alias for MTL_SWITCH - test: 100KB LONGTEXT via SET SESSION ob_default_lob_inrow_threshold=262144, no table option; special chars metadata test; remove §7 and c-abi-modification-reference references from comments and error messages
- Add LOB read paths for TEXT/JSON columns in result row serialization
- Use print_sql_literal for ObJsonType cells and expand buffer to 32KB
- Add col_is_metadata heuristic: write "{}" for empty metadata in fallback paths
- Add unit tests: very long document (100KB), special chars in metadata, empty JSON metadata
…mpat, install GTest on macOS
… OB_CCACHE in workflow
- macOS: put ccache in deps devtools/bin so Env.cmake finds it without code change - Linux: install base deps (incl. wget) first so dep_create.sh can download RPMs - Linux: try EPEL+ccache; use OB_USE_CCACHE=OFF when ccache not available in container
…cOS cross-build arch - Checkout PR head ref instead of merge ref to avoid conflict markers in CI - Restore darwin-x64 job (cross-compile on Apple Silicon for Intel Mac) - Env.cmake: use CMAKE_OSX_ARCHITECTURES for ARCHITECTURE on macOS x86_64 cross-build
So next push can restore cache and avoid full rebuild from scratch.
- workflow: cd to GITHUB_WORKSPACE/package/libseekdb for Linux pack (was relative to build_release) - zlib_lite: build QPL only on x86_64 Linux (NOT APPLE); macOS deps do not ship qpl
…, bzip2) for darwin-arm64 link
…he path before adding deps/bin to PATH
…SHDIR=1), add ccache -s after build
…o feat/embedded-mode
…sh, single refresh after write)
…o feat/embedded-mode
…est_seekdb_absolute, remove in-process close+open test
…o feat/embedded-mode
…o feat/embedded-mode
…ibc-header-start.h error
…II redirect before CALL_WITH_NEW_STACK
…o feat/embedded-mode
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Task Description
Solution Description
Passed Regressions
Upgrade Compatibility
Other Information
Release Note