Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-25 Thread via GitHub
yashmayya merged PR #15523: URL: https://github.com/apache/pinot/pull/15523 -- 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-unsubscr...@pinot.ap

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2057525988 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,211 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
yashmayya commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2057524805 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,211 @@ +/** + * Licensed to the Apache Software Foundation (AS

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2057507548 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/ServerQueryExecutorV1Impl.java: ## @@ -199,128 +191,37 @@ private InstanceResponseBlock executeInternal

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2057492572 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/ServerQueryExecutorV1Impl.java: ## @@ -199,128 +190,36 @@ private InstanceResponseBlock executeInternal

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056548535 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,197 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
Jackie-Jiang commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056686085 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/ServerQueryExecutorV1Impl.java: ## @@ -199,128 +190,36 @@ private InstanceResponseBlock executeIn

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056539639 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,197 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056543692 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,197 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056542955 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/TableExecutionInfo.java: ## @@ -0,0 +1,197 @@ +/** + * Licensed to the Apache Software Foundation (ASF)

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
vrajat commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2056520779 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/ServerQueryExecutorV1Impl.java: ## @@ -199,128 +191,37 @@ private InstanceResponseBlock executeInternal

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-23 Thread via GitHub
yashmayya commented on code in PR #15523: URL: https://github.com/apache/pinot/pull/15523#discussion_r2055953278 ## pinot-core/src/main/java/org/apache/pinot/core/query/executor/ServerQueryExecutorV1Impl.java: ## @@ -199,128 +191,37 @@ private InstanceResponseBlock executeInter

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-21 Thread via GitHub
Jackie-Jiang commented on PR #15523: URL: https://github.com/apache/pinot/pull/15523#issuecomment-2819693260 @gortiz @yashmayya Can you help review this? -- 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

Re: [PR] Move Query Execution Planning to TableExecutionInfo in SSE [pinot]

2025-04-17 Thread via GitHub
vrajat commented on PR #15523: URL: https://github.com/apache/pinot/pull/15523#issuecomment-2813314137 Test failure is unrelated and seems like a flaky test. It passed locally. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a