john-bodley edited a comment on pull request #11714: URL: https://github.com/apache/incubator-superset/pull/11714#issuecomment-738458838
In general I sense we should not be storing columns (or fields in JSON objects, etc.) which can easily be derived from other columns, i.e., `path_no_int` from `path`. This adds unnecessary bloat especially when the use case is not well defined. As I mentioned in [this](https://github.com/apache/incubator-superset/pull/11714#issuecomment-737573134) comment and as @etr2460 also stated using the `json` column seems like a better place to store experimental fields (or those subject to change) especially when they importance is secondary. For example the `json` column could include a `request` field with all the nested request properties (arguments, referrer, path, etc.). This provides a level of hierarchy/organization and flexibility. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
