KarmaGYZ commented on a change in pull request #11920:
URL: https://github.com/apache/flink/pull/11920#discussion_r426203278
##########
File path: flink-dist/src/main/assemblies/opt.xml
##########
@@ -75,6 +75,28 @@
<fileMode>0644</fileMode>
</file>
+ <!-- External Resource -->
+ <file>
+
<source>../flink-external-resources/flink-external-resource-gpu/target/flink-external-resource-gpu-${project.version}.jar</source>
+
<outputDirectory>opt/external-resource-gpu/</outputDirectory>
+
<destName>flink-external-resource-gpu-${project.version}.jar</destName>
+ <fileMode>0644</fileMode>
+ </file>
+
+ <file>
+
<source>../flink-external-resources/flink-external-resource-gpu/src/main/resources/gpu-discovery-common.sh</source>
+
<outputDirectory>opt/external-resource-gpu/</outputDirectory>
+ <destName>gpu-discovery-common.sh</destName>
+ <fileMode>0755</fileMode>
+ </file>
+
+ <file>
+
<source>../flink-external-resources/flink-external-resource-gpu/src/main/resources/nvidia-gpu-discovery.sh</source>
+
<outputDirectory>opt/external-resource-gpu/</outputDirectory>
+ <destName>nvidia-gpu-discovery.sh</destName>
+ <fileMode>0755</fileMode>
+ </file>
Review comment:
Do you worried about the total size of flink-dist? Since this feature is
only 22 kb (while most of the metric reporter plugin are over 100 kb), I think
it would not affect the time of downloading flink-dist. I also notice that
there is a discussion about offering a slim package and a fat package. If I
understand it correctly, it would be good to just include it in the fat package.
Regarding the usability, I think it would be easier to move the
`external-resource-gpu` from 'opt' to 'plugin' than searching and downloading
it from website. WDYT?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
