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

Resolution for previous concurrency issue #802

Merged

Conversation

noCharger
Copy link
Collaborator

@noCharger noCharger commented Oct 23, 2024

Description

Resolution for previous concurrency issue

Check List

  • Implemented unit tests
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@noCharger noCharger force-pushed the enable-heartbeat-after-job-start branch 2 times, most recently from 2a389e7 to f35a78c Compare October 23, 2024 22:03
@noCharger noCharger force-pushed the enable-heartbeat-after-job-start branch from f35a78c to 4dcc1ea Compare October 24, 2024 17:39
@noCharger noCharger marked this pull request as ready for review October 24, 2024 17:39
@noCharger noCharger requested a review from dai-chen October 24, 2024 17:40
Copy link
Collaborator

@penghuo penghuo left a comment

Choose a reason for hiding this comment

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

Please add IT in following PR.

@noCharger noCharger merged commit 54f4fa7 into opensearch-project:main Oct 24, 2024
4 checks passed
@noCharger noCharger self-assigned this Oct 24, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 24, 2024
* Resolution for previous concurrency issue

Signed-off-by: Louis Chu <[email protected]>

* resolve comments

Signed-off-by: Louis Chu <[email protected]>

---------

Signed-off-by: Louis Chu <[email protected]>
(cherry picked from commit 54f4fa7)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 24, 2024
* Resolution for previous concurrency issue

Signed-off-by: Louis Chu <[email protected]>

* resolve comments

Signed-off-by: Louis Chu <[email protected]>

---------

Signed-off-by: Louis Chu <[email protected]>
(cherry picked from commit 54f4fa7)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@noCharger
Copy link
Collaborator Author

Please add IT in following PR.

Tracking on #810

noCharger pushed a commit that referenced this pull request Oct 24, 2024
* Resolution for previous concurrency issue



* resolve comments



---------


(cherry picked from commit 54f4fa7)

Signed-off-by: Louis Chu <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
seankao-az pushed a commit that referenced this pull request Oct 30, 2024
* Resolution for previous concurrency issue



* resolve comments



---------


(cherry picked from commit 54f4fa7)

Signed-off-by: Louis Chu <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
kenrickyap pushed a commit to Bit-Quill/opensearch-spark that referenced this pull request Dec 11, 2024
* Resolution for previous concurrency issue

Signed-off-by: Louis Chu <[email protected]>

* resolve comments

Signed-off-by: Louis Chu <[email protected]>

---------

Signed-off-by: Louis Chu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants