sarankk commented on code in PR #165:
URL: https://github.com/apache/cassandra-sidecar/pull/165#discussion_r1893247431
##########
server-common/src/main/java/org/apache/cassandra/sidecar/common/server/exceptions/SchemaUnavailableException.java:
##########
@@ -16,21 +16,21 @@
* limitations under the License.
*/
-package org.apache.cassandra.sidecar.exceptions;
+package org.apache.cassandra.sidecar.common.server.exceptions;
/**
- * Exception thrown when {@link
org.apache.cassandra.sidecar.db.schema.TableSchema} is not prepared or expected
- * operations are unavailable.
+ * Exception thrown when {@link
org.apache.cassandra.sidecar.db.schema.TableSchema} does not exist.
+ * For instance, the connected Cassandra no longer has such table
*/
public class SchemaUnavailableException extends RuntimeException
{
- public SchemaUnavailableException(String message)
+ public SchemaUnavailableException(String keyspace, String table)
{
- super(message);
+ super(makeErrorMessage(keyspace, table));
}
- public SchemaUnavailableException(String message, Throwable cause)
+ private static String makeErrorMessage(String keyspace, String table)
{
- super(message, cause);
+ return "Table " + keyspace + '/' + table + " does not exist";
Review Comment:
This change is related to re-preparing schema statements. This is a
temporary fix. Will remove this, once we start Cassandra with authentication in
Integration tests.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
