1u0 commented on a change in pull request #9169: [FLINK-12998][docs] Update
documentation for file systems loading as plugins
URL: https://github.com/apache/flink/pull/9169#discussion_r305269463
##########
File path: docs/ops/filesystems/index.md
##########
@@ -83,17 +101,21 @@ fs.hdfs.hadoopconf: /path/to/etc/hadoop
This registers `/path/to/etc/hadoop` as Hadoop's configuration directory and
is where Flink will look for the `core-site.xml` and `hdfs-site.xml` files.
-## Adding new File System Implementations
+## Adding a new pluggable File System implementation
-File systems are represented via the `org.apache.flink.core.fs.FileSystem`
class, which captures the ways to access and modify files and objects in that
file system.
-Implementations are discovered by Flink through Java's service abstraction,
making it easy to add new file system implementations.
Review comment:
I've removed it because it was not exact:
* it is not, that `FileSystem` classes are discovered by Java's service
loading mechanism, but `FileSystemFactory`;
* there is still reference left to Java's service loading in the `Add a
service entry` step.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
