https://pantsbuild.org/ logo
#development
Title
# development
h

hundreds-father-404

03/13/2020, 7:31 AM
Every Target must have an address, correct? Does it sound like a performance issue for every
Field
instance to store the
Address
for its owning
Target
? This would allow substantially better error messages when validating fields, e.g. being able to say `"Invalid compatibility field for target //:lib`“` rather than
“Invalid compatibility field”
. We also need the
Address
for hydrating certain fields like
Sources
. But, storing
Address
on every field seems like a performance concern. Is it a premature optimization to worry about that performance concern? Is it worth it for the sake of substantially better error messages?