wuchong commented on a change in pull request #15230:
URL: https://github.com/apache/flink/pull/15230#discussion_r598127716
##########
File path: docs/content.zh/docs/dev/table/modules.md
##########
@@ -55,25 +55,358 @@ the `ModuleFactory` interface.
A module factory defines a set of properties for configuring the module when
the SQL CLI bootstraps.
Properties are passed to a discovery service where the service tries to match
the properties to
a `ModuleFactory` and instantiate a corresponding module instance.
-
-## Namespace and Resolution Order
+## Module Lifecycle and Resolution Order
-Objects provided by modules are considered part of Flink's system (built-in)
objects; thus, they don't have any namespaces.
+A module can be loaded, enabled, disabled and unloaded. When TableEnvironment
loads a module initially, it enables the module by default. Flink supports
multiple modules and keeps track of the loading order to resolve metadata.
*E.g.*, when there are two functions of the same name residing in two modules,
Flink always resolves the function reference to the one in the 1st enabled
module.
Review comment:
> to the one in the 1st enabled module.
This sounds we always resolve function by only looking up the first module
which is incorrect.
--
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
