geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (GEODE-2891) connect-timeout violation in C++ Native Client
Date Thu, 06 Jul 2017 16:18:02 GMT

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

ASF GitHub Bot commented on GEODE-2891:
---------------------------------------

Github user gregt5259 commented on the issue:

    https://github.com/apache/geode-native/pull/106
  
    In my mind we talk about the some kind of redesign rather than about correct change: the
original design is based de-facto exactly on using ‘magic numbers’; configurable measurements
units in an explicit form are missing in the original design. Yes, my solution solves the
problem in the code implementing accordingly to the original design and quite could be realized
in the code implementing original design without waiting  for future redesigns and refactoring,
especially with undefined due date.
    
    Thanks,
    Dr. Gregory Turovets
    
    "…We're all mad here. I'm mad. You're mad."
    "How do you know I'm mad?" said Alice.
    "You must be," said the Cat, "or you wouldn't have come here."
    Alice's Adventures in Wonderland by Lewis Carroll<http://www.livelib.ru/author/157108>.
    
    From: Jacob Barrett [mailto:notifications@github.com]
    Sent: Thursday, July 06, 2017 17:35
    To: apache/geode-native <geode-native@noreply.github.com>
    Cc: Gregory Turovets <gregory.turovets@amdocs.com>; Mention <mention@noreply.github.com>
    Subject: Re: [apache/geode-native] GEODE-2891 connect-timeout violation in C   Native
Client (#106)
    
    
    @gregt5259<https://github.com/gregt5259> This is a solution to the problem but not
the solution we as committers are comfortable committing as it directly conflicts with the
correct change, which is to use type safe durations rather than magic number math and system
wide properties to create a confusing array of time values.
    
    If you want this change sooner than later you could implement it using std::chrono::duration
as outlined in GEODE-3137 or maintain a fork with your change in it.
    
    —
    You are receiving this because you were mentioned.
    Reply to this email directly, view it on GitHub<https://github.com/apache/geode-native/pull/106#issuecomment-313414625>,
or mute the thread<https://github.com/notifications/unsubscribe-auth/AbZcfv8z-a_Qfl4VwbQmqK8WI7aM9k9aks5sLPB0gaJpZM4OOTnp>.
    This message and the information contained herein is proprietary and confidential and
subject to the Amdocs policy statement,
    
    you may review at https://www.amdocs.com/about/email-disclaimer <https://www.amdocs.com/about/email-disclaimer>



> connect-timeout violation in C++ Native Client
> ----------------------------------------------
>
>                 Key: GEODE-2891
>                 URL: https://issues.apache.org/jira/browse/GEODE-2891
>             Project: Geode
>          Issue Type: Bug
>          Components: native client
>            Reporter: Gregory Vortman
>            Assignee: Jacob S. Barrett
>         Attachments: GEODE-connect-timeout-violation.docx
>
>
> 1.	C++ native client doesn’t honour read-timeout-milli-sec in a consistent way while
connecting to a server
> 2.	The lock on the connection pool has a very high granularity. Even if the client can’t
connect to one server, all other threads which are working with totally different servers
get affected by it



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

Mime
View raw message