Github user ricellis commented on a diff in the pull request:

    https://github.com/apache/bahir/pull/57#discussion_r157478713
  
    --- Diff: 
sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantChangesConfig.scala
 ---
    @@ -67,7 +67,8 @@ class CloudantChangesConfig(protocol: String, host: 
String, dbName: String,
       }
     
       def getChangesReceiverUrl: String = {
    -    var url = dbUrl + "/" + defaultIndex + 
"?include_docs=true&feed=continuous&timeout=" + timeout
    +    var url = dbUrl + "/" + defaultIndex + 
"?include_docs=true&feed=normal" +
    +      "&seq_interval=10000&timeout=" + timeout
    --- End diff --
    
    WDYT about making the `seq_interval` the `bulkSize` instead of hard-coding?


---

Reply via email to