dosubot[bot] commented on issue #31695:
URL: https://github.com/apache/superset/issues/31695#issuecomment-2568918630

   <!-- Answer -->
   The issue you're encountering with the `convert_tbl_column_to_sqla_col` 
function, where `self.type` is used instead of `tbl_column.type`, does appear 
to be a bug. The use of `self.type` results in `None` for `column_spec`, which 
is not the intended behavior. Changing it to `tbl_column.type` resolves the 
issue by correctly using the type of the table column being converted 
<sup>[[1]](https://github.com/apache/superset/blob/master/superset/models/helpers.py)</sup>.
   
   There is a similar unsolved discussion on this topic, which you might find 
relevant: [Why use self.type in function 
"convert_tbl_column_to_sqla_col"](https://github.com/apache/superset/discussions/31680)
 <sup>[[2]](https://github.com/apache/superset/discussions/31680)</sup>.
   
   
   <!-- Dosu Comment Footer -->
   *To continue talking to [Dosu](https://dosu.dev), mention @dosu.*
   
   ---
   <sup>Help Dosu learn! Give it feedback: [Great 
Response](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=great_response)
 | [Irrelevant 
Answer](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=irrelevant_answer)
 | [Incorrect 
Sources](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=incorrect_sources)
 | [Too 
Verbose](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=hallucination)
 | 
[Other](https://app.dosu.dev/response-feedback/3e9934bb-e341-4f0a-a677-5d8e1175dbe5?feedback_type=other)</sup>
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to