Page MenuHomePhabricator

jmeador (James Meador)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Tuesday

  • Clear sailing ahead.

User Details

User Since
Jan 8 2016, 11:11 PM (563 w, 1 d)
Availability
Available

Recent Activity

Thu, Oct 22

jmeador added a comment to T13704: PHP2Shell.

A wild epriestley appears!

Thu, Oct 22, 6:33 PM · Security, Phacility

Nov 4 2022

jmeador added a comment to T13688: How do you reasonably plumb a 1/4" drain line into a 1-1/4" drain pipe?.

all my drains are belong to PVC

Nov 4 2022, 2:28 AM · Hardware

Nov 3 2022

jmeador added a comment to T13688: How do you reasonably plumb a 1/4" drain line into a 1-1/4" drain pipe?.

(You don't actually need a leak detector, I think there's enough absorbent material underneath the sink to last a long while before it damages the floor)

Nov 3 2022, 10:01 PM · Hardware
jmeador added a comment to T13688: How do you reasonably plumb a 1/4" drain line into a 1-1/4" drain pipe?.

Here is my bathroom sink. The right-most pipe coming out of the wall is the condensate drain from HVAC. The tailpiece is most definitely 1 1/4 and the rest of the piping is 1 1/2, at least to the naked eye.

Nov 3 2022, 9:41 PM · Hardware
jmeador added a comment to T13688: How do you reasonably plumb a 1/4" drain line into a 1-1/4" drain pipe?.

A few more thoughts:

Nov 3 2022, 8:31 PM · Hardware
jmeador added a comment to T13688: How do you reasonably plumb a 1/4" drain line into a 1-1/4" drain pipe?.

My terminology might be wrong throughout this but a couple thoughts:

Nov 3 2022, 4:51 PM · Hardware

May 3 2022

jmeador added a comment to T13676: Drydock may reclaim recently-used resources.

fill in the details a bit.

May 3 2022, 7:06 PM · Drydock
jmeador added a comment to T13676: Drydock may reclaim recently-used resources.

(one orthogonal bug I found is that bin/drydock lease discards any blueprints provided in an attributes JSON)

May 3 2022, 6:42 PM · Drydock

Sep 24 2019

jmeador added a comment to T13418: "arc liberate" incorrectly includes subdirectories of "src/extensions/" in library map.

I actually tried to use the extensions directory to hide Composer's vendored files and ran into this same issue.

Sep 24 2019, 4:33 PM · libphutil

Aug 24 2018

jmeador added a comment to T13177: ServiceProfiler integration should be a configurable part of Future, not hard-coded into subclasses (HTTPSFuture, ExecFuture).

We actually just ran into a situation where this would be extremely useful. We're adding a rudimentary start/end time tracing listener so we can keep tabs on our service calls. That listener uploads the data to an endpoint using an HTTPSFuture, but we want to configure it to not show up in the default echoListener (or to not be profiled at all).

Aug 24 2018, 6:28 PM · Infrastructure

Aug 22 2018

jmeador added a comment to T13187: Plans: 2018 Week 34 Bonus Content.

See PHI757. Leaving inline comments on the right hand side of files not touched by diff B in an interdiff between A and B raises an error.

This is at least somewhat involved. Related behavior:

  • In the standalone view, you can leave comments on either side -- and they swap sides!
  • Deleting comments from either side raises a JS error ("TypeError: null is not an object (evaluating 'templates.r')"). (Actually, this looks like it applies to all standalone deletion?)
Aug 22 2018, 5:46 AM · Plans

Jul 18 2018

jmeador added a comment to T13168: epriestley new laptop / not reading documentation setup issues.
  • Install XCode. Find the secret "Install Command Line Tools" option inside the Hidden Chamber of XCode and click it.

xcode-select --install should be possible without a full XCode installation.

Jul 18 2018, 5:37 PM · Phacility, Setup

Apr 24 2018

jmeador added a comment to T13130: Plans: 2018 Week 17/18 Bonus Content.

Offhand, two possible issues with that:

  • At least one install has an enormous number of packages (tens of thousands? Hundreds of thousands? Seventy-two billion?) and the minimum cost for a package is much, much smaller than the minimum cost of a Herald rule. If packages actually created Herald rules this install would probably explode.

13,636.

Apr 24 2018, 5:45 AM · Plans

Mar 29 2018

jmeador added a comment to T13114: Plans: 2018 Week 13 Bonus Content.

See PHI496. Multiple users have now reached the maximum level ("Heavy Wizard") on their Support characters, and we're overdue for an expansion pack to add new adventure content.

Mar 29 2018, 8:18 AM · Plans

Mar 16 2018

jmeador awarded T13107: Git, Slowly and Painfully a The World Burns token.
Mar 16 2018, 7:34 AM · Guides

Nov 7 2017

jmeador added a comment to T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential.

The harbormaster builds that we have running on revision creation can sometimes be rather slow; they're builds in our primary CI system, but with less of a priority than jobs in the submit queue. This will result in 10-30 minute delays for every revision in our monorepo. I understand the preference for only adding configuration settings when absolutely necessary, but since this may result in a significant time sink for any installs with expensive builds, is it worth keeping something around like differential.initial-state?

Nov 7 2017, 7:31 AM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential

Jul 19 2017

jmeador added a comment to T11823: Split RefCursor into a "name" table and a "ref" table so ref names have stable PHIDs.

In the refcursor table, our current autoincrement is at 312,036,016. This is only 7% of the way to int unsigned, but the fact that we've made it to 7% in 18 months might suggest that column should be BIGINT, unless there are performance impacts of that.

Jul 19 2017, 11:09 PM · Customer Impact, Diffusion, Bug Report

Jul 12 2017

jmeador added a comment to T12792: Consider an option to disable all popup notifications.

+1, we've also had a handful of users request this as a per-user preference.

Jul 12 2017, 6:11 PM · Feature Request, Customer Impact, Notifications

Jul 11 2017

jmeador created Image Macro "make-it-rain".
Jul 11 2017, 6:42 PM

Jul 6 2017

jmeador added a comment to T12896: It isn’t obvious when repository observation breaks.

This information may be entirely useless, but it is somewhat related. We have a large number of repositories, with multiple clusters so this affects us in a major way.

Jul 6 2017, 11:35 PM · Diffusion, Feature Request

Jul 3 2017

jmeador added a comment to T12893: Instance reports consistent bad repository device versions for clustered, observed Git repositories.

If you go through the process manually with the web UI, are the versions OK after initial creation? After adding an observed URI?

Jul 3 2017, 7:16 PM · Restricted Project, Clusters, Diffusion, Customer Impact

May 23 2017

jmeador added a comment to F4948659: Screen Shot 2017-05-09 at 1.40.25 PM.png.

screenshot on ambien

May 23 2017, 6:54 PM
jmeador updated subscribers of F4948659: Screen Shot 2017-05-09 at 1.40.25 PM.png.

@bizrad6

May 23 2017, 6:51 PM

May 17 2017

jmeador added a comment to T8909: Provide ability to hide all inline or done comments.

Do you bucket style guide violations with lint issues? While I obviously can't post internal reviews, I can give you a hand-waving indication that a non-zero percentage of comments are style-related things that aren't caught by linters.

May 17 2017, 11:17 PM · Inline Comments, Prioritized, Restricted Project, Restricted Project, Differential, FreeBSD

Apr 21 2017

jmeador added a comment to T3462: In Differential, expose "featurename (branched from master)" via Conduit.

That was unclear: we can parameterize on lots of things, just not the commit hash, which is conceivably what you want us to parameterize on.

Apr 21 2017, 5:29 PM · Differential, Arcanist
jmeador added a comment to T3462: In Differential, expose "featurename (branched from master)" via Conduit.

I have a related question regarding this quote:

Apr 21 2017, 5:26 PM · Differential, Arcanist

Apr 20 2017

jmeador added a comment to T12604: Pager hard to notice in Diffusion.

I agree, pagination way below the fold is rather inconspicuous

Apr 20 2017, 5:59 PM · Design, Bug Report, Diffusion
jmeador added a comment to T12604: Pager hard to notice in Diffusion.

Bystander input: try page 3: https://phabricator.kde.org/source/krita/browse/master/libs/ui/?offset=200

Apr 20 2017, 5:57 PM · Design, Bug Report, Diffusion

Apr 10 2017

jmeador moved T12244: Provide a crude way to selectively disable reads and writes of repository devices by setting flags on Almanac bindings from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Apr 10 2017, 10:33 PM · Restricted Project, Almanac

Mar 27 2017

jmeador added a project to T12244: Provide a crude way to selectively disable reads and writes of repository devices by setting flags on Almanac bindings: Restricted Project.
Mar 27 2017, 9:53 PM · Restricted Project, Almanac
jmeador added a project to T10883: Allow repository cluster nodes to be read-only: Restricted Project.
Mar 27 2017, 9:52 PM · Restricted Project, Diffusion, Clusters

Mar 24 2017

jmeador added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

FWIW, All packages in our monorepo have weak dominion.

Mar 24 2017, 9:56 PM · Owners, Audit, Restricted Project, Differential
jmeador added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

How do we envision these rules applying to the following situation?

Mar 24 2017, 9:12 PM · Owners, Audit, Restricted Project, Differential

Mar 14 2017

jmeador added a comment to T12392: Instance reporting that synchronizeWorkingCopyBeforeRead() effectively fails.

Do git ls-remote (which, curiously, seems to be significantly faster than git fetch even when git fetch is a no-op).

Mar 14 2017, 8:41 PM · Bug Report, Clusters, Diffusion

Mar 9 2017

jmeador added a comment to T12372: Long list of reviewers breaks to: field when metamta.one-mail-per-recipient set to false.

The default -- PhabricatorMailImplementationPHPMailerLiteAdapter

Mar 9 2017, 5:27 PM · Mail, Bug Report

Feb 24 2017

jmeador added a comment to T12002: Set header for CURL query for the Notification server status to HTTP:Upgrade = websocket.

@pouyana Here's a workaround for you. In your notification.servers Phabricator configuration, add path: "/ws/" to the client server. Then, modify your apache configuration like so:

Feb 24 2017, 10:53 PM · Bug Report

Feb 14 2017

jmeador added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

We could also implement "Mute Thread" in Phabricator, but I assume no one would be willing to click twice to mute a thread.

Feb 14 2017, 6:57 PM · Restricted Project, Mail, Feature Request

Jan 27 2017

jmeador added a comment to T11903: Please provide diff ids as an attachment in differential.search.

This has come up again with a slightly different use case. We want to intelligently purge tags from our staging areas. To do this, we examine refs/tags/* in the staging area, and parse out the diff IDs. It would be great if we could do this from differential.revision.search, but the attachments/constraints don't support it. The current working approach is differential.querydiffs with the IDs, parse the revision IDs out, then differential.query to get the revision statuses and their associated diffs list. This approach allows us to make significantly less queries than examining *all* closed/abandoned revisions, then calling the frozen/deprecated differential.query for the diff list.

Jan 27 2017, 11:33 PM · Restricted Project, Conduit, Feature Request

Jan 13 2017

jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

Awesome. Things appear to be working now. I suppose I can strip out all the overrides that the most recent commit stripped out. Sweetness.

Jan 13 2017, 11:50 PM · Installing & Upgrading, Differential
jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

so if isFieldEditable and isTemplateField are both false, will the field still show up in getcommitmessage?

Jan 13 2017, 11:43 PM · Installing & Upgrading, Differential
jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

giphy.gif (248×500 px, 477 KB)

Jan 13 2017, 5:09 PM · Installing & Upgrading, Differential
jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

Solid.

Jan 13 2017, 5:05 PM · Installing & Upgrading, Differential
jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

Awesome. What do you think the time delta will be between release promotion and the fix you have being merged to master?

Jan 13 2017, 5:03 PM · Installing & Upgrading, Differential

Jan 12 2017

jmeador added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

shouldAppearInCommitMessage() is marked as obsolete, but still seems necessary for me to implement a CustomField that behaves as follows:

Jan 12 2017, 12:32 AM · Installing & Upgrading, Differential

Nov 22 2016

jmeador created T11910: Cannot unset Build Plan Step's "Wait For Message" option.
Nov 22 2016, 8:55 PM · Custom Fields, Harbormaster, Bug Report

Aug 12 2016

jmeador added a comment to T11458: Make daemons less aggressive about cycling connections.

After merging with upstream, we were able to increase our taskmasters up to 32, and still not come anywhere close to the number of connections that were previously used. So far, so good on our end.

Aug 12 2016, 6:26 PM · Restricted Project, Daemons