Page MenuHomePhabricator

Allow "bin/bulk export" to merge multiple queries and accept more flexible flags
ClosedPublic

Authored by epriestley on Oct 8 2018, 5:19 PM.
Tags
None
Referenced Files
F18095149: D19738.id.diff
Thu, Aug 7, 10:32 PM
F18089193: D19738.id47158.diff
Wed, Aug 6, 10:37 AM
F18087152: D19738.id47159.diff
Wed, Aug 6, 2:38 AM
F18084290: D19738.diff
Tue, Aug 5, 2:40 PM
F17885450: D19738.id47164.diff
Tue, Jul 29, 12:23 AM
F17799783: D19738.id47159.diff
Fri, Jul 25, 5:30 AM
F17798920: D19738.id.diff
Fri, Jul 25, 4:16 AM
F17798577: D19738.id47158.diff
Fri, Jul 25, 3:42 AM
Subscribers
None

Details

Summary

Ref T13210. Minor usability improvements to "bin/bulk export":

  • Allow --class task to work (previously, only --class ManiphestTaskSearchEngine worked).
  • If you run --query jXIlzQyOYHPU, don't require --class, since the query identifies the class on its own.
  • Allow users to call --query A --query B --query C and get a union of all results.
Test Plan
  • Ran --class task, --query A --query B, --query X (with no --class), got good results.
  • Ran various flavors of bad combinations (queries from different engines, invalid engines, query and class differing, ambiguous/invalid --class name) and got sensible errors.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable