Skip to content
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

fix: jobInfoManager use online mode to get the job info table #1440

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

vagetablechicken
Copy link
Collaborator

No description provided.

@vagetablechicken vagetablechicken changed the title fix: JobInfoManager use online mode to get the job info table fix: jobInfoManager use online mode to get the job info table Mar 14, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 14, 2022

Linux Test Report

     112 files  ±0       238 suites  ±0   44m 16s ⏱️ - 2m 12s
  8 627 tests ±0    8 624 ✔️ ±0  3 💤 ±0  0 ±0 
12 759 runs  ±0  12 756 ✔️ ±0  3 💤 ±0  0 ±0 

Results for commit 1db72f3. ± Comparison against base commit fe2ee26.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Mar 14, 2022

Codecov Report

Merging #1440 (1db72f3) into main (fe2ee26) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               main    #1440   +/-   ##
=========================================
  Coverage     65.76%   65.77%           
  Complexity      236      236           
=========================================
  Files           571      571           
  Lines        107349   107350    +1     
  Branches        863      872    +9     
=========================================
+ Hits          70600    70606    +6     
+ Misses        36597    36592    -5     
  Partials        152      152           
Impacted Files Coverage Δ
...paradigm/openmldb/taskmanager/JobInfoManager.scala 0.00% <0.00%> (ø)
src/sdk/sql_cluster_router.cc 42.24% <0.00%> (-0.34%) ⬇️
src/sdk/db_sdk.cc 63.63% <0.00%> (ø)
src/nameserver/name_server_impl.cc 37.74% <0.00%> (+0.04%) ⬆️
src/tablet/tablet_impl.cc 38.08% <0.00%> (+0.08%) ⬆️
src/zk/zk_client.cc 83.81% <0.00%> (+0.53%) ⬆️
src/replica/log_replicator.cc 77.85% <0.00%> (+0.65%) ⬆️
src/catalog/tablet_catalog.cc 71.25% <0.00%> (+0.89%) ⬆️

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 fe2ee26...1db72f3. Read the comment docs.

Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@tobegit3hub tobegit3hub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tobegit3hub tobegit3hub merged commit 37cefd8 into main Mar 17, 2022
@tobegit3hub tobegit3hub deleted the vagetablechicken-patch-1 branch March 17, 2022 02:39
@dl239 dl239 mentioned this pull request Mar 29, 2022
@lumianph lumianph mentioned this pull request Apr 28, 2022
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.

3 participants