-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[BEAM-6869] fix dependency checking #8123
Merged
chamikaramj
merged 1 commit into
apache:master
from
yifanzou:BEAM-6869/fix_dependency_check
Mar 25, 2019
Merged
[BEAM-6869] fix dependency checking #8123
chamikaramj
merged 1 commit into
apache:master
from
yifanzou:BEAM-6869/fix_dependency_check
Mar 25, 2019
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Run Seed Job |
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 22, 2019 23:42
6427673
to
4281832
Compare
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 22, 2019 23:56
4281832
to
c0e5f04
Compare
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
2 times, most recently
from
March 23, 2019 00:09
272d664
to
e2cf06d
Compare
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 23, 2019 00:35
e2cf06d
to
abe7cb9
Compare
Run Dependency Check |
Run Seed Job |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 25, 2019 03:30
abe7cb9
to
7fb2a22
Compare
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 25, 2019 04:09
7fb2a22
to
e9c535e
Compare
Run Dependency Check |
yifanzou
force-pushed
the
BEAM-6869/fix_dependency_check
branch
from
March 25, 2019 05:01
e9c535e
to
73f09c4
Compare
yifanzou
changed the title
Do Not Merge [BEAM-6869] fix dependency checking
[BEAM-6869] fix dependency checking
Mar 25, 2019
+R: @chamikaramj |
LGTM. Thanks Yifan. |
Thanks! |
pl04351820
pushed a commit
to pl04351820/beam
that referenced
this pull request
Dec 20, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The dependency check was broken since the pyyaml yaml.load(file) was deprecated.
The dependency job passed with this PR https://builds.apache.org/view/A-D/view/Beam/view/All/job/beam_Dependency_Check/189/
Please review and merge.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username
).[BEAM-XXX] Fixes bug in ApproximateQuantiles
, where you replaceBEAM-XXX
with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.Post-Commit Tests Status (on master branch)
See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.