Commit ca30c37
committed
refactor(elasticsearch): implement data catalog compliance via MetaDatabaseHandler
Migrate Elasticsearch handler to inherit from MetaDatabaseHandler to comply with data catalog specifications per review feedback on PR mindsdb#11552.
Key changes:
- Change base class from DatabaseHandler to MetaDatabaseHandler
- Rename methods with meta_ prefix per data catalog API requirements
- Add meta_get_tables and meta_get_columns required methods
- Update column names to uppercase specification (TABLE_NAME, COLUMN_NAME, etc.)
- Calculate NULL_PERCENTAGE as percentage (0.0-100.0) instead of count
- Support multiple tables via Optional[List[str]] parameter
- Remove data catalog documentation from README
- Delete deprecated handler test files, consolidate tests in unit/handlers
- Update all 18 unit tests to match new API
- Maintain backward compatibility by preserving old methods
All tests passing (18/18) and pre-commit checks passed.1 parent ddde433 commit ca30c37
8 files changed
Lines changed: 252 additions & 1585 deletions
File tree
- mindsdb/integrations/handlers/elasticsearch_handler
- tests
- tests/unit/handlers
Lines changed: 0 additions & 50 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
143 | 93 | | |
144 | 94 | | |
145 | 95 | | |
| |||
Lines changed: 200 additions & 65 deletions
Large diffs are not rendered by default.
Lines changed: 0 additions & 55 deletions
This file was deleted.
0 commit comments