tree b984d16995ca511104a88bf1e1b018dd95bdb0c0
parent 119d8ecfba898799a9ad39eb756fa0247739a4a6
author nharmata <nharmata@google.com> 1521822491 -0700
committer Copybara-Service <copybara-piper@google.com> 1521822619 -0700

Remove synchronization in BazelPackageBuilderHelperForTesting#sanityCheckBazelPackageLoader.

I should have written this comment more precisely. I am sorry I didn't. I vaguely recall some issues where the PackageFactory usage would do some scan of all the skylark classes and do some mutation to global state (this wouldn't normally be a problem since for blaze/bazel, there's one PackageFactory instance created per blaze/bazel server, and it's created at the beginning of the server).

Nevertheless, I do not see any problems now.

RELNOTES: None
PiperOrigin-RevId: 190229913