Table schema cache and schema change event use different table names, the later with user_ prefix

[Charles Wang]

The schema change event reflects the real table name (with user_ prefix) while the table schema cache uses the table name without user_ prefix.

    def schema_change_listener(self, event):
        LOG.debug("Schema change event captured: %s" % event)

        tenant = event.get('keyspace')
        table_name = event.get('table')

        if (tenant is None) or (table_name is None):
            return

        if event['change_type'] == "DROPPED":
            self._remove_table_schema_from_cache(tenant, table_name)

Should be changed to:

    def schema_change_listener(self, event):
        LOG.debug("Schema change event captured: %s" % event)

        tenant = event.get('keyspace')
        table_name = event.get('table')[len(self.USER_TABLE_PREFIX):]

        if (tenant is None) or (table_name is None):
            return

        if event['change_type'] == "DROPPED":
            self._remove_table_schema_from_cache(tenant, table_name)

[Dmitriy Ukhlov (dukhlov)]

Charles, is this bug out of date after storage level refactoring for async tables? Because now schema change listener was removed at all

[Charles Wang (charles-wang)]

Yes it's no longer valid. Let's close the bug since the function has been removed.

[Charles Wang (charles-wang)]

Changed to fix committed since the buggy code was removed.