-
Notifications
You must be signed in to change notification settings - Fork 13.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TypeError: 'dict' object does not support indexing when filter contains "%" symbol #528
Comments
|
Is this still an issue with latest caravel? |
I can confirm that using the latest (Python 3 based) Caravel no longer gives this error. Relevant versions tested:
Thank you very much! |
I am getting the same issue with Superset 0.13.2 running on Python 3.5:
Here's the query (names changed):
Thank you for your help! |
What database engine / driver are you using? |
GreenPlum (Postgres 8.2 compatible). The same query works in pgAdmin without issue. |
What about the driver? psycopg2? What does the pre |
|
mind trying |
Same error with |
@mpacula @mistercrunch is this an issue with the psycopg2 library? I'm using it with sqlalchemy and getting same error when my sql contains % in WHERE clause criteria |
+1 same issue here. |
There are many issues described here (same error msg, different stack traces), and many of those have been reported a long time ago. Since then much work has been done. @joshuacano please open a new issue with more context (stack grace, version, connection string, ...) |
@mistercrunch Yep, Totally understand. I've done so here - #2710 |
* feat(plugin-chart-choropleth-map): scaffold and load map (apache#527) * feat: add package * feat: storybook working * feat: load usa and world map * refactor: clean up * fix: remove test data * refactor: utilize dynamic import * build: remove unused dependencies * fix: address pr comments * fix: comment * feat(plugin-chart-choropleth-map): add more country maps (apache#529) * feat(plugin-chart-choropleth-map): add zooming (apache#528) * feat: add zooming * feat: can zoom in and out * feat: add zoom controls * refactor: extract controls * fix: address comments * feat(plugin-chart-choropleth-map): add encoding (apache#541) * feat: add encoder * feat: add encoding * docs: add categorical * fix: any * docs: update storybook * feat(plugin-chart-choropleth-map): add tooltip (apache#548) * feat: support tooltip * feat: support tooltip fields * fix: default projection * build: bump dependency * build: update dependency * build: mark private
* feat(plugin-chart-choropleth-map): scaffold and load map (apache#527) * feat: add package * feat: storybook working * feat: load usa and world map * refactor: clean up * fix: remove test data * refactor: utilize dynamic import * build: remove unused dependencies * fix: address pr comments * fix: comment * feat(plugin-chart-choropleth-map): add more country maps (apache#529) * feat(plugin-chart-choropleth-map): add zooming (apache#528) * feat: add zooming * feat: can zoom in and out * feat: add zoom controls * refactor: extract controls * fix: address comments * feat(plugin-chart-choropleth-map): add encoding (apache#541) * feat: add encoder * feat: add encoding * docs: add categorical * fix: any * docs: update storybook * feat(plugin-chart-choropleth-map): add tooltip (apache#548) * feat: support tooltip * feat: support tooltip fields * fix: default projection * build: bump dependency * build: update dependency * build: mark private
* feat(plugin-chart-choropleth-map): scaffold and load map (apache#527) * feat: add package * feat: storybook working * feat: load usa and world map * refactor: clean up * fix: remove test data * refactor: utilize dynamic import * build: remove unused dependencies * fix: address pr comments * fix: comment * feat(plugin-chart-choropleth-map): add more country maps (apache#529) * feat(plugin-chart-choropleth-map): add zooming (apache#528) * feat: add zooming * feat: can zoom in and out * feat: add zoom controls * refactor: extract controls * fix: address comments * feat(plugin-chart-choropleth-map): add encoding (apache#541) * feat: add encoder * feat: add encoding * docs: add categorical * fix: any * docs: update storybook * feat(plugin-chart-choropleth-map): add tooltip (apache#548) * feat: support tooltip * feat: support tooltip fields * fix: default projection * build: bump dependency * build: update dependency * build: mark private
* feat(plugin-chart-choropleth-map): scaffold and load map (apache#527) * feat: add package * feat: storybook working * feat: load usa and world map * refactor: clean up * fix: remove test data * refactor: utilize dynamic import * build: remove unused dependencies * fix: address pr comments * fix: comment * feat(plugin-chart-choropleth-map): add more country maps (apache#529) * feat(plugin-chart-choropleth-map): add zooming (apache#528) * feat: add zooming * feat: can zoom in and out * feat: add zoom controls * refactor: extract controls * fix: address comments * feat(plugin-chart-choropleth-map): add encoding (apache#541) * feat: add encoder * feat: add encoding * docs: add categorical * fix: any * docs: update storybook * feat(plugin-chart-choropleth-map): add tooltip (apache#548) * feat: support tooltip * feat: support tooltip fields * fix: default projection * build: bump dependency * build: update dependency * build: mark private
Hi I get a TypeError: 'dict' object does not support indexing error when trying to filter on a value that contains the "%" character.
Caravel version (from amancevice/caravel:0.9.0):
I'm using postgresql 9.4 as the caravel database and datasource.
From the logs, this was the URL that caused the error:
/caravel/explore/table/1/?[...]&extra_filters={"1":{"record":["Net Margin %"]}}[...]
The extra_filter bit was encoded like this in the logs:
%7B%221%22%3A%7B%22record%22%3A%5B%22Net%20Margin%20%25%22%5D%7D%7D
Full stack trace is in next comment.
The text was updated successfully, but these errors were encountered: