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/06/24 08:29:38 UTC

[GitHub] [incubator-linkis] barry8023 opened a new pull request, #2338: Dev 1.2.0 issue 2321

barry8023 opened a new pull request, #2338:
URL: https://github.com/apache/incubator-linkis/pull/2338

   fix(linkis-engineplugin-hive): support cancel hive on tez task and engineconn is no longer busy
   
   1.HiveEngineConfiguration中增加wds.linkis.hive.engine.type配置来控制cancel时的操作,目前支持mr和tez
   2.HiveEngineConnExecutor中killtask时对于hive on tez进行了close driver的操作,确保engineconn不会卡在busy状态
   3.properties里增加wds.linkis.hive.engine.type选项,默认值为mr
   1.Add wds.linkis.hive.engine.type configuration to HiveEngineConfiguration to control the cancel operation, currently supports mr and tez
   2.When killing task in HiveEngineConnExecutor, close driver is performed on hive on tez to ensure that engineconn will not be stuck in busy state
   3.Add the wds.linkis.hive.engine.type option to the properties, the default value is mr
   
   Closes https://github.com/apache/incubator-linkis/issues/2321


-- 
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 #2338: Dev 1.2.0 issue 2321

Posted by GitBox <gi...@apache.org>.
codecov[bot] commented on PR #2338:
URL: https://github.com/apache/incubator-linkis/pull/2338#issuecomment-1165361435

   # [Codecov](https://codecov.io/gh/apache/incubator-linkis/pull/2338?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 [#2338](https://codecov.io/gh/apache/incubator-linkis/pull/2338?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (23cd466) into [dev-1.2.0](https://codecov.io/gh/apache/incubator-linkis/commit/b2f1efbfcf304f04f6990e24e25923825147ec03?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (b2f1efb) will **decrease** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@               Coverage Diff               @@
   ##             dev-1.2.0    #2338      +/-   ##
   ===============================================
   - Coverage        22.44%   22.44%   -0.01%     
     Complexity        1060     1060              
   ===============================================
     Files              455      455              
     Lines            13963    13966       +3     
     Branches          2200     2200              
   ===============================================
     Hits              3134     3134              
   - Misses           10391    10394       +3     
     Partials           438      438              
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-linkis/pull/2338?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...n/scala/org/apache/linkis/common/utils/Utils.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vdXRpbHMvVXRpbHMuc2NhbGE=) | `6.75% <0.00%> (-0.10%)` | :arrow_down: |
   | [...scala/org/apache/linkis/common/utils/JavaLog.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vdXRpbHMvSmF2YUxvZy5zY2FsYQ==) | `0.00% <0.00%> (ø)` | |
   | [...scala/org/apache/linkis/common/utils/Logging.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vdXRpbHMvTG9nZ2luZy5zY2FsYQ==) | `23.07% <0.00%> (ø)` | |
   | [...ala/org/apache/linkis/common/ServiceInstance.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vU2VydmljZUluc3RhbmNlLnNjYWxh) | `0.00% <0.00%> (ø)` | |
   | [...ala/org/apache/linkis/common/utils/LDAPUtils.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vdXRpbHMvTERBUFV0aWxzLnNjYWxh) | `0.00% <0.00%> (ø)` | |
   | [...org/apache/linkis/common/utils/VariableUtils.scala](https://codecov.io/gh/apache/incubator-linkis/pull/2338/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-bGlua2lzLWNvbW1vbnMvbGlua2lzLWNvbW1vbi9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2xpbmtpcy9jb21tb24vdXRpbHMvVmFyaWFibGVVdGlscy5zY2FsYQ==) | `60.11% <0.00%> (ø)` | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-linkis/pull/2338?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-linkis/pull/2338?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [b2f1efb...23cd466](https://codecov.io/gh/apache/incubator-linkis/pull/2338?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?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] peacewong merged pull request #2338: Dev 1.2.0 issue 2321

Posted by GitBox <gi...@apache.org>.
peacewong merged PR #2338:
URL: https://github.com/apache/incubator-linkis/pull/2338


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