adutra commented on code in PR #3758:
URL: https://github.com/apache/polaris/pull/3758#discussion_r2805040932
##########
polaris-core/src/main/java/org/apache/polaris/core/persistence/resolver/ResolverPath.java:
##########
@@ -18,76 +18,28 @@
*/
package org.apache.polaris.core.persistence.resolver;
-import com.google.common.collect.ImmutableList;
import java.util.List;
import org.apache.polaris.core.entity.PolarisEntityType;
-/** Simple class to represent a path within a catalog */
-public class ResolverPath {
-
- // name of the entities in that path. The parent of the first named entity
is the path is the
- // catalog
- private final List<String> entityNames;
-
- // all entities in a path are namespaces except the last one which can be a
table_like entity
- // versus a namespace
- private final PolarisEntityType lastEntityType;
-
- // true if this path is optional, i.e. failing to fully resolve it is not an
error
- private final boolean isOptional;
-
- /**
- * Constructor for an optional path
- *
- * @param entityNames set of entity names, all are namespaces except the
last one a namespa
- */
- public ResolverPath(List<String> entityNames) {
Review Comment:
This constructor was unused, so I removed it.
--
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]
