Skip to content

Feat:支持原生部署的vikingdb#84

Merged
MaojiaSheng merged 6 commits intovolcengine:mainfrom
chuanbao666:feat/vikingdb
Feb 6, 2026
Merged

Feat:支持原生部署的vikingdb#84
MaojiaSheng merged 6 commits intovolcengine:mainfrom
chuanbao666:feat/vikingdb

Conversation

@chuanbao666
Copy link
Collaborator

Summary

适配原生部署的vikingdb接口
注意:原生部署的vikingdb的数据集与索引需提前创建

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Documentation (docs)
  • Refactoring (refactor)
  • Other

Testing

Describe how to test these changes:

  • Unit tests pass
  • Manual testing completed

Checklist

  • Code follows project style guidelines
  • Tests added for new functionality
  • Documentation updated (if needed)
  • All tests pass

chuanbao666 and others added 6 commits February 5, 2026 21:12
…atibility issues (volcengine#73)

* refactor: Refactor S3 configuration structure and fix Python 3.9 compatibility issues
- Extract S3-related configurations from AGFSConfig into a new S3Config class
- Update agfs_manager.py to use the new S3Config structure
- Modify test_config_validation.py to adapt to the new configuration structure
- Update configuration example files to reflect the new configuration structure
- Fix Python 3.9 compatibility issue in embedding_queue.py by replacing `EmbeddingMsg | None` with `Optional[EmbeddingMsg]`

* fix: validate_config validate error
- Create centralized run_async() in openviking/utils/async_utils.py
- Remove duplicate _run_async from session.py
- Remove duplicate run_coroutine_sync from observers/async_utils.py
- Replace asyncio.run() calls in sync_client.py with run_async
- Update observers to use the unified run_async utility
- Update tests and documentation for new observer API
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ baojun-zhang
✅ chuanbao666
✅ kkkwjx07
❌ qin-ctx
You have signed the CLA already but the status is still pending? Let us recheck it.

from openviking.utils.logger import default_logger as logger


def get_or_create_vikingdb_project(
Copy link
Collaborator

Choose a reason for hiding this comment

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

vikingdb 这个名字太大了,最好能体现出是私有化的部署的场景

@MaojiaSheng MaojiaSheng merged commit e4251dc into volcengine:main Feb 6, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants