[GitHub] [flink] 1996fanrui commented on a diff in pull request #21447: [FLINK-30185] Provide the flame graph to the subtask level
1996fanrui commented on code in PR #21447: URL: https://github.com/apache/flink/pull/21447#discussion_r1054272283 ## flink-runtime/src/main/java/org/apache/flink/runtime/messages/ThreadInfoSample.java: ## @@ -32,25 +36,33 @@ */ public class ThreadInfoSample implements Serializable { +private final ExecutionAttemptID executionId; Review Comment: > I'd suggest to change TaskThreadInfoResponse to contain a Map. This class, as indicated by its name, is specific for a Flink task thread. Updated. -- 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-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [flink] 1996fanrui commented on a diff in pull request #21447: [FLINK-30185] Provide the flame graph to the subtask level
1996fanrui commented on code in PR #21447: URL: https://github.com/apache/flink/pull/21447#discussion_r1053913594 ## flink-runtime/src/main/java/org/apache/flink/runtime/messages/ThreadInfoSample.java: ## @@ -32,25 +36,33 @@ */ public class ThreadInfoSample implements Serializable { +private final ExecutionAttemptID executionId; Review Comment: Thanks for your quick feedback, I will refactor it asap. -- 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-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [flink] 1996fanrui commented on a diff in pull request #21447: [FLINK-30185] Provide the flame graph to the subtask level
1996fanrui commented on code in PR #21447: URL: https://github.com/apache/flink/pull/21447#discussion_r1053207231 ## flink-runtime/src/main/java/org/apache/flink/runtime/messages/ThreadInfoSample.java: ## @@ -32,25 +36,33 @@ */ public class ThreadInfoSample implements Serializable { +private final ExecutionAttemptID executionId; Review Comment: Sorry for missing this message yesterday. I added the `` to the `ThreadInfoSample` last weekend, could you help take a look first? If it isn't ok, I can refactor it according to your suggestion. -- 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-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [flink] 1996fanrui commented on a diff in pull request #21447: [FLINK-30185] Provide the flame graph to the subtask level
1996fanrui commented on code in PR #21447: URL: https://github.com/apache/flink/pull/21447#discussion_r1053207231 ## flink-runtime/src/main/java/org/apache/flink/runtime/messages/ThreadInfoSample.java: ## @@ -32,25 +36,33 @@ */ public class ThreadInfoSample implements Serializable { +private final ExecutionAttemptID executionId; Review Comment: Sorry for missing this message yesterday. I added the `` last weekend, could you help take a look first? If it isn't ok, I can refactor it according to your suggestion. -- 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-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [flink] 1996fanrui commented on a diff in pull request #21447: [FLINK-30185] Provide the flame graph to the subtask level
1996fanrui commented on code in PR #21447: URL: https://github.com/apache/flink/pull/21447#discussion_r1050345039 ## flink-runtime/src/main/java/org/apache/flink/runtime/messages/ThreadInfoSample.java: ## @@ -32,25 +36,33 @@ */ public class ThreadInfoSample implements Serializable { +private final ExecutionAttemptID executionId; Review Comment: Do you have any suggestions about it? When a TM has multiple slots, tm will return the thread result of all subtasks only once, JM needs to distinguish them, which subtask each ThreadInfoSample belongs to. How about using ` id` here? it can be any type, and not only serves for a flink execution. -- 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-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org