You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "TangSiyang2001 (via GitHub)" <gi...@apache.org> on 2023/04/29 05:53:08 UTC
[GitHub] [doris] TangSiyang2001 opened a new issue, #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
TangSiyang2001 opened a new issue, #19222:
URL: https://github.com/apache/doris/issues/19222
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/doris/issues?q=is%3Aissue) and found no similar issues.
### Description
Produce a S3 load, and run show load stmt. As seen, `cluster` is always N/A:
```
JobId: 10131
Label: lineitem_load
State: LOADING
Progress: ETL:100%; LOAD:77%
Type: BROKER
EtlInfo: NULL
TaskInfo: cluster:N/A; timeout(s):14400; max_filter_ratio:0.0
ErrorMsg: NULL
CreateTime: 2023-04-29 12:23:29
EtlStartTime: 2023-04-29 12:23:30
EtlFinishTime: 2023-04-29 12:23:30
LoadStartTime: 2023-04-29 12:23:30
LoadFinishTime: NULL
URL: NULL
JobDetails: {"Unfinished backends":{"de827555cbb2413d-a720be8937596b35":[10021]},"ScannedRows":36165536,"TaskNumber":1,"LoadBytes":6140395389,"All backends":{"de827555cbb2413d-a720be8937596b35":[10021]},"FileNumber":1,"FileSize":7895697364}
TransactionId: 3
ErrorTablets: {}
User: root
Comment:
```
### Solution
Actually, the `getCluster` method in LoadStmt, as the only access to the passed in `cluster`, is never called.
### Are you willing to submit PR?
- [ ] Yes I am willing to submit a PR!
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
--
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: commits-unsubscribe@doris.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] morningman closed issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
Posted by "morningman (via GitHub)" <gi...@apache.org>.
morningman closed issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
URL: https://github.com/apache/doris/issues/19222
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] TangSiyang2001 commented on issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
Posted by "TangSiyang2001 (via GitHub)" <gi...@apache.org>.
TangSiyang2001 commented on issue #19222:
URL: https://github.com/apache/doris/issues/19222#issuecomment-1538243884
> @TangSiyang2001 As you said there is no place to call the `cluster` method. Do we set a default value for it without "N/A", or just only use the `BY cluster` value. Looking forward to your advice
See in #19392 .
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] TangSiyang2001 commented on issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
Posted by "TangSiyang2001 (via GitHub)" <gi...@apache.org>.
TangSiyang2001 commented on issue #19222:
URL: https://github.com/apache/doris/issues/19222#issuecomment-1538040920
> @TangSiyang2001 As you said there is no place to call the `cluster` method. Do we set a default value for it without "N/A", or just only use the `BY cluster` value. Looking forward to your advice
I think we can use `By Cluster` value if it is set, otherwise use a default val like "default_cluster".
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] lvshaokang commented on issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
Posted by "lvshaokang (via GitHub)" <gi...@apache.org>.
lvshaokang commented on issue #19222:
URL: https://github.com/apache/doris/issues/19222#issuecomment-1537077280
plz assign to me, thk!
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
[GitHub] [doris] lvshaokang commented on issue #19222: [Enhancement] Make the 'cluster' in task info for LoadStmt not empty
Posted by "lvshaokang (via GitHub)" <gi...@apache.org>.
lvshaokang commented on issue #19222:
URL: https://github.com/apache/doris/issues/19222#issuecomment-1537934249
@TangSiyang2001 As you said there is no place to call the `cluster` method. Do we set a default value for it without "N/A", or just only use the `BY cluster` value. Looking forward to your advice
--
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: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org