You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2021/10/22 12:45:16 UTC

[GitHub] [superset] LeiZhangImo opened a new issue #17199: dashboard/dataset export raises schema issues

LeiZhangImo opened a new issue #17199:
URL: https://github.com/apache/superset/issues/17199


   A clear and concise description of what the bug is.
   
   #### How to reproduce the bug
   
   1. Go to '...'
   2. Click on '....'
   3. Scroll down to '....'
   4. See error
   
   ### Expected results
   
   what you expected to happen.
   
   ### Actual results
   When I use swagger for testing dataset export API in
   mysupersite.com/swagger/v1
   under the dataset/export part, after putting 1 in the 'q' filed, I'm getting this error:
   **{
     "message": "Not a valid rison schema '1' is not of type 'integer'\n\nFailed validating 'type' in schema['items']:\n    {'type': 'integer'}\n\nOn instance[0]:\n    '1'"
   }**
   
   #### Screenshots
   ![image](https://user-images.githubusercontent.com/8062171/138454825-bfab76a1-4b75-46ce-95d5-b2b679510dbf.png)
   ![image](https://user-images.githubusercontent.com/8062171/138455091-b0981636-1b45-4758-951d-3d0a981042a5.png)
   
   
   
   ### Environment
   
   - browser type and version: Chrome/94.0.4606.61
   - superset version:  superset docker: apache/superset:1.3.2
   - python version: Python 3.8.12
   - node.js version: not know
   - any feature flags active: connected to aws Anthena
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [x ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
   - [ x] I have reproduced the issue with at least the latest released version of superset.
   - [x ] I have checked the issue tracker for the same issue and I haven't found one similar.
   
   ### Additional context
   
   Add any other context about the problem here.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] LeiZhangImo commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
LeiZhangImo commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-950151662


   > Try this instead : `%5B1%2C%222%22%5D` , you will get error message about '2'. 1 will be successful since 1 is integer in this encoded string. Try the given string in https://meyerweb.com/eric/tools/dencoder/
   
   yes, you are right, the API work if do url encode my myself, the there must be some swagger bug.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] LeiZhangImo commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
LeiZhangImo commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-949958174


   > Looks like this is issue in one of Superset dependencies. Here is request url that swagger compose:
   > 
   > ```
   > /api/v1/dataset/export/?q=%5B%221%22%5D
   > ```
   > 
   > `%5B%221%22%5D` is really `["1"]`. Value send as string, instead of integer. I tend to think that the issue is in `flask-appbuilder` or in swagger itself.
   
   Thank for your reply,
   however, in the error log, it complains '1' is not type of 'integer instead of '["1"]' is not type of 'interger'
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] LeiZhangImo commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
LeiZhangImo commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-950153226


   after I manually fixed this error, the import/create API aways get ERROR:
   https://github.com/apache/superset/issues/17206


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] jayakrishnankk commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
jayakrishnankk commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-991423480


   > This endpoint is expecting `/api/v1/dataset/export/?q=(1,2,3)` for example, the REST API uses Rison parameters or a JSON encoded string. Looks like swagger is sending a JSON encoded string. I'll look into finding a solution
   
   @dpgaspar - wondering if you found a solution to this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] dpgaspar commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
dpgaspar commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-954558470


   This endpoint is expecting `/api/v1/dataset/export/?q=(1,2,3)` for example, the REST API uses Rison parameters or a JSON encoded string. Looks like swagger is sending a JSON encoded string. I'll look into finding a solution
    


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] jayakrishnankk commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
jayakrishnankk commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-991423480


   > This endpoint is expecting `/api/v1/dataset/export/?q=(1,2,3)` for example, the REST API uses Rison parameters or a JSON encoded string. Looks like swagger is sending a JSON encoded string. I'll look into finding a solution
   
   @dpgaspar - wondering if you found a solution to this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] StpMax commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
StpMax commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-949784563


   Looks like this is issue in one of Superset dependencies. Here is request url that swagger compose:
   ```
   /api/v1/dataset/export/?q=%5B%221%22%5D
   ```
   `%5B%221%22%5D` is really `["1"]`. Value send as string, instead of integer. I tend to think that the issue is in `flask-appbuilder` or in swagger itself.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] jayakrishnankk commented on issue #17199: dashboard/dataset export raises schema issues

Posted by GitBox <gi...@apache.org>.
jayakrishnankk commented on issue #17199:
URL: https://github.com/apache/superset/issues/17199#issuecomment-949979970


   Try this instead : `%5B1%2C%222%22%5D` , you will get error message about '2'. 1 will be successful since 1 is integer in this encoded string. Try the given string in https://meyerweb.com/eric/tools/dencoder/


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org