-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DOP-11904] - add has_data, raise_if_no_data methods in DBReader #203
Conversation
7cc9591
to
fef6ba5
Compare
...strategy_integration/tests_incremental_strategy_integration/test_strategy_increment_kafka.py
Outdated
Show resolved
Hide resolved
Сделай rebase, я тесты починил |
fef6ba5
to
43eb529
Compare
bd5ca1a
to
66c6bc5
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #203 +/- ##
===========================================
+ Coverage 94.08% 94.14% +0.06%
===========================================
Files 204 204
Lines 7798 7827 +29
Branches 1385 1395 +10
===========================================
+ Hits 7337 7369 +32
+ Misses 336 334 -2
+ Partials 125 124 -1 ☔ View full report in Codecov by Sentry. |
...strategy_integration/tests_incremental_strategy_integration/test_strategy_increment_kafka.py
Show resolved
Hide resolved
779151f
to
efef9e3
Compare
Благодаря тестам обнаружилась нетривиальная проблема при вызове метода При попытке обратиться к методу Возможные решения:
Минусы: a) в таком случае эту будет также работать внутри контекста, где мы можем и не игнорировать hwm, иными словами след код будет выполнятся всегда:
б) мы можем не совпадать с ожидаемым результатом пользователя, если после
Минусы: a) документацию никто не прочтет и вызов
|
Я не очень представляю себе работу вариантов 1 и 3. |
Предлагаю еще в |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
Co-authored-by: Maxim Martynov <msmarty5@mts.ru>
for more information, see https://pre-commit.ci
Change Summary
has_data
,raise_error_if_no_data
methods toDBReader
class.Oracle
andMSSQL
get_sql_query
method to handle their specific dialect withlimit
Related issue number
Checklist
docs/changelog/next_release/<pull request or issue id>.<change type>.rst
file added describing change(see CONTRIBUTING.rst for details.)