Skip to content
This repository has been archived by the owner on Jul 2, 2024. It is now read-only.

Use something better than null for isolation status #4762

Open
edongashi opened this issue Oct 16, 2020 · 2 comments
Open

Use something better than null for isolation status #4762

edongashi opened this issue Oct 16, 2020 · 2 comments

Comments

@edongashi
Copy link
Member

We get a null when isolation status is skipped. This is confusing and causes bugs.

@sebastian
Copy link
Member

For clarification: you mean in the code that needs to know whether a column is isolating or not inside the cloak, or outside the cloak in say air.

@edongashi
Copy link
Member Author

Air needs to know isolation status in some places.

  1. SHOW COLUMNS FROM ....
  2. The UI column background color and label.

When we say a column has isolation_status: null (and send it through the wire from cloak to air) it is very unclear what is meant by that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants