#general
- b
bored-art-40741
07/23/2015, 2:51 AMOne gross problem is if the cache server is overloaded and errors on a cache write. It's not fatal enough to break the build, but you silently have a less populated remote cache for users - e
enough-analyst-54434
07/23/2015, 2:54 AM@witty-crayon-22786: I tried to answer your question about the nested Factory classes in https://rbcommons.com/s/twitter/r/2511/, interested in what you think of my spiel - w
witty-crayon-22786
07/23/2015, 2:57 AMah, thanks... looking - w
witty-crayon-22786
07/23/2015, 3:03 AM@enough-analyst-54434: i think that generally makes sense... but in tests the options system is mocked out, so it's impossible to overdepend - w
witty-crayon-22786
07/23/2015, 3:03 AMwill shipit though. - e
enough-analyst-54434
07/23/2015, 3:03 AMThe depend is not my worry, its the comprehension - e
enough-analyst-54434
07/23/2015, 3:03 AMinstead of an obvious parameter, there a translation from registration classmethod string, to underscore param name ... - e
enough-analyst-54434
07/23/2015, 3:04 AMAnd no natural spot to hand param docs, etc - e
enough-analyst-54434
07/23/2015, 3:04 AMs/hand/hang/ - w
witty-crayon-22786
07/23/2015, 3:04 AMexcept on the options - e
enough-analyst-54434
07/23/2015, 3:04 AMwhich is great for the CLI user - e
enough-analyst-54434
07/23/2015, 3:04 AMshitty for the coder - e
enough-analyst-54434
07/23/2015, 3:04 AMAlthough clearly the dup is crappy too! - e
enough-analyst-54434
07/23/2015, 3:04 AMSo yeah - w
witty-crayon-22786
07/23/2015, 3:05 AMfunctions taking explicit inputs rather than reaching out for global state are great - e
enough-analyst-54434
07/23/2015, 3:05 AMexactly - w
witty-crayon-22786
07/23/2015, 3:05 AMbut without enforcement, it's a convention that needs to be super obvious - w
witty-crayon-22786
07/23/2015, 3:05 AM*shrug - c
cold-lizard-88792
07/23/2015, 5:43 AMI talked today to devinfra guys from Airbnb, they currently use gradle for their java and a full build itakes them 15 mins they consider pants -- should they be pointed to http://pantsbuild.github.io/setup_repo.html? - e
enough-analyst-54434
07/23/2015, 5:48 AMProbably that and http://pantsbuild.github.io/howto_contribute.html#join-the-conversation - e
enough-analyst-54434
07/23/2015, 5:54 AMI just added airbnb.com to the pantsbuild slack team in case they have questions - c
cold-lizard-88792
07/23/2015, 6:35 AMthanks! - a
astonishing-jelly-60479
07/23/2015, 7:31 PMhey, is it cool to ask questions in here? i've run into what i think is a bug in the interaction between python thrift generation and the python-eval/eval-targets phase - e
enough-analyst-54434
07/23/2015, 7:31 PMIt is, but we'll fork an issue if the answer is unclear - e
enough-analyst-54434
07/23/2015, 7:31 PMWhat's up? - a
astonishing-jelly-60479
07/23/2015, 7:33 PMso we have a .thrift file, set up as a python_thrift_library(), which only specifies a service but includes other thrift files that have structs, enums, etc. - e
enough-analyst-54434
07/23/2015, 7:33 PMdoes the target depend on the targets that own those other thrift files? - a
astonishing-jelly-60479
07/23/2015, 7:33 PMin our current setup things work fine as everything gets bundled in the pex as a wheel - a
astonishing-jelly-60479
07/23/2015, 7:33 PMthey're all sources of one target - e
enough-analyst-54434
07/23/2015, 7:34 PMk