[
https://issues.apache.org/jira/browse/CASSANDRA-17590?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Brandon Williams updated CASSANDRA-17590:
-----------------------------------------
Resolution: Not A Problem
Status: Resolved (was: Triage Needed)
Resolving as not a problem. Roles are part of CQL and used heavily in that
package.
> Move class RoleName into package org.apache.cassandra.cql3.statements
> ---------------------------------------------------------------------
>
> Key: CASSANDRA-17590
> URL: https://issues.apache.org/jira/browse/CASSANDRA-17590
> Project: Cassandra
> Issue Type: New Feature
> Reporter: yiy27
> Priority: Normal
> Attachments: Cassandra-RoleName-java.png
>
>
> *Issue Description*
>
> Package _*org.apache.cassandra.cql3*_ is a promiscuous package, and groups
> together miscellaneous functionalities that might be useful to different
> subsystems. The package structure violates the “high cohesion and low
> coupling” design rules. I found that class _*RoleName*_ is not used by
> classes in their package{_}.{_} The project has grown larger, leading to
> becoming increasingly hard to maintain. For example, if one modifies package
> _*org.apache.cassandra.cql3*_ (i.e., package rename), resulting in multiple
> classes of package _*org.apache.cassandra.cql3.statements*_ to be changed.
>
> Location: The source file can be found at path File
> src/java/org/apache/cassandra/cql3/RoleName.java **
>
> *Refactoring suggestions*
>
> I suggest to move class _*RoleName*_ into package
> {_}*org.apache.cassandra.cql3.statements*{_}.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
