ydb-embedded-ui icon indicating copy to clipboard operation
ydb-embedded-ui copied to clipboard

Indentify replication targets tables with a sign

Open antonkovalenko opened this issue 1 year ago • 1 comments

Tables that receive changes from another database by means of async replicatoin are in read-only state.

Let's mark them with a special ring. --It has to be an animated overlay on the standard table library--

Image

And also let's display this info on Info panel and overpanel as well.

@CyberROFL knows parameter that is returned in describe for such tables

describe for a table that is a replicated targert must contain attribute __async_replica

antonkovalenko avatar Mar 29 '24 14:03 antonkovalenko

https://github.com/ydb-platform/ydb/issues/3317

CyberROFL avatar Mar 29 '24 15:03 CyberROFL

Decided to mark tables with labels in Info instead of SchemaTree

Image

artemmufazalov avatar Jun 14 '24 07:06 artemmufazalov

@antonkovalenko @adameat Nonetheless, don't we want to show R/O label inside the schema tree? It's rather more clear as for me. And about icon. Animated part is not a good idea if speaking about performance. But what if we add a small lock? Something like Screenshot 2024-07-03 at 09 45 34

Raubzeug avatar Jul 03 '24 06:07 Raubzeug

@antonkovalenko @adameat Nonetheless, don't we want to show R/O label inside the schema tree? It's rather more clear as for me.

It's how our schema works. We cannot display R/O label without this object describe. So, to display labels in schema tree, we need either request data for all entities, which is rather uneffective, or we can display it only for opened entities, which is not very good as well.

artemmufazalov avatar Jul 03 '24 06:07 artemmufazalov

It's how our schema works. We cannot display R/O label without this object describe. So, to display labels in schema tree, we need either request data for all entities, which is rather uneffective, or we can display it only for opened entities, which is not very good as well.

I've totally missed this "small" thing :( And isn't it possible to return any additional field for children in describe?

Raubzeug avatar Jul 03 '24 07:07 Raubzeug

I've totally missed this "small" thing :( And isn't it possible to return any additional field for children in describe?

it won't be cheap

adameat avatar Jul 03 '24 09:07 adameat