You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2022/06/10 06:07:06 UTC

[GitHub] [iceberg] ymZhao1001 opened a new pull request, #5012: use 'SET sql-client.execution.result-mode = tableau' instead of 'SET …

ymZhao1001 opened a new pull request, #5012:
URL: https://github.com/apache/iceberg/pull/5012

   'SET execution.result-mode=tableau' does not work.
   use 'SET sql-client.execution.result-mode = tableau' instead of 'SET execution.result-mode=tableau';


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] ymZhao1001 commented on pull request #5012: Docs: use 'SET sql-client.execution.result-mode = tableau' instead of 'SET …

Posted by GitBox <gi...@apache.org>.
ymZhao1001 commented on PR #5012:
URL: https://github.com/apache/iceberg/pull/5012#issuecomment-1153389288

   Thank you for your reply.  @kbendick  it worked since flink 1.13 .  I will add it to the document.
   use 'SET execution.result-mode=tableau;'  at flink 1.12 or before
   u can see it at https://nightlies.apache.org/flink/flink-docs-release-1.12/dev/table/sqlClient.html
   after flink 1.13 , use 'SET sql-client.execution.result-mode = tableau'  
   u can see it at  https://nightlies.apache.org/flink/flink-docs-release-1.13/docs/dev/table/sqlclient/


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] ymZhao1001 commented on pull request #5012: Docs: use 'SET sql-client.execution.result-mode = tableau' instead of 'SET …

Posted by GitBox <gi...@apache.org>.
ymZhao1001 commented on PR #5012:
URL: https://github.com/apache/iceberg/pull/5012#issuecomment-1217509436

   > Since Flink 1.12 is no longer supported, I think we can just update this and remove the hint cc @ymZhao1001!
   > 
   > Once this is removed, this LGTM. Thanks @ymZhao1001!
   
   removed. @kbendick 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] kbendick commented on pull request #5012: Docs: use 'SET sql-client.execution.result-mode = tableau' instead of 'SET …

Posted by GitBox <gi...@apache.org>.
kbendick commented on PR #5012:
URL: https://github.com/apache/iceberg/pull/5012#issuecomment-1153314548

   Thanks @ymZhao1001! Do you know if this change occurred at a specific version (or which version you tested this with)?
   
   It would be great if you could link to specific documentation mentioning the added `sql-client` prefix, just in case we need to mention both depending on version. 🙂 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org


[GitHub] [iceberg] ymZhao1001 commented on pull request #5012: Docs: use 'SET sql-client.execution.result-mode = tableau' instead of 'SET …

Posted by GitBox <gi...@apache.org>.
ymZhao1001 commented on PR #5012:
URL: https://github.com/apache/iceberg/pull/5012#issuecomment-1167025137

   hello, @kbendick  is it necessary to mention both depending on version? because the "execution.runtime-mode" in the flink-getting-started.md only shows the nearest part. 


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org