https://dagster.io/ logo
Title
h

Harrison Conlin

03/29/2023, 11:37 PM
With PartitionMapping's can I just get some clarification on what 'Overriding PartitionMapping outside of Dagster is not supported' means? does this mean, please don't implement your own partitionmapping class?
:dagster-bot-resolve: 1
c

claire

03/29/2023, 11:46 PM
Hi Harrison. Partition mapping method signatures can change depending on internal callsites, so if you did implement your own partition mapping, it's possible that upgrading can lead to unexpected breakages if we changed the method signatures. If you don't mind this happening, you can implement your own partition mapping class. Another option is also opening up a PR to add your partition mapping to the dagster library, and we'll maintain it from there.
h

Harrison Conlin

03/29/2023, 11:48 PM
thanks. I'm use to my code breaking all the time, so that's not the biggest of issues 🙂
:blob-grin: 1