[jira] [Comment Edited] (GEODE-3136) Replace all public API time values with std::chrono types

["Jacob S. Barrett (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-3136?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16071673#comment-16071673 ] 

Jacob S. Barrett edited comment on GEODE-3136 at 7/2/17 2:58 PM:
-----------------------------------------------------------------

Investigating approach on this.

Leaning towards:

{code}

  template <class Rep, class Period>
  void setEntryIdleTimeout(
      ExpirationAction::Action action,
      const std::chrono::duration<Rep, Period>& idleTimeout);

{code}


Which allows for the caller to do:
{code}
// C++11
o.setEntryIdleTimeout(a, std::chrono::minutes(1));

// C++14
o.setEntryIdleTimeout(a, 1min);
{code}


was (Author: jbarrett):
Investigating approach on this.

Leaning towards:

{code}

  template <class Rep, class Period>
  void setEntryIdleTimeout(
      ExpirationAction::Action action,
      const std::chrono::duration<Rep, Period>& idleTimeout);

{code}


Which allows for the caller to do:
{code}
o.setEntryIdleTimeout(a, std::chrono::minutes(1));
{code}

> Replace all public API time values with std::chrono types
> ---------------------------------------------------------
>
>                 Key: GEODE-3136
>                 URL: https://issues.apache.org/jira/browse/GEODE-3136
>             Project: Geode
>          Issue Type: Improvement
>          Components: native client
>            Reporter: Jacob S. Barrett
>            Assignee: Jacob S. Barrett
>
> Remove ambiguity of time values by using [{{std::chrono}}|http://en.cppreference.com/w/cpp/chrono] types in the public API. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)