Page MenuHomePhabricator

jboning (Josiah Boning)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Oct 16 2012, 9:30 PM (595 w, 6 d)
Availability
Available

Recent Activity

Jan 23 2018

jboning added a comment to T13038: Meltdown and Spectre Speculative Branch Prediction Attacks.

Wow, I didn't mean for my two cents' worth of snark to cause such a stir! I do buy the argument that Phabricator isn't impacted by Meltdown/Spectre ("At least today, Herald rules are insufficiently expressive to allow an attacker to encode a speculative execution cache timing side channel attack.").

Jan 23 2018, 6:45 PM · Security

Jan 18 2018

jboning added a comment to T13038: Meltdown and Spectre Speculative Branch Prediction Attacks.

Among other advanced capabilities, PHP instructions execute too slowly to allow a runtime program to distinguish between L1 cache access and main memory access.

Jan 18 2018, 12:05 AM · Security

Sep 29 2017

jboning added a comment to D18663: Mark Owners package reviewers which own nothing in the current diff.

I wonder if an "Owns $n Changed Paths" note on currently-relevant packages would be useful, or just distracting.

Sep 29 2017, 9:10 PM

Sep 26 2017

jboning edited P2075 (An Untitled Masterwork).
Sep 26 2017, 2:11 AM
jboning created P2075 (An Untitled Masterwork).
Sep 26 2017, 2:10 AM

Jul 13 2017

jboning added a comment to T12920: Retracting requested changes can leave revision in "Needs Revision" state.

That sounds less surprising to me, in the sense that it's not an explicit reversal of having requested changes. I can imagine arguments in both directions for the resign case.

Jul 13 2017, 8:32 PM · Unknown Object (Project), Differential, Bug Report
jboning created T12920: Retracting requested changes can leave revision in "Needs Revision" state.
Jul 13 2017, 6:57 PM · Unknown Object (Project), Differential, Bug Report

Jun 19 2017

jboning added a comment to D18130: Use visibility hidden on remarkup checkboxes.

Much nicer! Thank you!

Jun 19 2017, 5:49 PM

Jun 17 2017

jboning created T12850: Checklists with dangling checkboxes overflow onto subsequent text.
Jun 17 2017, 1:49 AM · Remarkup, Bug Report

Jun 14 2017

jboning updated subscribers of T11664: Add option to allow owner packages to be added as reviewers to diffs, when the diff author is in the owner package.

I'm not totally sure about how/why users are expecting the behavior in the third case.

Jun 14 2017, 10:13 PM · Unknown Object (Project), Feature Request
jboning added a comment to T11664: Add option to allow owner packages to be added as reviewers to diffs, when the diff author is in the owner package.

When you put it that way, it does seem backwards.

Jun 14 2017, 9:31 PM · Unknown Object (Project), Feature Request
jboning added a project to T11664: Add option to allow owner packages to be added as reviewers to diffs, when the diff author is in the owner package: Unknown Object (Project).

We're interested in this option, since we find that users are often surprised by the existing behaviour. I think this would make sense as a per-package setting, rather than a global one.

Jun 14 2017, 5:57 PM · Unknown Object (Project), Feature Request

May 19 2017

jboning added a comment to T7664: Moving a task on a workboard is very slow (and sometimes times out with a fatal error).

Super! We'll report results when we've got it deployed.

May 19 2017, 11:45 PM · Unknown Object (Project), Workboards
jboning awarded D17959: Disperse task subpriorities in blocks a Love token.
May 19 2017, 5:27 AM
jboning added a comment to T7664: Moving a task on a workboard is very slow (and sometimes times out with a fatal error).

I'm not 100% sure, but from reading the subpriority movement code, it seems like our bell-curve distribution is self-perpetuating--when a block of tasks are moved, most of the time the next task is still relatively close, so $delta is small. I think this might be resolved by a big one-time readjustment to distribute tasks evenly throughout the space. I don't have an intuition on whether this distribution will redevelop over time, or if it's an artifact from the time before D12166.

May 19 2017, 4:57 AM · Unknown Object (Project), Workboards
jboning added a comment to T7664: Moving a task on a workboard is very slow (and sometimes times out with a fatal error).

It looks like even after D12166, the subpriority-assignment logic doesn't do a great job of spreading our subpriorities. P2053 shows that very many of our tasks have subpriorities near 0 (in fact, the vast majority of our tasks; a separate query without the > 1 limit indicates that only about 4k tasks have subpriority neighborhoods of their very own.)

May 19 2017, 3:08 AM · Unknown Object (Project), Workboards
jboning created P2053 (An Untitled Masterwork).
May 19 2017, 2:56 AM

Mar 23 2017

jboning added a project to T12443: Applying fulltext limits first causes missing results: Unknown Object (Project).
Mar 23 2017, 1:33 AM · Unknown Object (Project), Search, Bug Report

Mar 17 2017

epriestley awarded T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection a Mountain of Wealth token.
Mar 17 2017, 3:01 AM · Unknown Object (Project), Bug Report
jboning added a project to T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection: Unknown Object (Project).
Mar 17 2017, 1:02 AM · Unknown Object (Project), Bug Report
jboning created T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection.
Mar 17 2017, 12:43 AM · Unknown Object (Project), Bug Report

Mar 9 2017

jboning added a project to T12371: Support `viewerprojects()` and implement `viewerpackages()` for "Auditors" in Diffusion: Unknown Object (Project).
Mar 9 2017, 1:15 AM · Unknown Object (Project), Audit, Differential

Mar 8 2017

jboning added a comment to T12371: Support `viewerprojects()` and implement `viewerpackages()` for "Auditors" in Diffusion.

Cool, this would be great!

Mar 8 2017, 11:09 PM · Unknown Object (Project), Audit, Differential
jboning added a comment to T9430: "Needs Audit" no longer excludes author commits.

More generally: I think your decision to not try to migrate saved queries makes a lot of sense. In lieu of that, I'd like to cast a vote for more prominent mention in the changelog. It would be great just to see a note that some queries would break and some hints on identifying and replacing/repairing the broken ones.

Mar 8 2017, 10:44 PM · Restricted Project, Restricted Project, Audit
jboning added a comment to T9430: "Needs Audit" no longer excludes author commits.

I've hit the same problem that @cspeckmim ran into. As best I can tell, in the new system there's no way to quite replicate the behaviour of the old 'needs' or 'problem' audit queries. The closest attempts I've come up with so far are:

  • query for Audit Status = Audit Required/Needs Verification and Responsible Users = viewer(). But that includes results where the user is the diff author, not auditor.
  • query for Audit Status = Audit Required/Needs Verification and Auditors = viewer(). But that misses results where user is a member of a package or project auditor.

I think this would work if I could query for Auditors = viewer()/projects(viewer())/packages(viewer()), but it seems the functions aren't composable (at least in the UI).

Mar 8 2017, 10:22 PM · Restricted Project, Restricted Project, Audit

Jan 10 2017

jboning added a comment to T12087: Clusterizing a repository onto a service with multiple machines results in ambiguous leader.

Hurray, thank you!

Jan 10 2017, 9:18 PM · Unknown Object (Project), Bug Report
jboning added a comment to T12090: `bin/hunks migrate` fails on differential_hunks row with NULL changes field.

Aha, of course. Thank you!!

Jan 10 2017, 7:55 PM · Unknown Object (Project), Bug Report
jboning updated the task description for T12090: `bin/hunks migrate` fails on differential_hunks row with NULL changes field.
Jan 10 2017, 7:55 PM · Unknown Object (Project), Bug Report
jboning added a project to T12090: `bin/hunks migrate` fails on differential_hunks row with NULL changes field: Unknown Object (Project).
Jan 10 2017, 7:50 PM · Unknown Object (Project), Bug Report
jboning added a comment to T12090: `bin/hunks migrate` fails on differential_hunks row with NULL changes field.

I'd be happy just to have a good workaround, since I don't think we're particularly attached to these mostly-contentless rows. My ideas for possibilities so far are:

  • Delete the offending differential_hunks rows manually. I worry that this might cause some inconsistency.
  • bin/destroy the revisions containing the offending hunks. This seems a bit heavy-handed though.
Jan 10 2017, 7:50 PM · Unknown Object (Project), Bug Report
jboning created T12090: `bin/hunks migrate` fails on differential_hunks row with NULL changes field.
Jan 10 2017, 7:48 PM · Unknown Object (Project), Bug Report
jboning renamed T12087: Clusterizing a repository onto a service with multiple machines results in ambiguous leader from Clusterizing a repository onto a service with multiple machines results in non-working state to Clusterizing a repository onto a service with multiple machines results in ambiguous leader.
Jan 10 2017, 12:51 AM · Unknown Object (Project), Bug Report
jboning created T12087: Clusterizing a repository onto a service with multiple machines results in ambiguous leader.
Jan 10 2017, 12:51 AM · Unknown Object (Project), Bug Report

May 21 2016

jboning created T11004: git URI normalization is not reliable.
May 21 2016, 12:39 AM · Diffusion, Bug Report

Dec 4 2015

jboning added a comment to T9898: `git branch --contains X` has cost in the realm of O(commits * branches).

I think we're going to work around this by changing our base config value to git:upstream, git:merge-base(origin/master) and decreeing that if you want to do this sort of dependent feature thing, you have to explicitly set child-feature's upstream to feature. arc feature does that automatically, so we might just encourage people to start using that.

Dec 4 2015, 8:29 PM · Restricted Project, Unknown Object (Project), Arcanist
jboning added a comment to T9898: `git branch --contains X` has cost in the realm of O(commits * branches).

pretty big rough edges for the workflows it intends to enable

What are these, specifically? Is this entire task moot?

Dec 4 2015, 8:05 PM · Restricted Project, Unknown Object (Project), Arcanist

Oct 22 2015

jboning created T9611: arc upgrade doesn't check for git failures.
Oct 22 2015, 3:20 AM · Unknown Object (Project)