[GEODE-4723] Not Able to Parse geode.properties duration-timeout

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Not A Bug
• Component/s: native client
• Assignee: Unassigned
• Reporter: Michael Martell
• Created: 22/Feb/18 18:10
• Updated: 20/Apr/18 19:57
• Resolved: 16/Mar/18 19:22

Description

This looks like a bug related to the switch to std::chrono for all time values.

Plan:

• Create a new ThinClientSystemPropertiesTests.cs (we currently only have cli tests for the Auth params)
• Add a test to verify that duration-timeout is parsed properly (this will likely fail)
• Fix the failing test by updating the parser to support std::chrono.duration

We were able to workaround this problem by appending an s to the parameter in the file:

i.e., in geode.properties we changed duration-timeout as below:

changed from:

     duration-timeout = 12345 

to:

     duration-timeout = 12345s