Page MenuHomePhabricator
Feed Advanced Search

Jan 6 2015

joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Restricted Differential Revision.
Jan 6 2015, 10:21 AM · Infrastructure
joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Restricted Differential Revision.
Jan 6 2015, 10:20 AM · Infrastructure
joshuaspence added a revision to T6822: Fix methods which override other methods with the wrong visibility: Restricted Differential Revision.
Jan 6 2015, 10:15 AM · Infrastructure

Jan 5 2015

epriestley closed T6485: Feedback for Schema Adjustments as Resolved.

@kylejao, I don't know why mysql would hang indefinitely on an ALTER TABLE statement. I can't reproduce that, and it seems like an issue with MySQL, not Phabricator. I don't know if there are more granular tools available to help understand why ALTER TABLE isn't working or not. You could try googling or looking on StackOverflow. It looks like Percona might have some tools, although I haven't used them.

Jan 5 2015, 5:58 PM · Infrastructure

Jan 4 2015

epriestley added a comment to T6861: PHP sort function is unpredictable to human minds when dealing with mixed types.

(It's technically deterministic, I suppose, just very unpredictable.)

Jan 4 2015, 11:34 PM · Infrastructure
epriestley added a comment to T6861: PHP sort function is unpredictable to human minds when dealing with mixed types.

Here's an example which shows how zany things are:

Jan 4 2015, 9:32 PM · Infrastructure
joshuaspence added a revision to T6861: PHP sort function is unpredictable to human minds when dealing with mixed types: Restricted Differential Revision.
Jan 4 2015, 9:19 PM · Infrastructure
joshuaspence created T6861: PHP sort function is unpredictable to human minds when dealing with mixed types.
Jan 4 2015, 9:18 PM · Infrastructure
epriestley added a comment to T6859: It is difficult to keep track of used PhabricatorEdgeType IDs.

Some vague thinking in this vein:

Jan 4 2015, 4:59 PM · Infrastructure
joshuaspence added a project to T6859: It is difficult to keep track of used PhabricatorEdgeType IDs: Infrastructure.
Jan 4 2015, 11:45 AM · Infrastructure

Dec 29 2014

epriestley added a revision to T6822: Fix methods which override other methods with the wrong visibility: Restricted Differential Revision.
Dec 29 2014, 5:17 PM · Infrastructure
epriestley created T6822: Fix methods which override other methods with the wrong visibility.
Dec 29 2014, 4:53 PM · Infrastructure

Dec 19 2014

kylejao added a comment to T6485: Feedback for Schema Adjustments.

so is there anything I can do with this?

Dec 19 2014, 5:30 AM · Infrastructure

Dec 12 2014

qgil moved T6102: Improve typeahead behavior around exact matches from Backlog to Important on the Wikimedia board.
Dec 12 2014, 11:21 PM · Wikimedia, Support Impact, Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.

Still stuck..

Dec 12 2014, 6:34 AM · Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.

The machine shouldn't be slow. Let me wait longer. Thanks.

Dec 12 2014, 1:53 AM · Infrastructure
epriestley added a comment to T6485: Feedback for Schema Adjustments.

Looks like that table has 6.5GB of data in it, so it could take MySQL quite a while to ALTER it. I would not normally expect it to take 2 hours, but if you have a slow machine or disk maybe it's just legitimately taking that long.

Dec 12 2014, 1:45 AM · Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.
Dec 12 2014, 1:39 AM · Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.

The part of phabricator_repository:

phabricator_repository            6,551.8 MB   97.5%
    repository_branch                 0.0 MB    0.0%
    repository_badcommit              0.0 MB    0.0%
    edgedata                          0.0 MB    0.0%
    repository_summary                0.0 MB    0.0%
    repository_coverage               0.0 MB    0.0%
    repository_statusmessage          0.0 MB    0.0%
    repository_arcanistproject        0.0 MB    0.0%
    repository_symbol                 0.0 MB    0.0%
    repository_vcspassword            0.0 MB    0.0%
    repository_pushevent              0.0 MB    0.0%
    repository_mirror                 0.0 MB    0.0%
    repository_auditrequest           0.0 MB    0.0%
    repository_lintmessage            0.1 MB    0.0%
    repository                        0.1 MB    0.0%
    repository_pushlog                0.1 MB    0.0%
    repository_refcursor              0.3 MB    0.0%
    edge                              0.3 MB    0.0%
    repository_transaction            0.3 MB    0.0%
    repository_parents                0.4 MB    0.0%
    repository_commitdata             2.6 MB    0.0%
    repository_commit                 3.2 MB    0.0%
    repository_filesystem             5.0 MB    0.1%
    repository_pathchange           129.0 MB    1.9%
    repository_path               6,410.0 MB   95.4%
Dec 12 2014, 1:38 AM · Infrastructure
epriestley added a comment to T6485: Feedback for Schema Adjustments.

It depends on the size of the table.

Dec 12 2014, 1:36 AM · Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.

like 2 hours.
I'm running this command from MySQL. How long is it expected to finish?

Dec 12 2014, 1:35 AM · Infrastructure
epriestley added a comment to T6485: Feedback for Schema Adjustments.

Equivalently, if you run this command manually from MySQL, does it complete quickly?

Dec 12 2014, 1:05 AM · Infrastructure
epriestley added a comment to T6485: Feedback for Schema Adjustments.

If you have a large amount of repository data, the migration may take a very long time. How long is it hanging for?

Dec 12 2014, 1:05 AM · Infrastructure
kylejao added a comment to T6485: Feedback for Schema Adjustments.

Hi, my adjustment hangs. Wiith --trace I got

Dec 12 2014, 1:03 AM · Infrastructure

Dec 11 2014

btrahan reopened T6102: Improve typeahead behavior around exact matches as "Open".
Dec 11 2014, 6:15 PM · Wikimedia, Support Impact, Infrastructure
btrahan added a comment to T6102: Improve typeahead behavior around exact matches.

Oh, I thought this was about the global search. The existing patch only covers the global search in the upper right.

Dec 11 2014, 6:14 PM · Wikimedia, Support Impact, Infrastructure
epriestley added a comment to T6102: Improve typeahead behavior around exact matches.

I can reproduce it on the WMF install in tokenizers:

Dec 11 2014, 6:07 PM · Wikimedia, Support Impact, Infrastructure
btrahan added a comment to T6102: Improve typeahead behavior around exact matches.

When I type "Wikidata" on your install its the top result:

Dec 11 2014, 5:51 PM · Wikimedia, Support Impact, Infrastructure
qgil added a project to T6102: Improve typeahead behavior around exact matches: Wikimedia.

Wikimedia upgraded yesterday, and while typing "Wikidata" seems to bring "Wikidata" among the top five suggestions, it is not the top result, which makes us think that we can still run into scenarios like the one described here.

Dec 11 2014, 7:40 AM · Wikimedia, Support Impact, Infrastructure

Dec 8 2014

epriestley added a comment to T6710: Support "timeout" in vanilla MySQL connections.

Ugh. I guess we'd have to this thing, then:

Dec 8 2014, 10:27 PM · Infrastructure
richardvanvelzen added a comment to T6710: Support "timeout" in vanilla MySQL connections.

There seems to be no other way than setting the timeout via ini_set().

Dec 8 2014, 10:26 PM · Infrastructure
epriestley added a revision to T6710: Support "timeout" in vanilla MySQL connections: Restricted Differential Revision.
Dec 8 2014, 1:22 PM · Infrastructure
epriestley created T6710: Support "timeout" in vanilla MySQL connections.
Dec 8 2014, 1:21 PM · Infrastructure

Nov 24 2014

epriestley added a comment to T6615: Worker task table has some remaining awkward keys.

Let us know if you're still seeing issues with large queues after those patches, but I think the meat of this issue is fixed.

Nov 24 2014, 8:17 PM · Infrastructure, Daemons
epriestley renamed T6615: Worker task table has some remaining awkward keys from Selection query on task table is garbage with many unlocked tasks to Worker task table has some remaining awkward keys.
Nov 24 2014, 8:16 PM · Infrastructure, Daemons
epriestley added a comment to T6485: Feedback for Schema Adjustments.

We narrowed this down in IRC to the call to set_charset('utf8mb4') failing with this error:

Nov 24 2014, 4:06 PM · Infrastructure

Nov 20 2014

cburroughs added a comment to T6485: Feedback for Schema Adjustments.

Adjust has been run on our db and /config/database/ reports 'No Schema Issues'. However, if I try to create a ticket with a cute title like ? ? ? ? ? ? ? ? ? I get:

Nov 20 2014, 1:45 PM · Infrastructure

Nov 17 2014

btrahan closed T6102: Improve typeahead behavior around exact matches as Resolved.

Closed by commit rP81a13ed8bda8.

Nov 17 2014, 11:32 PM · Wikimedia, Support Impact, Infrastructure
btrahan added a revision to T6102: Improve typeahead behavior around exact matches: Restricted Differential Revision.
Nov 17 2014, 11:11 PM · Wikimedia, Support Impact, Infrastructure

Nov 15 2014

btrahan claimed T6102: Improve typeahead behavior around exact matches.
Nov 15 2014, 2:54 AM · Wikimedia, Support Impact, Infrastructure

Nov 6 2014

epriestley updated the task description for T6485: Feedback for Schema Adjustments.
Nov 6 2014, 10:30 PM · Infrastructure
epriestley created T6485: Feedback for Schema Adjustments.
Nov 6 2014, 10:04 PM · Infrastructure

Oct 3 2014

epriestley added a comment to T2794: Implement Phage (like Hypershell).

I want to build this some day but it isn't going to happen for a while and doesn't block SAAS.

Oct 3 2014, 4:44 AM · Phage, Infrastructure
epriestley lowered the priority of T2794: Implement Phage (like Hypershell) from Normal to Wishlist.
Oct 3 2014, 4:44 AM · Phage, Infrastructure
epriestley added a project to T6102: Improve typeahead behavior around exact matches: Support Impact.

Support Impact This is broken with no real workaround.

Oct 3 2014, 3:41 AM · Wikimedia, Support Impact, Infrastructure

Oct 2 2014

epriestley closed T6228: Error log spew after failing to set utf8mb4 charset as Resolved.

Closed by commit rPHUb7c05df35726.

Oct 2 2014, 4:50 PM · Infrastructure
epriestley added a comment to T6228: Error log spew after failing to set utf8mb4 charset.
  • Use Config → Database Status and Config → Database Issues from the web UI to sanity check things before applying the schema adjustments. It's expected that there will be a ton of warnings (around 3,000) but no errors.
      • The majority of these should be things which are currently utf8_general_ci or utf8_bin that are "expected" to be binary.
      • If there are errors, let us know.
    • Run bin/storage adjust to apply the changes, optionally adding --trace if you want more insight / better logging into what's going on.
      • If this is unable to apply any changes, let us know. (I've done one schema conversion to binary but it was a few patches ago and it might have some issues now.)
    • Config should now show no warnings.
    • Set $has_utf8mb4 = true; near line 329 of libphutil/src/aphront/storage/connection/mysql/AphrontBaseMySQLDatabaseConnection.php to actually disable the exception and fix the issue.
Oct 2 2014, 4:36 PM · Infrastructure
fabe added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

as mentioned this is a dev install just for me. i'd be happy to test with a mysql 5.1 if you like. doesen't matter if it completely breaks ;)

Oct 2 2014, 4:29 PM · Infrastructure
epriestley added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

They're merged and available, but you have to run a dangerous secret script to actually apply them and do the conversions. We'll make that part of bin/storage upgrade once it stabilizes.

Oct 2 2014, 4:27 PM · Infrastructure
fabe added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

Ah i see. I thought the changes were already merged. My dev installation i've made earlier today still has utf8_general_ci.

Oct 2 2014, 4:25 PM · Infrastructure
epriestley added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

T1191 converts the schemata to utf8mb4 where available, and binary where not available, and fixes the issue correctly (by permitting storage of 4-byte characters on all MySQL versions). It's running on this install (which is why emoji work here) but not ready for general use yet (we're still uncovering a handful of minor issues). You can find a great deal more discussion in that task. The issue isn't old versions of MySQL, it's our use of utf8 character sets (which are basically completely broken).

Oct 2 2014, 4:19 PM · Infrastructure
fabe added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

Upon importing the phabricator repo the import will stop at 99% because the last commit containing the unicode char can't be imported.
Error is the same that initially triggered the whole problem:

Oct 2 2014, 4:16 PM · Infrastructure
epriestley added a revision to T6228: Error log spew after failing to set utf8mb4 charset: Restricted Differential Revision.
Oct 2 2014, 3:48 PM · Infrastructure
epriestley added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

Thanks! I just wanted to make sure @ resolves it, since my versions of things don't reproduce it locally. I'll get a patch out.

Oct 2 2014, 3:46 PM · Infrastructure
fabe added a comment to T6228: Error log spew after failing to set utf8mb4 charset.

I've just set up a new dev phabricator today using a mysql 5.1.41
Error logs are flooded with (for every query): ERROR 2: mysqli::set_charset(): Error executing query at [.....libphutil/src/aphront/storage/connection/mysql/AphrontMySQLiDatabaseConnection.php:70]

Oct 2 2014, 3:45 PM · Infrastructure
epriestley created T6228: Error log spew after failing to set utf8mb4 charset.
Oct 2 2014, 3:37 PM · Infrastructure

Oct 1 2014

epriestley closed T6095: Schema changes required to support utf8mb4 as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6128: `phabricator_dashboard.edge` has wrong column type for `type` column as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6128: `phabricator_dashboard.edge` has wrong column type for `type` column, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6130: `phabricator_conpherence.edge` has wrong `type` type as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6130: `phabricator_conpherence.edge` has wrong `type` type, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6135: `phabricator_fund.edge` has wrong type column as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6135: `phabricator_fund.edge` has wrong type column, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:39 PM · Infrastructure
epriestley closed T6137: `phabricator_legalpad.edge` has wrong `type` column type as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6137: `phabricator_legalpad.edge` has wrong `type` column type, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6138: `phabricator_slowvote.edge` has wrong `type` column type, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6138: `phabricator_slowvote.edge` has wrong `type` column type as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6149: `phabricator2_nuance.edge` has wrong `type` column type, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6149: `phabricator2_nuance.edge` has wrong `type` column type as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6150: `phabricator2_nuance.nuance_requestorsource` has overlong key, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6150: `phabricator2_nuance.nuance_requestorsource` has overlong key as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6151: `phabricator_passphrase.edge` has wrong `type` column type, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6151: `phabricator_passphrase.edge` has wrong `type` column type as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6098: Key `key_provider` on `phabricator_auth.auth_providerconfig` is too long, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6098: Key `key_provider` on `phabricator_auth.auth_providerconfig` is too long as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6099: Key `key_token` on `phabricator_auth.auth_temporarytoken` is too long as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6099: Key `key_token` on `phabricator_auth.auth_temporarytoken` is too long, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6105: `phabricator_repository.repository_commitdata` authorName field should be longtext, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6129: `phabricator_conpherence.conpherence_transaction_comment` has overlong `key_draft` key, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6129: `phabricator_conpherence.conpherence_transaction_comment` has overlong `key_draft` key as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6133: `phabricator_file.file_imagemacro` has a too-long key on `name`, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6133: `phabricator_file.file_imagemacro` has a too-long key on `name` as Resolved.
Oct 1 2014, 3:38 PM · Infrastructure
epriestley closed T6134: `phabricator_file.file_transformedfile` has too-long key as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6134: `phabricator_file.file_transformedfile` has too-long key, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6146: `phabricator2_meta_data.patch_status` has overlong key, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6146: `phabricator2_meta_data.patch_status` has overlong key as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6147: `phabricator_metamta.metamta_mail` has overlong `status` key as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6147: `phabricator_metamta.metamta_mail` has overlong `status` key, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6148: `phabricator2_metamta.metamta_mailinglist` has multiple overlong keys as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure
epriestley closed T6148: `phabricator2_metamta.metamta_mailinglist` has multiple overlong keys, a subtask of T6095: Schema changes required to support utf8mb4, as Resolved.
Oct 1 2014, 3:37 PM · Infrastructure

Sep 28 2014

epriestley added a revision to T6133: `phabricator_file.file_imagemacro` has a too-long key on `name`: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6099: Key `key_token` on `phabricator_auth.auth_temporarytoken` is too long: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6129: `phabricator_conpherence.conpherence_transaction_comment` has overlong `key_draft` key: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6134: `phabricator_file.file_transformedfile` has too-long key: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6150: `phabricator2_nuance.nuance_requestorsource` has overlong key: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6147: `phabricator_metamta.metamta_mail` has overlong `status` key: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6146: `phabricator2_meta_data.patch_status` has overlong key: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6148: `phabricator2_metamta.metamta_mailinglist` has multiple overlong keys: Restricted Differential Revision.
Sep 28 2014, 4:59 PM · Infrastructure
epriestley added a revision to T6130: `phabricator_conpherence.edge` has wrong `type` type: Restricted Differential Revision.
Sep 28 2014, 1:15 PM · Infrastructure
epriestley added a revision to T6128: `phabricator_dashboard.edge` has wrong column type for `type` column: Restricted Differential Revision.
Sep 28 2014, 1:15 PM · Infrastructure