noticket: Minor InMemoryQueries cleanup

- Use switch expression in visitors
- Relax generic bounds in `InMemoryQueries.toComparator()`
(`T` need **not** extend `Entity<T>`)

Author: nvamelichev

repository/src/main/java/tech/ydb/yoj/repository/db/list/InMemoryQueries.java

@@ -103,28 +103,20 @@ public Predicate<T> visitScalarExpr(@NonNull ScalarExpr<T> scalarExpr) {
 
             @Override
             public Predicate<T> visitNullExpr(@NonNull NullExpr<T> nullExpr) {
-                switch (nullExpr.getOperator()) {
-                    case IS_NULL:
-                        return nullExpr::isActualValueNull;
-                    case IS_NOT_NULL:
-                        return not(nullExpr::isActualValueNull);
-                    default:
-                        throw new UnsupportedOperationException("Unsupported operator in nullability expression: " + nullExpr.getOperator());
-                }
+                return switch (nullExpr.getOperator()) {
+                    case IS_NULL -> nullExpr::isActualValueNull;
+                    case IS_NOT_NULL -> not(nullExpr::isActualValueNull);
+                };
             }
 
             @Override
             public Predicate<T> visitListExpr(@NonNull ListExpr<T> listExpr) {
                 Function<T, Comparable<?>> getActual = listExpr::getActualValue;
                 List<Comparable<?>> expected = listExpr.getExpectedValues();
-                switch (listExpr.getOperator()) {
-                    case IN:
-                        return obj -> expected.contains(getActual.apply(obj));
-                    case NOT_IN:
-                        return obj -> !expected.contains(getActual.apply(obj));
-                    default:
-                        throw new UnsupportedOperationException("Unsupported operator in filter expression: " + listExpr.getOperator());
-                }
+                return switch (listExpr.getOperator()) {
+                    case IN -> obj -> expected.contains(getActual.apply(obj));
+                    case NOT_IN -> obj -> !expected.contains(getActual.apply(obj));
+                };
             }
 
             @Override
@@ -148,7 +140,7 @@ public Predicate<T> visitNotExpr(@NonNull NotExpr<T> notExpr) {
         });
     }
 
-    public static <T extends Entity<T>> Comparator<T> toComparator(@NonNull OrderExpression<T> orderBy) {
+    public static <T> Comparator<T> toComparator(@NonNull OrderExpression<T> orderBy) {
         if (orderBy.isUnordered()) {
             // Produces a randomly-ordering, but stable Comparator. UUID.randomUUID() collisions are extremely unlikely, so we ignore them.
             Map<Object, UUID> randomIds = new IdentityHashMap<>();

repository/src/main/java/tech/ydb/yoj/repository/db/list/ViewListResult.java

@@ -45,10 +45,12 @@ public ViewListResult<T, V> returnWithParams(@NonNull ListRequest.ListingParams<
     }
 
     @NonNull
-    private static <T extends Entity<T>, V extends Table.View> ViewSchema<V> getViewSchema(@NonNull ListRequest<T> request, @NonNull Class<V> viewClass) {
+    private static <T extends Entity<T>, V extends Table.View> ViewSchema<V> getViewSchema(
+            @NonNull ListRequest<T> request,
+            @NonNull Class<V> viewClass
+    ) {
         if (!(request.getSchema() instanceof EntitySchema<?> entitySchema)) {
-            throw new IllegalArgumentException("Expected ListRequest for an entity, but got a non-entity schema of type "
-                    + "<" + request.getSchema().getClass().getName() + ">");
+            throw new IllegalArgumentException("Expected ListRequest for an entity, but got a non-entity schema: " + request.getSchema());
         }
         return entitySchema.getViewSchema(viewClass);
     }