[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126402723 --- Diff: sql-cloudant/README.md --- @@ -52,39 +51,71 @@ Here each subsequent configuration overrides the previous one. Thus, configurati

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126428690 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -30,81 +28,83 @@ import org.apache.bahir.cloudant.common

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126438628 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/DefaultSource.scala --- @@ -98,29 +99,89 @@ class DefaultSource extends RelationProvider

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126409663 --- Diff: sql-cloudant/README.md --- @@ -52,39 +51,71 @@ Here each subsequent configuration overrides the previous one. Thus, configurati

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126435531 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantReceiver.scala --- @@ -16,23 +16,20 @@ */ package

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126401728 --- Diff: sql-cloudant/README.md --- @@ -31,15 +31,14 @@ The `--packages` argument can also be used with `bin/spark-submit`. Submit a job

[GitHub] bahir pull request #45: [WIP] [BAHIR-110] Implement _changes API for non-str...

2017-07-10 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/45#discussion_r126402346 --- Diff: sql-cloudant/README.md --- @@ -52,39 +51,71 @@ Here each subsequent configuration overrides the previous one. Thus, configurati

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-18 Thread ricellis
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

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-18 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r157479639 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/internal/ChangesReceiver.scala --- @@ -39,56 +37,38 @@ class ChangesReceiver(config

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-18 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r157480164 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/DefaultSource.scala --- @@ -125,7 +125,7 @@ class DefaultSource extends RelationProvider

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-18 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r157484722 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/internal/ChangesReceiver.scala --- @@ -39,56 +37,38 @@ class ChangesReceiver(config

[GitHub] bahir-website pull request #13: Fixed typos in Apache CouchDB

2017-11-17 Thread ricellis
GitHub user ricellis opened a pull request: https://github.com/apache/bahir-website/pull/13 Fixed typos in Apache CouchDB Apache CouchDB is incorrectly referred to as `CounchDB` twice on the website. This PR corrects those typos. You can merge this pull request into a Git

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-09 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r160470451 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159845032 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159845769 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159889787 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/JsonStoreRDD.scala --- @@ -191,20 +199,30 @@ class JsonStoreRDD(sc: SparkContext

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159890761 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/JsonStoreRDD.scala --- @@ -191,20 +199,30 @@ class JsonStoreRDD(sc: SparkContext

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159889840 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/JsonStoreRDD.scala --- @@ -191,20 +199,30 @@ class JsonStoreRDD(sc: SparkContext

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159845855 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159889644 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/JsonStoreRDD.scala --- @@ -174,7 +178,11 @@ class JsonStoreRDD(sc: SparkContext

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159847773 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -54,20 +147,21 @@ class CloudantConfig(val protocol: String, val

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159845874 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159889542 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/JsonStoreRDD.scala --- @@ -101,24 +105,24 @@ class JsonStoreRDD(sc: SparkContext

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159845718 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-05 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159848274 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -95,15 +189,49 @@ class CloudantConfig(val protocol: String, val

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-04 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159699798 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-04 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159699042 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -16,34 +16,127 @@ */ package org.apache.bahir.cloudant

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-04 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159709382 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -95,15 +189,49 @@ class CloudantConfig(val protocol: String, val

[GitHub] bahir pull request #61: [BAHIR-154] Refactor sql-cloudant to use Cloudant's ...

2018-01-04 Thread ricellis
Github user ricellis commented on a diff in the pull request: https://github.com/apache/bahir/pull/61#discussion_r159709935 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/CloudantConfig.scala --- @@ -95,15 +189,49 @@ class CloudantConfig(val protocol: String, val