You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/04/17 14:41:30 UTC
[GitHub] [incubator-kyuubi] zhaomin1423 opened a new issue, #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
zhaomin1423 opened a new issue, #2403:
URL: https://github.com/apache/incubator-kyuubi/issues/2403
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
### Search before asking
- [X] I have searched in the [issues](https://github.com/apache/incubator-kyuubi/issues?q=is%3Aissue) and found no similar issues.
### What would you like to be improved?
move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
### How should we improve?
_No response_
### Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
--
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: notifications-unsubscribe@kyuubi.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] yaooqinn commented on issue #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
Posted by GitBox <gi...@apache.org>.
yaooqinn commented on issue #2403:
URL: https://github.com/apache/incubator-kyuubi/issues/2403#issuecomment-1106519659
+1 for code revamp
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] ulysses-you closed issue #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
Posted by GitBox <gi...@apache.org>.
ulysses-you closed issue #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
URL: https://github.com/apache/incubator-kyuubi/issues/2403
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] lightning-L commented on issue #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
Posted by GitBox <gi...@apache.org>.
lightning-L commented on issue #2403:
URL: https://github.com/apache/incubator-kyuubi/issues/2403#issuecomment-1110666078
Hi, I'd like to do this refactoring.
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] zhaomin1423 commented on issue #2403: [Improvement] move addTimeoutMonitor to AbstractOperation because it was used in multiple engines
Posted by GitBox <gi...@apache.org>.
zhaomin1423 commented on issue #2403:
URL: https://github.com/apache/incubator-kyuubi/issues/2403#issuecomment-1100892935
spark-sql-engine
<img width="791" alt="image" src="https://user-images.githubusercontent.com/49054376/163719491-e17d98bc-9785-4b57-b65f-eebf381d9b1b.png">
flink-sql-engine
<img width="784" alt="image" src="https://user-images.githubusercontent.com/49054376/163719501-5191a388-0cb9-43f3-98ff-0ca8fd9524b2.png">
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org