ifesdjeen commented on code in PR #3274:
URL: https://github.com/apache/cassandra/pull/3274#discussion_r1593510594


##########
src/java/org/apache/cassandra/config/DatabaseDescriptor.java:
##########
@@ -2238,6 +2247,91 @@ public static long getMinRpcTimeout(TimeUnit unit)
                          getTruncateRpcTimeout(unit));
     }
 
+    public static Config.CQLStartTime getCQLStartTime()
+    {
+        return conf.cql_start_time;
+    }
+
+    public static void setCQLStartTime(Config.CQLStartTime value)
+    {
+        conf.cql_start_time = value;
+    }
+
+    /**
+     * How much time the item is allowed to spend in (currently only Native) 
queue, compared to {@link #nativeTransportIdleTimeout()},

Review Comment:
   Right; the idea was that for starters we begin with this feature effectively 
disabled. Keep in mind I did not know how this feature is going to be perceived 
since it does change many notions simultaneously, but since both Jon and 
Brendon have tested it and found it useful, we can go with 1.0 here and 12 
seconds of deadline as defaults.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to