Skip to content

Commit 1f3e8d3

Browse files
authored
CNDB-14848 CC5 remove unused duplicated constants from SchemaConstants (#1891)
### What is the issue Fixes CNDB-14848, removing unused, duplicated constants from `SchemaConstants` ### What does this PR fix and why was it fixed `SchemaConstants` contains unused and duplicated constants left over from previous rebases.
1 parent 0c2d619 commit 1f3e8d3

File tree

1 file changed

+1
-15
lines changed

1 file changed

+1
-15
lines changed

src/java/org/apache/cassandra/schema/SchemaConstants.java

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,6 @@ public final class SchemaConstants
5151
public static final String VIRTUAL_SCHEMA = "system_virtual_schema";
5252

5353
public static final String VIRTUAL_VIEWS = "system_views";
54-
public static final String SCHEMA_VIRTUAL_KEYSPACE_NAME = "system_virtual_schema";
55-
public static final String SYSTEM_VIEWS_KEYSPACE_NAME = "system_views";
5654

5755
public static final String DUMMY_KEYSPACE_OR_TABLE_NAME = "--dummy--";
5856

@@ -66,9 +64,6 @@ public final class SchemaConstants
6664
/* replicate system keyspace names (the ones with a "true" replication strategy) */
6765
public static final Set<String> REPLICATED_SYSTEM_KEYSPACE_NAMES = ImmutableSet.of(TRACE_KEYSPACE_NAME, AUTH_KEYSPACE_NAME, DISTRIBUTED_KEYSPACE_NAME);
6866

69-
/* virtual keyspace names */
70-
public static final Set<String> VIRTUAL_KEYSPACE_NAMES = ImmutableSet.of(SCHEMA_VIRTUAL_KEYSPACE_NAME, SYSTEM_VIEWS_KEYSPACE_NAME);
71-
7267
/**
7368
* Longest acceptable file name. Longer names lead to file write or read errors.
7469
*/
@@ -225,19 +220,10 @@ public static Set<String> getLocalAndReplicatedSystemTableNames()
225220
.build();
226221
}
227222

228-
/**
229-
* @return whether or not the keyspace is a virtual keyspace (system_virtual_schema, system_views)
230-
*/
231-
public static boolean isVirtualKeyspace(String keyspaceName)
232-
{
233-
return VIRTUAL_KEYSPACE_NAMES.contains(keyspaceName.toLowerCase());
234-
}
235-
236223
public static boolean isInternalKeyspace(String keyspaceName)
237224
{
238225
return isLocalSystemKeyspace(keyspaceName)
239-
|| isReplicatedSystemKeyspace(keyspaceName)
240-
|| isVirtualKeyspace(keyspaceName);
226+
|| isReplicatedSystemKeyspace(keyspaceName);
241227
}
242228

243229
/**

0 commit comments

Comments
 (0)