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
F15453384: D19738.diff
Sat, Mar 29, 11:24 AM
F15452361: D19738.id47164.diff
Sat, Mar 29, 4:11 AM
F15451097: D19738.id47158.diff
Fri, Mar 28, 8:08 PM
F15449162: D19738.id.diff
Fri, Mar 28, 8:40 AM
F15443412: D19738.id47159.diff
Thu, Mar 27, 4:14 AM
F15440461: D19738.diff
Wed, Mar 26, 12:56 PM
F15436859: D19738.id47164.diff
Tue, Mar 25, 4:40 PM
F15417133: D19738.id47159.diff
Thu, Mar 20, 3:33 PM
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