raminqaf commented on code in PR #27994:
URL: https://github.com/apache/flink/pull/27994#discussion_r3129148460


##########
docs/content/docs/sql/reference/queries/changelog.md:
##########
@@ -61,8 +62,9 @@ SELECT * FROM FROM_CHANGELOG(
 | Parameter    | Required | Description                                        
                                                                                
                                                                                
                                                                                
                                                                                
                                                                                
                             |
 
|:-------------|:---------|:------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | `input`      | Yes      | The input table. Must be append-only.              
                                                                                
                                                                                
                                                                                
                                                                                
                                                                                
                             |
-| `op`         | No       | A `DESCRIPTOR` with a single column name for the 
operation code column. Defaults to `op`. The column must exist in the input 
table and be of type STRING. The column may be declared nullable, but a NULL 
value at runtime fails the job with a `TableRuntimeException` — every changelog 
row must carry an operation code.                                               
                                                                                
                                      |
-| `op_mapping` | No       | A `MAP<STRING, STRING>` mapping user-defined codes 
to Flink change operation names. Keys are user-defined codes (e.g., `'c'`, 
`'u'`, `'d'`), values are Flink change operation names (`INSERT`, 
`UPDATE_BEFORE`, `UPDATE_AFTER`, `DELETE`). Keys can contain comma-separated 
codes to map multiple codes to the same operation (e.g., `'c, r'`). Receiving 
an op code not present in the mapping fails the job at runtime with a 
`TableRuntimeException`. Each change operation may appear at most once across 
all entries. |
+| `op`         | No       | A `DESCRIPTOR` with a single column name for the 
operation code column. Defaults to `op`. The column must exist in the input 
table and be of type STRING. The column may be declared nullable; runtime 
behavior for `NULL` op codes is governed by `invalid_op_handling`.              
                                                                                
                                                                                
                                         |

Review Comment:
   Thanks! addressed



##########
docs/content/docs/sql/reference/queries/changelog.md:
##########
@@ -125,6 +127,21 @@ SELECT * FROM FROM_CHANGELOG(
 -- The operation column named 'operation' is used instead of 'op'
 ```
 
+#### Invalid operation code handling

Review Comment:
   Addressed



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