You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@linkis.apache.org by GitBox <gi...@apache.org> on 2022/08/21 16:51:57 UTC
[GitHub] [incubator-linkis] yyuser5201314 opened a new pull request, #2916: EnginePlugin managment module (#2914)
yyuser5201314 opened a new pull request, #2916:
URL: https://github.com/apache/incubator-linkis/pull/2916
Add EnginePlugin managment module (#2914)
Related issues: #2914
--
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@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org
[GitHub] [incubator-linkis] casionone merged pull request #2916: EnginePlugin managment module (#2914)
Posted by GitBox <gi...@apache.org>.
casionone merged PR #2916:
URL: https://github.com/apache/incubator-linkis/pull/2916
--
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@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org
[GitHub] [incubator-linkis] codecov[bot] commented on pull request #2916: EnginePlugin managment module (#2914)
Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on PR #2916:
URL: https://github.com/apache/incubator-linkis/pull/2916#issuecomment-1221588162
# [Codecov](https://codecov.io/gh/apache/incubator-linkis/pull/2916?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#2916](https://codecov.io/gh/apache/incubator-linkis/pull/2916?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (fc7b3dd) into [dev-1.3.0](https://codecov.io/gh/apache/incubator-linkis/commit/9e279fb72d2c5bcd5ae793e3e69259b7ba0ff095?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (9e279fb) will **increase** coverage by `0.00%`.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## dev-1.3.0 #2916 +/- ##
============================================
Coverage 14.59% 14.60%
- Complexity 1177 1178 +1
============================================
Files 776 776
Lines 25222 25222
Branches 3567 3567
============================================
+ Hits 3682 3684 +2
+ Misses 21047 21045 -2
Partials 493 493
```
| [Impacted Files](https://codecov.io/gh/apache/incubator-linkis/pull/2916?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [...apache/linkis/scheduler/queue/LoopArrayQueue.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2916/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-bGlua2lzLWNvbW1vbnMvbGlua2lzLXNjaGVkdWxlci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9zY2hlZHVsZXIvcXVldWUvTG9vcEFycmF5UXVldWUuc2NhbGE=) | `32.72% <0.00%> (+1.81%)` | :arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
--
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@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org
[GitHub] [incubator-linkis] casionone commented on pull request #2916: EnginePlugin managment module (#2914)
Posted by GitBox <gi...@apache.org>.
casionone commented on PR #2916:
URL: https://github.com/apache/incubator-linkis/pull/2916#issuecomment-1221774720
It is recommended to use a unified PageInfo for classification
The restful api interface needs to add swagger annotations
Unit tests need to be added later
Follow-up needs to be supplemented with related interfaces and usage documents
------
建议使用统一的PageInfo 做分类
restful api接口需要添加swagger注解说明
后续需要增加单元测试
后续需要补充相关的接口和使用文档
--
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@linkis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@linkis.apache.org
For additional commands, e-mail: notifications-help@linkis.apache.org