[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 After more than 10 successful builds I'd say that the flaky `sql-cloudant` tests are more stable now than they have ever been. Good work @emlaver. Once you have confirmed that there is no conflict with the copyright for the new code I am happy to merge this. ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/138/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :red_circle: Build failed, see build log for details ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/137/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/136/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/135/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/134/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/133/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/132/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/131/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/130/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/129/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/128/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 Uh oh, @emlaver unless someone else was running the same tests concurrently to the last Jenkins test run, there may still be some work to be done. ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Uh oh, @emlaver unless someone else was running the same tests concurrently to the last Jenkins test run, there may still work to be done. retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/127/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :red_circle: Build failed, see build log for details ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 uh, maybe one more, just to be sure :-) retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/126/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 one more for good emasure retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/125/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :red_circle: Build failed, see build log for details ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/124/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 @emlaver -- I may have influenced the test execution by running the `mvn test slq-cloudant -q` locally while the Jenkins PR builder was running the same tests (I was using the same Cloudant account as Jenkins). I just restarted the build. ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :red_circle: Build failed, see build log for details ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/123/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/57 retest this please ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/122/ ---
[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 :white_check_mark: Build successful ---
[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user emlaver commented on a diff in the pull request:
https://github.com/apache/bahir/pull/57#discussion_r156206675
--- Diff:
sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/ChangesRow.java ---
@@ -0,0 +1,74 @@
+/*
+ * Copyright (c) 2017 IBM Cloudant. All rights reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may
not use this file
+ * except in compliance with the License. You may obtain a copy of the
License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
distributed under the
+ * License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR
CONDITIONS OF ANY KIND,
+ * either express or implied. See the License for the specific language
governing permissions
+ * and limitations under the License.
+ */
+package org.apache.bahir.cloudant.common;
+
+import com.google.gson.JsonElement;
+import com.google.gson.JsonObject;
+
+import java.util.List;
+
+/**
+ * Class representing a single row in a changes feed. Structure:
+ *
+ * {
+ * last_seq": 5
+ * "results": [
+ * ---*** This next items is the ChangesRow ***---
+ * {
+ * "changes": [ {"rev": "2-eec205a9d413992850a6e32678485900"}, ... ],
+ * "deleted": true,
+ * "id": "deleted",
+ * "seq": 5,
+ * "doc": ... structure ...
+ * }
+ * ]
+ * }
+ */
+public class ChangesRow {
--- End diff --
Fixed in 5da88e8.
---
[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test
Github user emlaver commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r156123828 --- Diff: sql-cloudant/src/main/scala/org/apache/bahir/cloudant/common/ChangesRow.java --- @@ -0,0 +1,74 @@ +/* + * Copyright (c) 2017 IBM Cloudant. All rights reserved. --- End diff -- @ckadner It's valid and the author is OK with this existing in the open-source. From my understanding, we're (Cloudant) allowed to retain the copyright on code we open source, but the license effectively grants free use. ---
[GitHub] bahir-flink pull request #26: [BAHIR-153] Support flink-library-siddhi scala...
GitHub user haoch opened a pull request: https://github.com/apache/bahir-flink/pull/26 [BAHIR-153] Support flink-library-siddhi scala DSL You can merge this pull request into a Git repository by running: $ git pull https://github.com/haoch/bahir-flink scala_dsl Alternatively you can review and apply these changes as the patch at: https://github.com/apache/bahir-flink/pull/26.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #26 commit dbb9f48a5453e08f78a4e5beae3ca987b720c884 Author: Aleksandr Chermenin Date: 2017-12-01T14:09:34Z Improved integration test cases commit 5f19f7f28ef100bdc0d2b880d06bf2cd766845b4 Author: Hao Chen Date: 2017-12-11T08:19:39Z Resolve map type info warning commit ca86a4d8fb59d2ab5f98eee405baca2d8c6d87c0 Author: Hao Chen Date: 2017-12-11T08:22:51Z Merge branch 'it_case_fixes' of https://github.com/chermenin/bahir-flink commit 6d6bd775b02402487b082a66ce816b95ef4565a6 Author: Hao Chen Date: 2017-12-11T10:31:11Z Add scala DSL ---
