Changes between Version 2 and Version 3 of Ticket #34944, comment 14


Ignore:
Timestamp:
Nov 7, 2023, 8:18:59 PM (6 months ago)
Author:
Simon Charette

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #34944, comment 14

    v2 v3  
    77> The SQL code for the generated column has an automatically inferred max length of only 255 characters (varchar(255))
    88
    9 I wish we were more rigid in this particular case when dealing with parameterized fields such as `CharFied`, `ArrayField`, `DecimalField` but it's non-trivial and often backward incompatible.
     9I wish we were more rigid in this particular case when dealing with parameterized fields such as `CharFied`, `ArrayField`, `DecimalField` but it's non-trivial and often backward incompatible. In this particular case it's tempting to adjust the logic of `Concat._resolve_output_field` to be smarter and generate a `CharField(max_length=511)` but I suspect it might have unintended effect and not something we'd like to include in 5.0 at this point.
    1010
    1111I suggest that we take the same approach here and expect the user to specify an explicit `output_field`.
Back to Top