4paradigm / 4paradigm/OpenMLDB

ut: cpp ut support option zk_host:zk_port

Đang mở
#2,149 0 bình luận 0 reaction 1 người được giao Được @vagetablechicken nhận Xem trên GitHub
enhancement test
Ngôn ngữ chính
C++
Star
1.7k
Fork
331
Merge trung bình
12 ngày 12 giờ
Pull request đã merge (30 ngày)
1

Mô tả

Cpp ut defines zk port 6181 in code. If we want to modify it, we should rebuild the binary files. It's not good for testing.
We should support:
```
./build/bin/xxx_test --port 2181
```

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

The issue mentions C++ unit tests that hardcode a ZooKeeper port. Look for test binaries in ./build/bin/ that end with _test. The change involves adding command-line argument parsing (like --port) to these test executables, likely in their main() function. Check existing test code for similar argument handling. After modifying, rebuild the test binary and verify it accepts the new port option.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
testing
Loại issue
Tính năng
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
40/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.