This will require us to leave login credentials on the server that is pulling the file, no? I was more interested in some limiting the scope of access to a single project -- thereby something like <some_access_key>.

Yeah, my problem is explaining to users what they should do, no so much with the actual blocking configuration. Although, if they do arc-diff and the only problem is my landing script not working, than I can special case it on my end instead of teaching them things.

(That doesn't solve the "engineers don't know how to initialize a new Git repository with git commands" problem, but I think that one's a bit more involved.)

I'm assuming the "empty" issue is that you have a rule like this?

The problem I'm facing is that users just don't know how to push code to a new branch; They expect to git clone a repository that already "works"; Commands like git remote or git push with any arguments are simply beyond the git-fu of some engineers.

If the major immediate issue is Herald disallowing branch creation with git push, we could add a Herald condition like "Repository is empty".

Another hitch when creating a git repo, is that it has no branches, which is confusing for less-advanced users, and very confusing if you otherwise dissallow git push; GH solves this by adding a README file to create a branch with a commit.
I think a more powerful approach is to support some kind of "templates", so that we can create a new "Android App" repo or "COM Module" repo directly, similar to how IDEs have project templates.

Here's what I'm planning to do for imports:

Also, reindexing is required after switching to innodb.

Does InnoDB FULLTEXT respect ft_boolean_syntax? If no: is there another similar option? If no: we need to parse queries ourselves and submit the +A +B version to the backend (see also T10642).

That's some pretty awesome support for an unsupported prototype. Kudos!

Hrrm -- what is max_allowed_packet set to in MySQL?

How about this， it's solved or not?

This helped! Thanks for another fast fix.

I have a patch to Arc that will use a non-printable control character (INFORMATION SEPARATOR THREE) instead of | for the separator.

T11747 wants a more advanced version of this (reviewing diffs between rendered documents), although the specific document type it wants (IPython notebooks) is likely very complex to render.

I'm going to merge this into T5698. That isn't quite the same task, but would be the first step toward this.

I believe this should now be fixed in HEAD of master. It should promote to stable within 24 hours.

T11744 has a vaguely-related special case where "split + count" was unreasonably slow for very large inputs.

D16683 makes us bail early if the combined length of the old and new lines is larger than 1280 bytes, which is a crude test for approximately two reasonable-length lines of emoji source code.

It looks like this is largely just PHP being PHP. To handle UTF8 combining characters correctly, we're producing a list of strings like this internally as the atoms of comparison:

This should now be fixed in HEAD of master.

In T3353#196761, @chad wrote:

Can you do that today with view and edit policy?

I'll look at this tomorrow but can probably figure it out from this. Initial guess is that we're doing this (which is O(big constant * string length)):

Thanks also for the tip about SIGHUP to get a backtrace! Had not known that, and it looks extremely useful.

Just my 2c, but we've seen "table needs repair" on such a regular interval over the past 2-3 years of using Phabricator that our startup script includes a call to MySQL to automatically run "REPAIR TABLE" on startup. It's actually part of the Docker image because a broken MyISAM table will prevent bin/storage upgrade from running:

Sweet -- thanks for the fast fix! I'll try applying it locally to clear out the affected tasks we have, and to make sure any new ones don't get me paged tonight. :-)

The root issue was that we would compute a ProseEditDistanceMatrix (roughly, a Levenshtein distance matrix) for large inputs, but this is O(N*M) in the size of the inputs and both of these inputs had ~2,200 "atoms" at the top prose diff level, so the matrix had ~5M elements.

Oh, one diagnostic tool that I didn't mention is that you can send SIGHUP to a running bin/* command to get a stacktrace of what it's doing dumped to /tmp/phabricator_backtrace_<pid> (usually). This one looks like it is a smoking gun:

Thank you! Not every day someone invites me to cause a crash on their production server. :-D

Awesome, thanks! I'll dig into it and update you once I get somewhere.

Just made P2011, and I think that's a repro -- the "Show Details" link hangs.

Yeah, just take a shot at making a paste here (secure.phabricator.com) with the same degenerate behavior. If you can get us crashing I suspect I can get a fix in about 5 minutes through the power of adding var_dump() all over the place on the production server. If not I have a few ideas but there's a lot of code that could be causing the problem and that trace isn't an obvious smoking gun to me.

Nope -- pstree shows no subprocesses when the worker is stuck.

One other quick diagnostic is: when the worker is stuck, is there a pygmentize subprocess running?

Meaning here on secure.phabricator.com? Sure -- no guarantee it'll work, but I have some guesses I can throw at it.

Can you attempt to create a paste on this install which reproduces the issue? I have a few ideas about what's up, but that's almost certainly the fastest path to a fix if it works.

I don't think we should change this.

From the Twitterverse, we should look at making payments as super easy as possible, especially if ur deadbeat.

actually will follow up in T7607

We'll accept this as a bug report if you describe the conditions which lead to a newline after "Summary:". The expected behavior is that there is no newline.

Oh, sorry, apparently @chad got this right then and I misunderstood.

I should probably have filed this as a bug report. The issue is that the commit message populated by arcanist looks different, in a surprising way, to the way the Summary information is displayed on the Differential page.

To be clear, I started with the Side-by-Side view, which I guess is the default. And I can't return directly to the side-by-side view with the original amount of context after "Show All Context".