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: job execute again if exception & insert_memory_usage_limit config #3726

Merged
merged 2 commits into from
Jan 31, 2024

Conversation

vagetablechicken
Copy link
Collaborator

closes #3724 , closes #3718

@github-actions github-actions bot added batch-engine openmldb batch(offline) engine storage-engine openmldb storage engine. nameserver & tablet labels Jan 25, 2024
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b2f85fb) 40.18% compared to head (6611ca5) 74.90%.
Report is 4 commits behind head on main.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #3726       +/-   ##
=============================================
+ Coverage     40.18%   74.90%   +34.71%     
  Complexity      655      655               
=============================================
  Files           193      732      +539     
  Lines         11384   131613   +120229     
  Branches       1366     1366               
=============================================
+ Hits           4575    98588    +94013     
- Misses         6510    32726    +26216     
  Partials        299      299               

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

Copy link
Contributor

github-actions bot commented Jan 31, 2024

SDK Test Report

101 files  ±0  101 suites  ±0   2m 7s ⏱️ +2s
344 tests ±0  331 ✅ ±0  13 💤 ±0  0 ❌ ±0 
469 runs  ±0  456 ✅ ±0  13 💤 ±0  0 ❌ ±0 

Results for commit 6611ca5. ± Comparison against base commit b2f85fb.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jan 31, 2024

Linux Test Report

    57 files  ±0     244 suites  ±0   1h 18m 6s ⏱️ +56s
12 574 tests ±0  12 567 ✅ ±0  7 💤 ±0  0 ❌ ±0 
17 831 runs  ±0  17 824 ✅ ±0  7 💤 ±0  0 ❌ ±0 

Results for commit 6611ca5. ± Comparison against base commit b2f85fb.

♻️ This comment has been updated with latest results.

@vagetablechicken vagetablechicken merged commit 7d755ff into 4paradigm:main Jan 31, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
batch-engine openmldb batch(offline) engine storage-engine openmldb storage engine. nameserver & tablet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spark job execute twice if cat exception pass insert_memory_usage_limit to spark config error
3 participants