I was going to do performance testing as well to verify there are no regressions. Regarding keeping the old implementation I’m a bit concerned. Eventually it should be removed if there are no usages in Pants OSS. So let’s keep it deprecated for now so you’ll be able to move to the new one or move

LazySourceMapper

to Foursquare internal. Does it sound good?