You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/03/01 05:54:34 UTC
[GitHub] [accumulo] ctubbsii opened a new pull request #2536: Bump dependencies
ctubbsii opened a new pull request #2536:
URL: https://github.com/apache/accumulo/pull/2536
* Bump jackson and use newer jakarta artifact names instead of jakarta
classifiers
* Bump guava and use newer murmur3_32 method that fixes the previous
broken implementation
* Bump other deps: curator, micrometer, opentelemetry, jakartaee,
log4j2, jetty, hk2, jaxb, jersey, caffeine, auto-service, gson,
error_prone_annotations, commons-vfs2, checker-qual, jakarta.el,
hibernate-validator, jboss-logging, jline, snakeyaml
* Ensure the legacy junit version is explicitly managed, in case it gets
brought in transitively and conflicts with other dependencies
* Bump formatter-maven-plugin to recently released, and remove redundant
default configs
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on pull request #2536: Bump dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #2536:
URL: https://github.com/apache/accumulo/pull/2536#issuecomment-1055738774
> LGTM - but noticed that there were recent jenkins failures that look they could be related to these changes - or could just be jenkins.
>
> ```
> Accumulo-PR - PR #2536/head (Build #92) Unstable after 4 hr 38 min (Open)
> 1 Failed Tests:
> xml.[failed-to-read] after 0 ms
> ```
I saw that. This was not a failure in the IT, but in the failsafe plugin, because the test output XML got corrupted. I'm not sure why that happened. I reran it locally and I had a gson serialization problem because the HDFS client threw a weird exception, which seemed entirely unrelated. I have not been able to reproduce either of those failures since. Everything indicates they are unrelated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii commented on pull request #2536: Bump dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on pull request #2536:
URL: https://github.com/apache/accumulo/pull/2536#issuecomment-1055740079
Given that these seem unrelated, and the above approvals, I'm going to merge this. If there are further issues that indicate these are a problem, we can roll it back in part or in full, to address it. But I can't reproduce it now, so I think it must've been a fluke.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii merged pull request #2536: Bump dependencies
Posted by GitBox <gi...@apache.org>.
ctubbsii merged pull request #2536:
URL: https://github.com/apache/accumulo/pull/2536
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org