lhotari commented on a change in pull request #13716:
URL: https://github.com/apache/pulsar/pull/13716#discussion_r788749779
##########
File path: pulsar-io/flume/pom.xml
##########
@@ -106,7 +125,7 @@
<dependency>
<groupId>com.google.guava</groupId>
<artifactId>guava</artifactId>
- <version>18.0</version>
+ <version>${guava.version}</version>
Review comment:
I believe that there must have been a reason why the version was pinned
to 18.0 . I doubt that the tests in pulsar-io/flume validate the full
functionality. @tuteng could you shade some light into this since you are the
original author?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
