davidradl commented on code in PR #27490:
URL: https://github.com/apache/flink/pull/27490#discussion_r2747092921


##########
docs/content/docs/dev/table/sql/create.md:
##########
@@ -951,4 +951,19 @@ WITH (
 );
 ```
 
+```sql
+CREATE MODEL triton_text_classifier
+INPUT (input STRING COMMENT 'Input text for classification')
+OUTPUT (output STRING COMMENT 'Classification result')
+COMMENT 'A Triton-based text classification model'
+WITH (
+    'provider' = 'triton',
+    'endpoint' = 'http://localhost:8000/v2/models',
+    'model-name' = 'text-classification',
+    'model-version' = '1',
+    'timeout' = '10000',

Review Comment:
   thanks for your response. This means that for short lived bearer tokens , 
the tables need to redefined with a new header definition on token expiry, 
which is not great. For api keys normally the client libraries obtain and use 
the bearer tokens under the covers request new ones prior to expiry. We do this 
in the [HTTP 
connector](https://github.com/apache/flink-connector-http/blob/main/flink-connector-http/src/main/java/org/apache/flink/connector/http/auth/OidcAccessTokenManager.java).
   
   For these reasons  I assume API keys will not work as is.



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