jmeador (James Meador)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Jan 8 2016, 11:11 PM (127 w, 5 d)
Availability
Available

Recent Activity

Thu, Jun 7

jmeador accepted D19474: Fix an issue where scrolling down, then up, then down fails to show changeset header in Differential.
Thu, Jun 7, 5:30 PM
jmeador accepted D19475: In "arc patch", update submodules slightly later.
Thu, Jun 7, 5:29 PM
jmeador accepted D19476: Include owners packages in the MailableFunction datasource.
Thu, Jun 7, 5:28 PM
jmeador awarded D19476: Include owners packages in the MailableFunction datasource a Cup of Joe token.
Thu, Jun 7, 5:28 PM

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 22 2018

jmeador added a comment to rPc8583b016daf: When workflow dialog buttons are clicked, disable the button.

Thanks, D19248.

Mar 22 2018, 6:32 PM
jmeador added inline comments to rPc8583b016daf: When workflow dialog buttons are clicked, disable the button.
Mar 22 2018, 7:23 AM

Mar 16 2018

jmeador added a comment to rP667955b8aeca: Clarify that a PHUITagView API argument is boolean, not a value in "millimeters".

This is a buff to API humor

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

Mar 2 2018

jmeador awarded D19168: Reduce the cost of generating default user profile images a Burninate token.
Mar 2 2018, 1:26 AM

Feb 8 2018

jmeador added inline comments to D18176: Correct a datasource issue when viewing repository URIs in "Manage Repository".
Feb 8 2018, 8:21 PM

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
jmeador added inline comments to D18651: Make "arc diff" sort of detect dependent revisions.
Nov 7 2017, 7:15 AM

Aug 29 2017

jmeador accepted D18493: Make legacy revision statuses from "differential.query" have type "string" again.
Aug 29 2017, 8:04 PM

Jul 21 2017

jmeador accepted D18264: Reduce the strength of "arc executing on arc" from an error to a warning.
Jul 21 2017, 7:05 PM

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

Jun 14 2017

jmeador accepted D18124: Queue a worker task to send mail only after committing the mail transaction.
Jun 14 2017, 7:26 PM

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 26 2017

jmeador accepted D17793: Add an Owners Package hovercard.

Really a very very beautiful hovercard. It's going to be great. Fantastic job. Really really proud of this hovercard.

Apr 26 2017, 6:58 AM

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 & Planning, 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 & Planning, 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 28 2017

jmeador accepted D17570: Correct spelling of "Dasbhoard".

pollice lock us in prisun.

Mar 28 2017, 5:02 PM

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 19 2017

jmeador awarded D17229: Make Differential inline events actually trigger comment preview refreshes a 100 token.
Jan 19 2017, 7:56 PM

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.

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 30 2016

jmeador awarded D16472: Add a summary view of all repository errors to the repository cluster screen a 100 token.
Aug 30 2016, 4:16 PM

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

Aug 11 2016

jmeador awarded D16391: When already running as the daemon user, don't "sudo" daemon commands a 100 token.
Aug 11 2016, 11:12 PM