Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-04-05 Thread via GitHub
szetszwo commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r2009165252 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/execution/flowcontrol/ExecutionContext.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache So

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-07 Thread via GitHub
sumitagrawl merged PR #7602: URL: https://github.com/apache/ozone/pull/7602 -- 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...@ozone.apa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-07 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1906476605 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/ExecutionContext.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-07 Thread via GitHub
kerneltime commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1905828830 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/ExecutionContext.java: ## @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundat

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-07 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1905082198 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-07 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1905082198 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-06 Thread via GitHub
adoroszlai commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1905000366 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMan

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-06 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1904980986 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-06 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1904980986 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-06 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1904980986 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-06 Thread via GitHub
adoroszlai commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1903917569 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMan

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-05 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1903633554 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMa

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-04 Thread via GitHub
adoroszlai commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1902774504 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMan

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2025-01-04 Thread via GitHub
adoroszlai commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1902774504 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/OMClientRequest.java: ## @@ -140,12 +140,15 @@ public void handleRequestFailure(OzoneMan

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2024-12-30 Thread via GitHub
sumitagrawl commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1899402286 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/ExecutionContext.java: ## @@ -0,0 +1,47 @@ +/** + * Licensed to the Apache Software Found

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2024-12-30 Thread via GitHub
sumitagrawl commented on PR #7602: URL: https://github.com/apache/ozone/pull/7602#issuecomment-2565204651 > > ExecutionContext here have just one entry, but in new flow, it’s context for execution which will contains other members as required in future like both index and termIndex. Additio

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2024-12-23 Thread via GitHub
adoroszlai commented on PR #7602: URL: https://github.com/apache/ozone/pull/7602#issuecomment-2559326805 > ExecutionContext here have just one entry, but in new flow, it’s context for execution which will contains other members as required in future like both index and termIndex. Additional

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2024-12-22 Thread via GitHub
devmadhuu commented on code in PR #7602: URL: https://github.com/apache/ozone/pull/7602#discussion_r1895351389 ## hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/ratis/ExecutionContext.java: ## @@ -0,0 +1,47 @@ +/** + * Licensed to the Apache Software Foundat

Re: [PR] HDDS-11975. wrap TermIndex in ExecutionContext [ozone]

2024-12-22 Thread via GitHub
sumitagrawl commented on PR #7602: URL: https://github.com/apache/ozone/pull/7602#issuecomment-2559045895 > Thanks @sumitagrawl for the patch. > > I think we can keep using `TermIndex` even if the index is not generated by Ratis, similar to how it's used currently when Ratis is disabl