aromanenko-dev commented on a change in pull request #14585:
URL: https://github.com/apache/beam/pull/14585#discussion_r617510945



##########
File path: 
sdks/java/testing/tpcds/src/main/java/org/apache/beam/sdk/tpcds/TpcdsParametersReader.java
##########
@@ -17,17 +17,25 @@
  */
 package org.apache.beam.sdk.tpcds;
 
+import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Set;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 /** Get and check the TpcdsOptions' parameters, throw exceptions when user 
input is invalid. */
 public class TpcdsParametersReader {
+
   /** The data sizes that have been supported. */
   private static final Set<String> supportedDataSizes =
       Stream.of("1G", "10G", 
"100G").collect(Collectors.toCollection(HashSet::new));
 
+  private static final String QUERY_PREFIX = "query";
+
+  public static final List<String> SUPPORTED_QUERIES = getSupportedQueries();

Review comment:
       I renamed it to `ALL_QUERY_NAMES` since this list contains only query 
names.




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to