[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added inline comments. Comment at: clang-tools-extra/clangd/index/remote/server/Server.cpp:84 +llvm::cl::opt IdleTimeoutSeconds( +"idle-timeout", llvm::cl::init(10), +llvm::cl::desc("Maximum time a channel may stay idle until server closes " samm

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG1a5dfb7db23e: [clangd][remote] Add flag to set idletimeout (authored by kadircet). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 326667. kadircet marked an inline comment as done. kadircet added a comment. - change default from 10 to 8 minutes Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97536/new/ https://reviews.llvm.org/D97536 File

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. Comment at: clang-tools-extra/clangd/index/remote/server/Server.cpp:84 +llvm::cl::opt IdleTimeoutSeconds( +"idle-timeout", llvm::cl::init(10), +llvm::cl::desc("Maximum time a channel may stay idle until se

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added inline comments. Comment at: clang-tools-extra/clangd/index/remote/server/Server.cpp:84 +llvm::cl::opt IdleTimeoutSeconds( +"idle-timeout", llvm::cl::init(10), +llvm::cl::desc("Maximum time a channel may stay idle until server closes " samm

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet updated this revision to Diff 326632. kadircet marked an inline comment as done. kadircet added a comment. Change 10 seconds to 600 seconds. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97536/new/ https://reviews.llvm.org/D97536 Files:

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/index/remote/server/Server.cpp:84 +llvm::cl::opt IdleTimeoutSeconds( +"idle-timeout", llvm::cl::init(10), +llvm::cl::de

[PATCH] D97536: [clangd][remote] Add flag to set idletimeout

2021-02-26 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet created this revision. kadircet added a reviewer: sammccall. Herald added subscribers: usaxena95, arphaman. kadircet requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. By default gRPC has no idletimeout and some