Skip to content

Remove deprecation warning #829

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Aug 15, 2021
Merged

Remove deprecation warning #829

merged 2 commits into from
Aug 15, 2021

Conversation

alonre24
Copy link
Collaborator

Remove the deprecation warning logging in MODELRUN command - for HPE.

@alonre24 alonre24 requested a review from DvirDukhan August 12, 2021 15:50
@codecov
Copy link

codecov bot commented Aug 12, 2021

Codecov Report

Merging #829 (2d40882) into master (3cf1354) will decrease coverage by 0.01%.
The diff coverage is 90.90%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #829      +/-   ##
==========================================
- Coverage   79.99%   79.97%   -0.02%     
==========================================
  Files          53       53              
  Lines        8032     8011      -21     
==========================================
- Hits         6425     6407      -18     
+ Misses       1607     1604       -3     
Impacted Files Coverage Δ
src/redisai.c 87.25% <ø> (+0.25%) ⬆️
src/execution/DAG/dag.c 91.73% <88.88%> (+1.47%) ⬆️
src/execution/parsing/dag_parser.c 96.01% <100.00%> (+0.01%) ⬆️
src/execution/parsing/deprecated.c 79.70% <100.00%> (-1.69%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db9857c...2d40882. Read the comment docs.

@alonre24 alonre24 merged commit b74cba7 into master Aug 15, 2021
@alonre24 alonre24 deleted the remove_deprecation_warning branch August 15, 2021 07:38
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants