Revert "Autocomplete in the table browser in SQL lab is broken (#7736)"#7743
Conversation
|
+1 |
|
Looks like the backend should return the tables in a standardized format, and it's not? |
Codecov Report
@@ Coverage Diff @@
## master #7743 +/- ##
==========================================
- Coverage 65.78% 65.75% -0.03%
==========================================
Files 459 459
Lines 21925 21924 -1
Branches 2410 2410
==========================================
- Hits 14424 14417 -7
- Misses 7380 7386 +6
Partials 121 121
Continue to review full report at Codecov.
|
|
I believe the observed bug has its origins in changes introduced by #7453, which changes table names from strings to dicts/objects:
The change was done because many engines support periods in the object names, which confused the frontend. However, apparently there were side-effects that were not covered by unit tests. If having tables in object format causes problems, one solution could be to make them strings again, but make sure periods in object names are properly escaped. |
…e#7736)" (apache#7743) * Revert "Autocomplete in the table browser in SQL lab is broken (apache#7736)" This reverts commit 7139d8d. * Kick off build
Accidentally broke connecting to tables after searching. Reverting this change.
This reverts commit d1b81e0.
REVIEWERS
@betodealmeida @DiggidyDave @xtinec