[
https://issues.apache.org/jira/browse/IGNITE-24211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17912946#comment-17912946
]
Andrey Mashenkov commented on IGNITE-24211:
-------------------------------------------
[~phillippko], [~ibessonov],
I've fixed a bug in DisasterRecoveryManager.
Shortly:
The Catalog and DisasterRecoveryManager (see RestartPartitionsRequest) both
clams they operates with normalized names (case sensitive and non-quoted).
Assume, we have schema "A" with a table "\"B.C\"" and schema "\"A.B\"" with
table "C".
What happens when I send a request for restarting partitions for "A.B.C"? Which
table partitions will be affected?
Please, take a look at the PR.
> Sql. Avoid accessing of Catalog object directly from outside the module.
> ------------------------------------------------------------------------
>
> Key: IGNITE-24211
> URL: https://issues.apache.org/jira/browse/IGNITE-24211
> Project: Ignite
> Issue Type: Bug
> Components: sql
> Reporter: Andrey Mashenkov
> Priority: Major
> Labels: ignite-3, tech-debt
> Fix For: 3.1
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Catalog object is an internal container for descriptors and wasn't designed
> to be used directly outside the module.
> The class has public modifier just to allow Catalog commands access it from
> the different package of the same (ignite-catalog) module moved for better
> code organization.
> Now, ignite-rest and ignite-table modules access Catalog object directly.
> Moreover, they exploits it's internals in wrong way: cache table descriptors
> by "canonical name" as concatenation of schemaName and tableName via dot
> notation without escaping.
> Let's avoid direct using of Catalog objects.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
