David Knupp has posted comments on this change. (
http://gerrit.cloudera.org:8080/15796 )
Change subject: IMPALA-9649: Exclude shiro* and add to banned dependency maven
plugin
......................................................................
Patch Set 7:
> Patch Set 7:
>
> Running a gerrit-verify-dryrun-external on this. Assuming that passes, I
> think this makes sense. We are planning to get rid of Sentry, so it is ok to
> remove some Sentry specific tests.
The dryrun you started failed for some Jenkins/infra issue:
an exception which occurred:
in field com.cloudbees.groovy.cps.impl.ConstantBlock.value
in object com.cloudbees.groovy.cps.impl.ConstantBlock@19d90a01
in field com.cloudbees.groovy.cps.impl.CollectionLiteralBlock.argExps
in object com.cloudbees.groovy.cps.impl.MapBlock@30f60e7
in field com.cloudbees.groovy.cps.impl.FunctionCallBlock.argExps
in object com.cloudbees.groovy.cps.impl.FunctionCallBlock@2d57145e
in field com.cloudbees.groovy.cps.impl.CollectionLiteralBlock.argExps
in object com.cloudbees.groovy.cps.impl.ListBlock@371b11c5
Note that I did my own dryrun earlier, which had passed.
https://jenkins.impala.io/job/gerrit-code-review-checks/5906/
--
To view, visit http://gerrit.cloudera.org:8080/15796
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9f9994bf81c1d2e025a03925e8eccb147c34d66e
Gerrit-Change-Number: 15796
Gerrit-PatchSet: 7
Gerrit-Owner: David Knupp <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Dinesh Garg (430)
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Thu, 30 Apr 2020 16:40:18 +0000
Gerrit-HasComments: No
