[GitHub] [flink] twalthr commented on a diff in pull request #20840: [FLINK-29309][streaming-java] Relax allow-client-job-configurations for Table API and parameters

Mon, 19 Sep 2022 02:11:30 -0700 


twalthr commented on code in PR #20840:
URL: https://github.com/apache/flink/pull/20840#discussion_r974028170


##########
docs/layouts/shortcodes/generated/deployment_configuration.html:
##########
@@ -26,6 +20,18 @@
             <td>List&lt;String&gt;</td>
             <td>Custom JobListeners to be registered with the execution 
environment. The registered listeners cannot have constructors with 
arguments.</td>
         </tr>
+        <tr>
+            <td><h5>execution.program-config.enabled</h5></td>

Review Comment:
   This is not entirely correct. In Table API, we always call it `table 
program` throughout code and documentation. Also classes such as 
`PackagedProgram` use the term "program". 
   
   See also `ExecutionOptions`:
   ```
   /** {@link ConfigOption}s specific for a single execution of a user program. 
*/
   ```



-- 
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]

Reply via email to