Skip to content

Conversation

ekuvardin
Copy link
Contributor

@ekuvardin ekuvardin commented Oct 10, 2025

Connected to
#455
#456

Add 4 main methods for users
executeScriptYql - execute scipt using new ScriptingProtos (Can be discussed)
executeScript - execure scipt using query
waitForScript - wait for execute script
fetchScriptResults - fetch results for script

Also call to executeScript and fetchScriptResults under the hood consists of UNARY call and can't be converted to QueryStream

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 74.21875% with 33 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.21%. Comparing base (3b6d6e6) to head (1c48a51).
⚠️ Report is 9 commits behind head on master.

Files with missing lines Patch % Lines
...java/tech/ydb/query/tools/SessionRetryContext.java 33.33% 13 Missing and 1 partial ⚠️
...src/main/java/tech/ydb/query/impl/SessionImpl.java 76.92% 5 Missing and 4 partials ⚠️
...in/java/tech/ydb/query/result/OperationResult.java 50.00% 5 Missing ⚠️
...tech/ydb/query/settings/ExecuteScriptSettings.java 84.00% 4 Missing ⚠️
...ery/src/main/java/tech/ydb/query/QuerySession.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #541      +/-   ##
============================================
+ Coverage     61.63%   62.21%   +0.58%     
- Complexity     2323     2393      +70     
============================================
  Files           329      334       +5     
  Lines         13560    13877     +317     
  Branches       1355     1385      +30     
============================================
+ Hits           8358     8634     +276     
- Misses         4564     4590      +26     
- Partials        638      653      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

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.

2 participants