Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
KKcorps merged PR #17679: URL: https://github.com/apache/pinot/pull/17679 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2796847260
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigValidationUtils.java:
##
@@ -0,0 +1,113 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pinot.controller.api.resources;
+
+import javax.annotation.Nullable;
+import org.apache.pinot.controller.ControllerConf;
+import org.apache.pinot.controller.helix.core.PinotHelixResourceManager;
+import org.apache.pinot.controller.helix.core.minion.PinotTaskManager;
+import org.apache.pinot.controller.helix.core.rebalance.TableRebalancer;
+import org.apache.pinot.controller.util.TaskConfigUtils;
+import org.apache.pinot.segment.local.utils.TableConfigUtils;
+import org.apache.pinot.spi.config.table.TableConfig;
+import org.apache.pinot.spi.config.table.TableType;
+import org.apache.pinot.spi.data.Schema;
+import org.apache.pinot.spi.utils.builder.TableNameBuilder;
+
+
+/**
+ * Utility class that encapsulates table config validation logic shared across
+ * {@link PinotTableRestletResource} and StarTree's managed logical table
resource.
+ *
+ * This lives in {@code pinot-controller} (not {@code
pinot-segment-local}'s {@code TableConfigUtils})
+ * because validation requires controller-level dependencies like {@link
PinotHelixResourceManager},
+ * {@link ControllerConf}, {@link PinotTaskManager}, and {@link
TableRebalancer}.
+ */
+public final class TableConfigValidationUtils {
+
+ private TableConfigValidationUtils() {
+ }
+
+ /**
+ * Validates a table config against the given schema and controller
configuration.
+ *
+ * Performs the following validations in order:
+ *
+ * Core validation ({@link TableConfigUtils#validate})
+ * Table name validation ({@link
TableConfigUtils#validateTableName})
+ * Min replicas enforcement
+ * Storage quota constraints
+ * Hybrid table config check (if both OFFLINE and REALTIME versions
exist)
+ * Task config validation (skipped if {@code taskManager} is null)
+ * Instance assignment validation
+ *
+ *
+ * NOT included (caller responsibility):
+ *
+ * Schema retrieval — caller resolves it
+ * Tuner config application — CREATE-only, mutates config
+ * Active tasks check — caller-specific
+ *
+ *
+ * @param tableConfig the table config to validate
+ * @param schemathe schema for the table (must not be null)
+ * @param typesToSkip comma-separated list of validation types to skip
(ALL|TASK|UPSERT), or null
+ * @param resourceManager the Helix resource manager
+ * @param controllerConf the controller configuration
+ * @param taskManager the task manager, or null to skip task validation
+ */
+ public static void validateTableConfig(TableConfig tableConfig, Schema
schema,
+ @Nullable String typesToSkip, PinotHelixResourceManager resourceManager,
+ ControllerConf controllerConf, @Nullable PinotTaskManager taskManager) {
+TableConfigUtils.validate(tableConfig, schema, typesToSkip);
+TableConfigUtils.validateTableName(tableConfig);
+TableConfigUtils.ensureMinReplicas(tableConfig,
controllerConf.getDefaultTableMinReplicas());
+TableConfigUtils.ensureStorageQuotaConstraints(tableConfig,
controllerConf.getDimTableMaxSize());
+checkHybridTableConfig(resourceManager, tableConfig);
+TaskConfigUtils.validateTaskConfigs(tableConfig, schema, taskManager,
typesToSkip);
+validateInstanceAssignment(resourceManager, tableConfig);
+ }
+
+ private static void checkHybridTableConfig(PinotHelixResourceManager
resourceManager, TableConfig tableConfig) {
+String rawTableName =
TableNameBuilder.extractRawTableName(tableConfig.getTableName());
+if (tableConfig.getTableType() == TableType.REALTIME) {
+ if (resourceManager.hasOfflineTable(rawTableName)) {
+TableConfigUtils.verifyHybridTableConfigs(rawTableName,
+resourceManager.getOfflineTableConfig(rawTableName), tableConfig);
+ }
+} else {
+ if (resourceManager.hasRealtimeTable(rawTableName)) {
+TableConfigUtils.verifyHybridTableConfigs(rawTableName, tableConfig
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2796846698
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigValidationUtils.java:
##
@@ -0,0 +1,113 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pinot.controller.api.resources;
+
+import javax.annotation.Nullable;
+import org.apache.pinot.controller.ControllerConf;
+import org.apache.pinot.controller.helix.core.PinotHelixResourceManager;
+import org.apache.pinot.controller.helix.core.minion.PinotTaskManager;
+import org.apache.pinot.controller.helix.core.rebalance.TableRebalancer;
+import org.apache.pinot.controller.util.TaskConfigUtils;
+import org.apache.pinot.segment.local.utils.TableConfigUtils;
+import org.apache.pinot.spi.config.table.TableConfig;
+import org.apache.pinot.spi.config.table.TableType;
+import org.apache.pinot.spi.data.Schema;
+import org.apache.pinot.spi.utils.builder.TableNameBuilder;
+
+
+/**
+ * Utility class that encapsulates table config validation logic shared across
+ * {@link PinotTableRestletResource} and StarTree's managed logical table
resource.
+ *
+ * This lives in {@code pinot-controller} (not {@code
pinot-segment-local}'s {@code TableConfigUtils})
+ * because validation requires controller-level dependencies like {@link
PinotHelixResourceManager},
+ * {@link ControllerConf}, {@link PinotTaskManager}, and {@link
TableRebalancer}.
+ */
+public final class TableConfigValidationUtils {
+
+ private TableConfigValidationUtils() {
+ }
+
+ /**
+ * Validates a table config against the given schema and controller
configuration.
+ *
+ * Performs the following validations in order:
+ *
+ * Core validation ({@link TableConfigUtils#validate})
+ * Table name validation ({@link
TableConfigUtils#validateTableName})
+ * Min replicas enforcement
+ * Storage quota constraints
+ * Hybrid table config check (if both OFFLINE and REALTIME versions
exist)
+ * Task config validation (skipped if {@code taskManager} is null)
Review Comment:
Done
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigValidationUtils.java:
##
@@ -0,0 +1,113 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pinot.controller.api.resources;
+
+import javax.annotation.Nullable;
+import org.apache.pinot.controller.ControllerConf;
+import org.apache.pinot.controller.helix.core.PinotHelixResourceManager;
+import org.apache.pinot.controller.helix.core.minion.PinotTaskManager;
+import org.apache.pinot.controller.helix.core.rebalance.TableRebalancer;
+import org.apache.pinot.controller.util.TaskConfigUtils;
+import org.apache.pinot.segment.local.utils.TableConfigUtils;
+import org.apache.pinot.spi.config.table.TableConfig;
+import org.apache.pinot.spi.config.table.TableType;
+import org.apache.pinot.spi.data.Schema;
+import org.apache.pinot.spi.utils.builder.TableNameBuilder;
+
+
+/**
+ * Utility class that encapsulates table config validation logic shared across
+ * {@link PinotTableRestletResource} and StarTree's managed logical table
resource.
+ *
+ * This lives in {@code pinot-controller} (not {@code
pinot-segment-local}'s {@code TableConfigUtils})
+ * because validation requires controller-level dependencies like {@link
PinotHelixResourceManager},
+ * {@link ControllerConf}, {@link PinotTas
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2796841390
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/ControllerRequestClient.java:
##
@@ -209,6 +209,30 @@ public void deleteLogicalTable(String logicalTableName)
}
}
+ public String getLogicalTable(String logicalTableName)
+ throws IOException {
+try {
+ SimpleHttpResponse response = HttpClient.wrapAndThrowHttpException(
+ _httpClient.sendGetRequest(
+ new
URI(_controllerRequestURLBuilder.forLogicalTableGet(logicalTableName)),
_headers));
+ return response.getResponse();
+} catch (HttpErrorStatusException | URISyntaxException e) {
+ throw new IOException(e);
+}
+ }
+
+ public List getLogicalTableNames()
+ throws IOException {
+try {
+ SimpleHttpResponse response = HttpClient.wrapAndThrowHttpException(
+ _httpClient.sendGetRequest(
+ new URI(_controllerRequestURLBuilder.forLogicalTableNamesGet()),
_headers));
+ return JsonUtils.stringToObject(response.getResponse(), List.class);
+} catch (HttpErrorStatusException | URISyntaxException e) {
+ throw new IOException(e);
+}
+ }
Review Comment:
Not a Problem
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2796840414
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/TableConfigUtils.java:
##
@@ -165,7 +165,7 @@ public static void validate(TableConfig tableConfig, Schema
schema) {
* TODO: Add more validations for each section (e.g. validate conditions are
met for aggregateMetrics)
*/
public static void validate(TableConfig tableConfig, Schema schema,
@Nullable String typesToSkip) {
-Preconditions.checkArgument(schema != null, "Schema should not be null");
+Preconditions.checkArgument(schema != null, "Schema should not be null for
table: %s", tableConfig.getTableName());
Review Comment:
Table config won't be null.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2793026294
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotLogicalTableResource.java:
##
@@ -148,7 +148,7 @@ public SuccessResponse addLogicalTable(
return new ConfigSuccessResponse(successResponse.getStatus(),
logicalTableConfigAndUnrecognizedProps.getRight());
}
- private void translatePhysicalTableNamesWithDB(LogicalTableConfig
logicalTableConfig, HttpHeaders headers) {
+ public static void translatePhysicalTableNamesWithDB(LogicalTableConfig
logicalTableConfig, HttpHeaders headers) {
Review Comment:
Moved to LogicalTableConfigUtils
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
-
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
Copilot commented on code in PR #17679:
URL: https://github.com/apache/pinot/pull/17679#discussion_r2792964850
##
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/utils/TableConfigUtils.java:
##
@@ -165,7 +165,7 @@ public static void validate(TableConfig tableConfig, Schema
schema) {
* TODO: Add more validations for each section (e.g. validate conditions are
met for aggregateMetrics)
*/
public static void validate(TableConfig tableConfig, Schema schema,
@Nullable String typesToSkip) {
-Preconditions.checkArgument(schema != null, "Schema should not be null");
+Preconditions.checkArgument(schema != null, "Schema should not be null for
table: %s", tableConfig.getTableName());
Review Comment:
`tableConfig.getTableName()` is dereferenced inside the `schema != null`
precondition message, which can throw an NPE if `tableConfig` is null (or if
`getTableName()` returns null) before the intended argument validation failure.
Consider adding a separate precondition for `tableConfig` earlier, or avoid
dereferencing `tableConfig` in the message (e.g., use a safe fallback like
`String.valueOf(tableConfig)` or guard the table name lookup).
##
pinot-controller/src/main/java/org/apache/pinot/controller/helix/ControllerRequestClient.java:
##
@@ -209,6 +209,30 @@ public void deleteLogicalTable(String logicalTableName)
}
}
+ public String getLogicalTable(String logicalTableName)
+ throws IOException {
+try {
+ SimpleHttpResponse response = HttpClient.wrapAndThrowHttpException(
+ _httpClient.sendGetRequest(
+ new
URI(_controllerRequestURLBuilder.forLogicalTableGet(logicalTableName)),
_headers));
+ return response.getResponse();
+} catch (HttpErrorStatusException | URISyntaxException e) {
+ throw new IOException(e);
+}
+ }
+
+ public List getLogicalTableNames()
+ throws IOException {
+try {
+ SimpleHttpResponse response = HttpClient.wrapAndThrowHttpException(
+ _httpClient.sendGetRequest(
+ new URI(_controllerRequestURLBuilder.forLogicalTableNamesGet()),
_headers));
+ return JsonUtils.stringToObject(response.getResponse(), List.class);
+} catch (HttpErrorStatusException | URISyntaxException e) {
+ throw new IOException(e);
+}
+ }
Review Comment:
`JsonUtils.stringToObject(..., List.class)` returns a raw `List` and relies
on an unchecked conversion to `List`. To make this type-safe (and avoid
runtime surprises if the payload changes), deserialize with an explicit generic
type (e.g., Jackson `TypeReference>` equivalent used by
`JsonUtils`, if available).
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotLogicalTableResource.java:
##
@@ -148,7 +148,7 @@ public SuccessResponse addLogicalTable(
return new ConfigSuccessResponse(successResponse.getStatus(),
logicalTableConfigAndUnrecognizedProps.getRight());
}
- private void translatePhysicalTableNamesWithDB(LogicalTableConfig
logicalTableConfig, HttpHeaders headers) {
+ public static void translatePhysicalTableNamesWithDB(LogicalTableConfig
logicalTableConfig, HttpHeaders headers) {
Review Comment:
Changing this helper from `private` to `public static` expands the public
API surface of a REST resource class for an internal utility operation. If this
is only meant for reuse within the package/module, prefer package-private
(`static` with no access modifier) or move it into a dedicated utility class
(keeping the REST resource focused on request handling).
```suggestion
static void translatePhysicalTableNamesWithDB(LogicalTableConfig
logicalTableConfig, HttpHeaders headers) {
```
##
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigValidationUtils.java:
##
@@ -0,0 +1,113 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pinot.controller.api.resources;
+
+import javax.annotation.Nullable;
+import org.apache.pinot.controller.ControllerConf;
+import org.apache.pinot.controller.helix.core.PinotHelixResourceManager;
+import org.apache.pinot.controller.helix.
Re: [PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
codecov-commenter commented on PR #17679: URL: https://github.com/apache/pinot/pull/17679#issuecomment-3883230044 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/17679?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report :x: Patch coverage is `0%` with `1 line` in your changes missing coverage. Please review. :white_check_mark: Project coverage is 55.60%. Comparing base ([`4ef8e3e`](https://app.codecov.io/gh/apache/pinot/commit/4ef8e3eafd14a3c656689afe910bc3a3c615773d?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`3291810`](https://app.codecov.io/gh/apache/pinot/commit/3291810be91186abe9ff24d27987f763b0ebb953?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)). :warning: Report is 9 commits behind head on master. | [Files with missing lines](https://app.codecov.io/gh/apache/pinot/pull/17679?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines | |---|---|---| | [...he/pinot/segment/local/utils/TableConfigUtils.java](https://app.codecov.io/gh/apache/pinot/pull/17679?src=pr&el=tree&filepath=pinot-segment-local%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fpinot%2Fsegment%2Flocal%2Futils%2FTableConfigUtils.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-cGlub3Qtc2VnbWVudC1sb2NhbC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvcGlub3Qvc2VnbWVudC9sb2NhbC91dGlscy9UYWJsZUNvbmZpZ1V0aWxzLmphdmE=) | 0.00% | [0 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/pinot/pull/17679?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | > :exclamation: There is a different number of reports uploaded between BASE (4ef8e3e) and HEAD (3291810). Click for more details. > > HEAD has 36 uploads less than BASE > >| Flag | BASE (4ef8e3e) | HEAD (3291810) | >|--|--|--| >|java-21|5|1| >|unittests1|2|1| >|unittests|4|1| >|temurin|10|1| >|java-11|5|0| >|unittests2|2|0| >|integration|6|0| >|integration2|2|0| >|integration1|2|0| >|custom-integration1|2|0| > Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #17679 +/- ## - Coverage 63.25% 55.60% -7.65% + Complexity 1499 720 -779 Files 3174 2475 -699 Lines190323 140301 -50022 Branches 2908022349-6731 - Hits 12038178013 -42368 + Misses6060655709-4897 + Partials 9336 6579-2757 ``` | [Flag](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [custom-integration1](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration1](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [integration2](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [java-11](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [java-21](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `55.60% <0.00%> (-7.62%)` | :arrow_down: | | [temurin](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `55.60% <0.00%> (-7.65%)` | :arrow_down: | | [unittests](https://app.codecov.io/gh/apache/pinot/pull/17679/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_camp
[PR] Refactor table config validations for create and update table to TableConfigValidationUtils [pinot]
krishan1390 opened a new pull request, #17679: URL: https://github.com/apache/pinot/pull/17679 Refactor table config validations for create and update table to TableConfigValidationUtils and thus reducing code duplication. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] - To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
