Skip to content

IGNITE-24254 Thin client: add schema name placeholder to protocol#5077

Merged
ptupitsyn merged 7 commits intoapache:mainfrom
ptupitsyn:ignite-24254
Jan 20, 2025
Merged

IGNITE-24254 Thin client: add schema name placeholder to protocol#5077
ptupitsyn merged 7 commits intoapache:mainfrom
ptupitsyn:ignite-24254

Conversation

@ptupitsyn
Copy link
Contributor

@ptupitsyn ptupitsyn commented Jan 20, 2025

Reserve space for schema name in TABLE_GET and TABLES_GET payloads.

@ptupitsyn ptupitsyn self-assigned this Jan 20, 2025
@ptupitsyn ptupitsyn changed the title IGNITE-24254 Thin client: add schema name placeholder to table ops IGNITE-24254 Thin client: add schema name placeholder to protocol Jan 20, 2025
@ptupitsyn ptupitsyn requested a review from AMashenkov January 20, 2025 12:09
@ptupitsyn ptupitsyn requested a review from isapego January 20, 2025 12:31
@AMashenkov
Copy link
Member

@ptupitsyn Do we need a similar fix for Compute API or anything else?
I've mention only the classes, where I saw failures in my branch (these ones you've changed), but I could miss smth.

@ptupitsyn
Copy link
Contributor Author

@AMashenkov Everywhere else we use int tableId, I've double checked. We should be good.

@ptupitsyn ptupitsyn merged commit 05c673f into apache:main Jan 20, 2025
1 check passed
@ptupitsyn ptupitsyn deleted the ignite-24254 branch January 20, 2025 15:54
ptupitsyn added a commit that referenced this pull request Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants