https://pantsbuild.org/ logo
Powered by
#general
  • e

    enough-analyst-54434

    09/24/2015, 7:44 PM
    @happy-kitchen-89482: unclear - looks like 'root' is the kwarg used by commons - which this delgates to
  • w

    witty-crayon-22786

    09/24/2015, 7:44 PM
    @bored-art-40741: well, we've been trying to track down something similar
  • e

    enough-analyst-54434

    09/24/2015, 7:44 PM
    I don;t think we pass it, and it decaults to os.curdir if not passed
  • e

    enough-analyst-54434

    09/24/2015, 7:44 PM
    so sketchy still
  • e

    enough-analyst-54434

    09/24/2015, 7:44 PM
    from that root - a walk
  • e

    enough-analyst-54434

    09/24/2015, 7:45 PM
    oh - we do pass root
  • e

    enough-analyst-54434

    09/24/2015, 7:45 PM
    all good
  • h

    happy-kitchen-89482

    09/24/2015, 7:47 PM
    Right, so I'm starting to think that our BUILD file parsing times are not (as we'd previously thought) because we have some rglobs.
  • b

    bored-art-40741

    09/24/2015, 7:48 PM
    This is very easy to profile
  • h

    happy-kitchen-89482

    09/24/2015, 7:48 PM
    Yeah, will do so.
  • b

    bored-art-40741

    09/24/2015, 7:50 PM
    Is 
    -XX:+PrintGCDetails
    combined with 
    -XX:+PrintGCTimeStamps
    sufficiently verbose that it will have perf implications?
  • w

    witty-crayon-22786

    09/24/2015, 7:51 PM
    generally not no
  • w

    witty-crayon-22786

    09/24/2015, 7:51 PM
    they print on the order of GC
  • b

    bored-art-40741

    09/24/2015, 7:51 PM
    True
  • w

    witty-crayon-22786

    09/24/2015, 7:51 PM
    and if you're GC'ing too much, you have bigger problems
  • b

    bored-art-40741

    09/24/2015, 7:51 PM
    yup
  • w

    witty-crayon-22786

    09/24/2015, 7:52 PM
    it's < 100 bytes/second
  • l

    limited-country-20626

    09/24/2015, 7:52 PM
    I read CMS would be better for this
  • b

    bored-art-40741

    09/24/2015, 7:54 PM
    Weird, does changing 
    compile.zinc
    jvm_options
    not invalidate all of the artifacts?
  • h

    happy-kitchen-89482

    09/24/2015, 7:54 PM
    Hmm, pants doesn't support PANTS_PROFILE outside its own repo.
  • f

    fancy-queen-20734

    09/24/2015, 7:54 PM
    no
  • b

    bored-art-40741

    09/24/2015, 7:54 PM
    It does not
  • b

    bored-art-40741

    09/24/2015, 7:54 PM
    We have a sweet script for this benjy
  • h

    happy-kitchen-89482

    09/24/2015, 7:54 PM
    where?
  • b

    bored-art-40741

    09/24/2015, 7:54 PM
    I've been meaning to plumb it
  • h

    happy-kitchen-89482

    09/24/2015, 7:55 PM
    not scripts/profiling/fs.sh?
  • b

    bored-art-40741

    09/24/2015, 7:55 PM
    Yeah
  • h

    happy-kitchen-89482

    09/24/2015, 7:55 PM
    I don't think that works?
  • h

    happy-kitchen-89482

    09/24/2015, 7:55 PM
    That just sets PANTS_PROFILE
  • b

    bored-art-40741

    09/24/2015, 7:55 PM
    It works but we need to pick up a little fix in 
    pants
1...451452453...2420Latest