C++ integration test

[leekeiabstraction]

Purpose

Linked issue: close #338

Brief change log

• Add C++ client integration tests
• Update CI to trigger CI test more selectively

[leekeiabstraction]

@fresh-borzoni Appreciate your review here 🙏

[fresh-borzoni]

@leekeiabstraction, it might be the case that we would need to get #351 merged first
I changes some C++ API to match between clients:

• Size() to Count() on ScanRecord, it matches Java/Rust
• Empty() to isEmpty() on ScanRecords, it matches Java/Rust
• we still support flat iteration: for (const auto& rec : scan_records) , but not indexing as it isn't exposed in Java/Rust, so I removed it, still exposed it in Python as it's more pythonic, but it's debatable

[leekeiabstraction]

That is fine. We can update this afterwards

[leekeiabstraction]

Still appreciate your early review on the other parts within this PR though! 🙏🏼

[fresh-borzoni]

@leekeiabstraction Ty for the PR.

quickly looked through. Left couple of comments, but still 'review in progress'
PTAL

[fresh-borzoni]

we shall remove it as it just returns empty vector in the end.
But it actually made me think - mb we should just use shared_ptr and I overcomplicated this pattern, it basically would be Arc from Rust.

prepared the fix for this: seems to work and better overall: #353

[leekeiabstraction]

Thanks, will update the test to use the latest APIs once #331 is also merged

[fresh-borzoni]

@leekeiabstraction finished review, left comments. PTAL

also once scan by buckets PR is merged - we need to add some tests for it

[luoyuxia]

@leekeiabstraction Hi, since #331 is merge, maybe you need to update the pr

[leekeiabstraction]

@fresh-borzoni Updated C++ IT to also use per-bucket scan records view. Appreciate if you can have a look. TY.