search

Smartling / smartling-cli

Smartling-cli is a library and CLI tool for managing Smartling projects. See the project's Wiki for user documentation.
https://github.com/Smartling/smartling-cli/wiki
MIT License
16 stars 12 forks source link

File list command is too slow #40

Open dimitrystd opened 6 years ago

dimitrystd commented 6 years ago

I have a project with 1.2K xml files. I wanted to delete all files. So, first of all, i tried to build a list of all files by command

smartling-cli files list -s -p 2a1xxxxxx

After 5 mins of waiting, i don't see file URIs. Then i tried to run with -vv and looks like it works, but it requests details for each file. In my case, i don't need any details and explicitly added -s.

Can we optimize files list command for -s parameter because it's impossible to get a result for projects with 200+ files?

dimitrystd commented 6 years ago

@enegron-sl we designed files list as command that provides full information about remote and local files. By default we print nice table with progress for every locale. Should we make files list really simple like corresponding api call and design another command for detailed status?

seletskiy commented 6 years ago

@dimitrystd: Files list is not requesting any other information from remote server:

↓ https://github.com/Smartling/smartling-cli/blob/master/do_files_list.go#L32 ↓ https://github.com/Smartling/smartling-cli/blob/master/glob_files.go#L35 • https://github.com/Smartling/api-sdk-go/blob/master/client_list_files.go#L57

It maybe paging issue, because it should do several requests to obtain full file list.