[jira] [Commented] (STREAMS-533) Fix JavaDoc links in streams-provider-twitter
[ https://issues.apache.org/jira/browse/STREAMS-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215459#comment-16215459 ] ASF GitHub Bot commented on STREAMS-533: b-hodge commented on issue #393: STREAMS-533: Fix JavaDoc links in streams-provider-twitter URL: https://github.com/apache/streams/pull/393#issuecomment-338728224 LGTM š This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Fix JavaDoc links in streams-provider-twitter > - > > Key: STREAMS-533 > URL: https://issues.apache.org/jira/browse/STREAMS-533 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > On 9/26/17 Twitter pushed a new API reference website. As a result most of > the links in the java docs are now dead. The new reference is hosted at > https://developer.twitter.com/en/docs/api-reference-index - virtually every > endpoint has a new URL so they'll have to be researched and updated by hand. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-533) Fix JavaDoc links in streams-provider-twitter
[ https://issues.apache.org/jira/browse/STREAMS-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215583#comment-16215583 ] ASF GitHub Bot commented on STREAMS-533: steveblackmon closed pull request #393: STREAMS-533: Fix JavaDoc links in streams-provider-twitter URL: https://github.com/apache/streams/pull/393 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-provider-twitter/pom.xml b/streams-contrib/streams-provider-twitter/pom.xml index 304dddee8..503da218b 100644 --- a/streams-contrib/streams-provider-twitter/pom.xml +++ b/streams-contrib/streams-provider-twitter/pom.xml @@ -25,10 +25,12 @@ 4.0.0 streams-provider-twitter +streams-provider-twitter + +Twitter Provider true - http://streams.peoplepattern.com.s3.amazonaws.com/test-data/ diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Account.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Account.java index dfe0c4e83..960992717 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Account.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Account.java @@ -20,6 +20,7 @@ import org.apache.streams.twitter.pojo.User; +import org.apache.juneau.remoteable.Body; import org.apache.juneau.remoteable.RemoteMethod; import org.apache.juneau.remoteable.Remoteable; @@ -34,8 +35,8 @@ /** * Returns settings (including current trend, geo and sleep time information) for the authenticating user. * - * @return AccountSettingsResponse - * @see https://dev.twitter.com/rest/reference/get/account/settings";>https://dev.twitter.com/rest/reference/get/account/settings + * @return {@link org.apache.streams.twitter.api.AccountSettings} + * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-settings";>https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-settings * */ @RemoteMethod(httpMethod = "GET", path = "/settings.json") @@ -44,11 +45,31 @@ /** * Returns user credentials for the authenticating user. * - * @return User - * @see https://dev.twitter.com/rest/reference/get/account/verify_credentials";>https://dev.twitter.com/rest/reference/get/account/verify_credentials + * @return {@link org.apache.streams.twitter.pojo.User} + * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-verify_credentials";>https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-verify_credentials * */ @RemoteMethod(httpMethod = "GET", path = "/verify_credentials.json") public User verifyCredentials(); + /** + * Sets some values that users are able to set under the āAccountā tab of their settings page. Only the parameters specified will be updated. + * + * @return {@link org.apache.streams.twitter.pojo.User} + * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/post-account-update_profile";>https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/post-account-update_profile + * + */ + @RemoteMethod(httpMethod = "POST", path = "/update_profile.json") + public User updateProfile(@Body UpdateProfileRequest parameters); + + /** + * Updates the authenticating userās settings. + * + * @return {@link org.apache.streams.twitter.api.AccountSettings} + * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/post-account-settings";>https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/post-account-settings + * + */ + @RemoteMethod(httpMethod = "POST", path = "/update_settings.json") + public AccountSettings updateSettings(@Body UpdateProfileRequest parameters); + } diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/AccountActivity.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/AccountActivity.java index fe13db099..114a8a607 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/AccountActivity.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/AccountActivity.java @@ -32,7 +32,7 @@ /** * Interface for /account_activity methods. *
[jira] [Commented] (STREAMS-546) Allow rat plugin to ignore .conf files in project root
[ https://issues.apache.org/jira/browse/STREAMS-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16216204#comment-16216204 ] ASF GitHub Bot commented on STREAMS-546: steveblackmon opened a new pull request #396: STREAMS-546: Allow rat plugin to ignore .conf files in project root URL: https://github.com/apache/streams/pull/396 (https://issues.apache.org/jira/browse/STREAMS-546) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Allow rat plugin to ignore .conf files in project root > -- > > Key: STREAMS-546 > URL: https://issues.apache.org/jira/browse/STREAMS-546 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > The release process involves creating conf files in the project root to > support integration tests. There's no reason to require a developer to put > apache license headers on their personal conf files, and there are no other > conf files in the project files which should be so licensed, so it seems > appropriate to just ignore all conf files in project root. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-547) add any missing AL headers
[ https://issues.apache.org/jira/browse/STREAMS-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16216205#comment-16216205 ] ASF GitHub Bot commented on STREAMS-547: steveblackmon opened a new pull request #397: STREAMS-547: add any missing AL headers URL: https://github.com/apache/streams/pull/397 (https://issues.apache.org/jira/browse/STREAMS-547) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > add any missing AL headers > -- > > Key: STREAMS-547 > URL: https://issues.apache.org/jira/browse/STREAMS-547 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > add any missing AL headers before 0.6.0 release -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-533) Fix JavaDoc links in streams-provider-twitter
[ https://issues.apache.org/jira/browse/STREAMS-533?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16218721#comment-16218721 ] ASF GitHub Bot commented on STREAMS-533: Abdelrahman-Hasan commented on issue #393: STREAMS-533: Fix JavaDoc links in streams-provider-twitter URL: https://github.com/apache/streams/pull/393#issuecomment-339346591 Hi, org.apache.streams.twitter.pojo package doesn't exist This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Fix JavaDoc links in streams-provider-twitter > - > > Key: STREAMS-533 > URL: https://issues.apache.org/jira/browse/STREAMS-533 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > On 9/26/17 Twitter pushed a new API reference website. As a result most of > the links in the java docs are now dead. The new reference is hosted at > https://developer.twitter.com/en/docs/api-reference-index - virtually every > endpoint has a new URL so they'll have to be researched and updated by hand. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-547) add any missing AL headers
[ https://issues.apache.org/jira/browse/STREAMS-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219579#comment-16219579 ] ASF GitHub Bot commented on STREAMS-547: rawkintrevo commented on issue #397: STREAMS-547: add any missing AL headers URL: https://github.com/apache/streams/pull/397#issuecomment-339479913 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > add any missing AL headers > -- > > Key: STREAMS-547 > URL: https://issues.apache.org/jira/browse/STREAMS-547 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > add any missing AL headers before 0.6.0 release -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-546) Allow rat plugin to ignore .conf files in project root
[ https://issues.apache.org/jira/browse/STREAMS-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219582#comment-16219582 ] ASF GitHub Bot commented on STREAMS-546: rawkintrevo commented on issue #396: STREAMS-546: Allow rat plugin to ignore .conf files in project root URL: https://github.com/apache/streams/pull/396#issuecomment-339480018 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Allow rat plugin to ignore .conf files in project root > -- > > Key: STREAMS-546 > URL: https://issues.apache.org/jira/browse/STREAMS-546 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > The release process involves creating conf files in the project root to > support integration tests. There's no reason to require a developer to put > apache license headers on their personal conf files, and there are no other > conf files in the project files which should be so licensed, so it seems > appropriate to just ignore all conf files in project root. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-546) Allow rat plugin to ignore .conf files in project root
[ https://issues.apache.org/jira/browse/STREAMS-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219600#comment-16219600 ] ASF GitHub Bot commented on STREAMS-546: steveblackmon closed pull request #396: STREAMS-546: Allow rat plugin to ignore .conf files in project root URL: https://github.com/apache/streams/pull/396 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 037e11890..b2c57509d 100644 --- a/pom.xml +++ b/pom.xml @@ -658,6 +658,7 @@ .git/** .idea/** .java-version +*.conf **/DEPENDENCIES **/LICENSE **/MANIFEST.MF @@ -676,7 +677,7 @@ **/*.svg dependency-reduced-pom.xml rat.txt -apache-maven-3.3.9/**/* +apache-maven-*/**/* This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Allow rat plugin to ignore .conf files in project root > -- > > Key: STREAMS-546 > URL: https://issues.apache.org/jira/browse/STREAMS-546 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > The release process involves creating conf files in the project root to > support integration tests. There's no reason to require a developer to put > apache license headers on their personal conf files, and there are no other > conf files in the project files which should be so licensed, so it seems > appropriate to just ignore all conf files in project root. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-547) add any missing AL headers
[ https://issues.apache.org/jira/browse/STREAMS-547?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219601#comment-16219601 ] ASF GitHub Bot commented on STREAMS-547: steveblackmon closed pull request #397: STREAMS-547: add any missing AL headers URL: https://github.com/apache/streams/pull/397 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-config/src/test/resources/testResolve.conf b/streams-config/src/test/resources/testResolve.conf index b1619ba19..5515e 100644 --- a/streams-config/src/test/resources/testResolve.conf +++ b/streams-config/src/test/resources/testResolve.conf @@ -1,3 +1,18 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# +# 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. include "more.conf" evenmore = { include "evenmore.conf"} samemessage = ${evenmore.message} diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Media.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Media.java index e79b6b590..ae05cd538 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Media.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Media.java @@ -1,3 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.api; import org.apache.streams.twitter.pojo.Tweet; diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/TwitterSecurity.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/TwitterSecurity.java index f02ea9390..502699eaf 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/TwitterSecurity.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/TwitterSecurity.java @@ -1,3 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.api; import org.apache.commons.codec.binary.Base64; diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/WelcomeMessageRules.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/WelcomeMessageRules.java index 91f1a10df..4046dc97b 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/WelcomeMessageRules.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/WelcomeMessageRules.java @@ -1,3 +1,21 @
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16220602#comment-16220602 ] ASF GitHub Bot commented on STREAMS-548: steveblackmon opened a new pull request #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398 https://issues.apache.org/jira/browse/STREAMS-548 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222577#comment-16222577 ] ASF GitHub Bot commented on STREAMS-548: steveblackmon commented on issue #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398#issuecomment-340008347 will reopen and hopefully jenkins will work this time This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222578#comment-16222578 ] ASF GitHub Bot commented on STREAMS-548: steveblackmon opened a new pull request #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398 https://issues.apache.org/jira/browse/STREAMS-548 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222572#comment-16222572 ] ASF GitHub Bot commented on STREAMS-548: steveblackmon closed pull request #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index b2c57509d..5c5098d50 100644 --- a/pom.xml +++ b/pom.xml @@ -336,7 +336,7 @@ 1.8.1 0.4.1 0.1.10 -6.4.0-incubating +7.0.0 0.4.10 0.13.1 0.11.1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222624#comment-16222624 ] ASF GitHub Bot commented on STREAMS-548: jamesbognar commented on issue #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398#issuecomment-340017629 The change looks good, but I don't have write access to merge the pull request :-( This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-548) update juneau dependency to 7.0.0
[ https://issues.apache.org/jira/browse/STREAMS-548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222639#comment-16222639 ] ASF GitHub Bot commented on STREAMS-548: steveblackmon closed pull request #398: STREAMS-548: update juneau dependency to 7.0.0 URL: https://github.com/apache/streams/pull/398 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index b2c57509d..5c5098d50 100644 --- a/pom.xml +++ b/pom.xml @@ -336,7 +336,7 @@ 1.8.1 0.4.1 0.1.10 -6.4.0-incubating +7.0.0 0.4.10 0.13.1 0.11.1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > update juneau dependency to 7.0.0 > - > > Key: STREAMS-548 > URL: https://issues.apache.org/jira/browse/STREAMS-548 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > update juneau dependency to 7.0.0 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-493) streams-examples jenkins build fails
[ https://issues.apache.org/jira/browse/STREAMS-493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222781#comment-16222781 ] ASF GitHub Bot commented on STREAMS-493: steveblackmon opened a new pull request #400: STREAMS-493: streams-examples jenkins build fails URL: https://github.com/apache/streams/pull/400 (https://issues.apache.org/jira/browse/STREAMS-493) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > streams-examples jenkins build fails > > > Key: STREAMS-493 > URL: https://issues.apache.org/jira/browse/STREAMS-493 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Suneel Marthi > Fix For: 0.6 > > Original Estimate: 4h > Remaining Estimate: 4h > > even after streams-project completes a build and installs its snapshots. > for example, see: > https://builds.apache.org/job/streams-project/350/ > https://builds.apache.org/job/streams-examples/193/ > error is: > [ERROR] Plugin > org.apache.streams.plugins:streams-plugin-pojo:0.5-incubating-SNAPSHOT or one > of its dependencies could not be resolved: Could not find artifact > org.apache.streams.plugins:streams-plugin-pojo:jar:0.5-incubating-SNAPSHOT > i think the problem is that our master pom does not specify > snapshots.apache.org as a pluginRepository -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-493) streams-examples jenkins build fails
[ https://issues.apache.org/jira/browse/STREAMS-493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222802#comment-16222802 ] ASF GitHub Bot commented on STREAMS-493: steveblackmon closed pull request #400: STREAMS-493: streams-examples jenkins build fails URL: https://github.com/apache/streams/pull/400 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 5c5098d50..8bb3cce81 100644 --- a/pom.xml +++ b/pom.xml @@ -259,6 +259,31 @@ + + +apache-repo +Apache Repository + https://repository.apache.org/content/repositories/releases + +true + + +false + + + +apache-snapshots +Apache Repository + https://repository.apache.org/content/repositories/snapshots + +false + + +true + + + + streams.releases This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > streams-examples jenkins build fails > > > Key: STREAMS-493 > URL: https://issues.apache.org/jira/browse/STREAMS-493 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Suneel Marthi > Fix For: 0.6 > > Original Estimate: 4h > Remaining Estimate: 4h > > even after streams-project completes a build and installs its snapshots. > for example, see: > https://builds.apache.org/job/streams-project/350/ > https://builds.apache.org/job/streams-examples/193/ > error is: > [ERROR] Plugin > org.apache.streams.plugins:streams-plugin-pojo:0.5-incubating-SNAPSHOT or one > of its dependencies could not be resolved: Could not find artifact > org.apache.streams.plugins:streams-plugin-pojo:jar:0.5-incubating-SNAPSHOT > i think the problem is that our master pom does not specify > snapshots.apache.org as a pluginRepository -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-493) streams-examples jenkins build fails
[ https://issues.apache.org/jira/browse/STREAMS-493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222801#comment-16222801 ] ASF GitHub Bot commented on STREAMS-493: steveblackmon commented on issue #400: STREAMS-493: streams-examples jenkins build fails URL: https://github.com/apache/streams/pull/400#issuecomment-340082219 trivial change, no downsides, merging This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > streams-examples jenkins build fails > > > Key: STREAMS-493 > URL: https://issues.apache.org/jira/browse/STREAMS-493 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Suneel Marthi > Fix For: 0.6 > > Original Estimate: 4h > Remaining Estimate: 4h > > even after streams-project completes a build and installs its snapshots. > for example, see: > https://builds.apache.org/job/streams-project/350/ > https://builds.apache.org/job/streams-examples/193/ > error is: > [ERROR] Plugin > org.apache.streams.plugins:streams-plugin-pojo:0.5-incubating-SNAPSHOT or one > of its dependencies could not be resolved: Could not find artifact > org.apache.streams.plugins:streams-plugin-pojo:jar:0.5-incubating-SNAPSHOT > i think the problem is that our master pom does not specify > snapshots.apache.org as a pluginRepository -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-538) Switch from incubator SVN to not-incubator GIT for website publishing
[ https://issues.apache.org/jira/browse/STREAMS-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16223638#comment-16223638 ] ASF GitHub Bot commented on STREAMS-538: steveblackmon opened a new pull request #401: STREAMS-538: switch to git for website publishing URL: https://github.com/apache/streams/pull/401 Have confirmed that with git credentials scm-publish:publish-scm works as desired after this change. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Switch from incubator SVN to not-incubator GIT for website publishing > - > > Key: STREAMS-538 > URL: https://issues.apache.org/jira/browse/STREAMS-538 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Switch from publishing the streams website to the incubator SVN to instead > publish using gitpubsub on the streams-project and streams-examples repos -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-538) Switch from incubator SVN to not-incubator GIT for website publishing
[ https://issues.apache.org/jira/browse/STREAMS-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16223639#comment-16223639 ] ASF GitHub Bot commented on STREAMS-538: steveblackmon commented on issue #401: STREAMS-538: switch to git for website publishing URL: https://github.com/apache/streams/pull/401#issuecomment-340203646 trivial change, no potential for problems, merging This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Switch from incubator SVN to not-incubator GIT for website publishing > - > > Key: STREAMS-538 > URL: https://issues.apache.org/jira/browse/STREAMS-538 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Switch from publishing the streams website to the incubator SVN to instead > publish using gitpubsub on the streams-project and streams-examples repos -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-538) Switch from incubator SVN to not-incubator GIT for website publishing
[ https://issues.apache.org/jira/browse/STREAMS-538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16223640#comment-16223640 ] ASF GitHub Bot commented on STREAMS-538: steveblackmon closed pull request #401: STREAMS-538: switch to git for website publishing URL: https://github.com/apache/streams/pull/401 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 8bb3cce81..c7d17669c 100644 --- a/pom.xml +++ b/pom.xml @@ -298,7 +298,7 @@ site.streams.project -scm:git:https://github.com/apache/streams.git +scm:git:g...@github.com:apache/streams.git This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Switch from incubator SVN to not-incubator GIT for website publishing > - > > Key: STREAMS-538 > URL: https://issues.apache.org/jira/browse/STREAMS-538 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Switch from publishing the streams website to the incubator SVN to instead > publish using gitpubsub on the streams-project and streams-examples repos -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-519) Support parsing dates with micro and nano precision with StreamsJacksonMapper
[ https://issues.apache.org/jira/browse/STREAMS-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16225078#comment-16225078 ] ASF GitHub Bot commented on STREAMS-519: steveblackmon opened a new pull request #402: STREAMS-519: Support parsing dates with micro and nano precision with StreamsJacksonMapper (https://issues.apache.org/jira/browse/STREAMS-519) URL: https://github.com/apache/streams/pull/402 STREAMS-519: Support parsing dates with micro and nano precision with StreamsJacksonMapper (https://issues.apache.org/jira/browse/STREAMS-519) Once I dug it I saw that micro and nano precision was anticipated. Joda doesn't handle any precision past millis but StreamsJacksonMapper is able to parse dates with higher precision and I added new test cases confirming that is true. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Support parsing dates with micro and nano precision with StreamsJacksonMapper > - > > Key: STREAMS-519 > URL: https://issues.apache.org/jira/browse/STREAMS-519 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > StreamsJacksonMapper does not currently seem to support parsing dates with > greater than millisecond precision. Not surprising as Joda supports millis > as its smallest interval, but it should be possible at least to parse a date > with greater precision without throwing an exception. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-519) Support parsing dates with micro and nano precision with StreamsJacksonMapper
[ https://issues.apache.org/jira/browse/STREAMS-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16225824#comment-16225824 ] ASF GitHub Bot commented on STREAMS-519: steveblackmon commented on issue #402: STREAMS-519: Support parsing dates with micro and nano precision with StreamsJacksonMapper (https://issues.apache.org/jira/browse/STREAMS-519) URL: https://github.com/apache/streams/pull/402#issuecomment-340598466 This is just new tests - merging This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Support parsing dates with micro and nano precision with StreamsJacksonMapper > - > > Key: STREAMS-519 > URL: https://issues.apache.org/jira/browse/STREAMS-519 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > StreamsJacksonMapper does not currently seem to support parsing dates with > greater than millisecond precision. Not surprising as Joda supports millis > as its smallest interval, but it should be possible at least to parse a date > with greater precision without throwing an exception. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-519) Support parsing dates with micro and nano precision with StreamsJacksonMapper
[ https://issues.apache.org/jira/browse/STREAMS-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16225823#comment-16225823 ] ASF GitHub Bot commented on STREAMS-519: steveblackmon closed pull request #402: STREAMS-519: Support parsing dates with micro and nano precision with StreamsJacksonMapper (https://issues.apache.org/jira/browse/STREAMS-519) URL: https://github.com/apache/streams/pull/402 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-pojo/src/test/java/org/apache/streams/pojo/test/RFC3339UtilsTest.java b/streams-pojo/src/test/java/org/apache/streams/pojo/test/RFC3339UtilsTest.java index bce756aec..947a11335 100644 --- a/streams-pojo/src/test/java/org/apache/streams/pojo/test/RFC3339UtilsTest.java +++ b/streams-pojo/src/test/java/org/apache/streams/pojo/test/RFC3339UtilsTest.java @@ -69,6 +69,26 @@ public void validUTCSubSecondMultiDigit() { } @Test + public void validUTCMicroSecond() { +DateTime parsed = parseUTC("2014-12-25T12:00:00.235235Z"); +assertThat(parsed.minuteOfHour().get(), is(equalTo(0))); +assertThat(parsed.hourOfDay().get(), is(equalTo(12))); +assertThat(parsed.dayOfMonth().get(), is(equalTo(25))); +assertThat(parsed.monthOfYear().get(), is(equalTo(12))); +assertThat(parsed.millisOfSecond().get(), is(equalTo(235))); + } + + @Test + public void validUTCNanoSecond() { +DateTime parsed = parseUTC("2014-12-25T12:00:00.123456789Z"); +assertThat(parsed.minuteOfHour().get(), is(equalTo(0))); +assertThat(parsed.hourOfDay().get(), is(equalTo(12))); +assertThat(parsed.dayOfMonth().get(), is(equalTo(25))); +assertThat(parsed.monthOfYear().get(), is(equalTo(12))); +assertThat(parsed.millisOfSecond().get(), is(equalTo(123))); + } + + @Test public void validEST() { DateTime parsed = parseUTC("2014-12-25T12:00:00-05:00"); assertThat(parsed.minuteOfHour().get(), is(equalTo(0))); This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Support parsing dates with micro and nano precision with StreamsJacksonMapper > - > > Key: STREAMS-519 > URL: https://issues.apache.org/jira/browse/STREAMS-519 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > > StreamsJacksonMapper does not currently seem to support parsing dates with > greater than millisecond precision. Not surprising as Joda supports millis > as its smallest interval, but it should be possible at least to parse a date > with greater precision without throwing an exception. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-549) Bug in PropertyUtil.mergeProperties
[ https://issues.apache.org/jira/browse/STREAMS-549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16242291#comment-16242291 ] ASF GitHub Bot commented on STREAMS-549: steveblackmon opened a new pull request #403: STREAMS-549: Bug in PropertyUtil.mergeProperties URL: https://github.com/apache/streams/pull/403 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Bug in PropertyUtil.mergeProperties > --- > > Key: STREAMS-549 > URL: https://issues.apache.org/jira/browse/STREAMS-549 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > ValueNodes should over-write even when present on the fallback -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-549) Bug in PropertyUtil.mergeProperties
[ https://issues.apache.org/jira/browse/STREAMS-549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16242346#comment-16242346 ] ASF GitHub Bot commented on STREAMS-549: steveblackmon commented on issue #403: STREAMS-549: Bug in PropertyUtil.mergeProperties URL: https://github.com/apache/streams/pull/403#issuecomment-342543447 this finished - https://builds.apache.org/job/streams-project-pullrequests/54/ This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Bug in PropertyUtil.mergeProperties > --- > > Key: STREAMS-549 > URL: https://issues.apache.org/jira/browse/STREAMS-549 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > ValueNodes should over-write even when present on the fallback -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-549) Bug in PropertyUtil.mergeProperties
[ https://issues.apache.org/jira/browse/STREAMS-549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16243241#comment-16243241 ] ASF GitHub Bot commented on STREAMS-549: steveblackmon closed pull request #403: STREAMS-549: Bug in PropertyUtil.mergeProperties URL: https://github.com/apache/streams/pull/403 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-util/src/main/java/org/apache/streams/util/PropertyUtil.java b/streams-util/src/main/java/org/apache/streams/util/PropertyUtil.java index 93ff76af6..63678f68b 100644 --- a/streams-util/src/main/java/org/apache/streams/util/PropertyUtil.java +++ b/streams-util/src/main/java/org/apache/streams/util/PropertyUtil.java @@ -128,6 +128,10 @@ public static ObjectNode mergeProperties(ObjectNode content, ObjectNode parent) if( merged.get(fieldId) != null ) { if( merged.get(fieldId).getNodeType().equals(JsonNodeType.OBJECT)) { merged.put(fieldId, mergeProperties(field.getValue().deepCopy(), (ObjectNode)merged.get(fieldId))); +} else if ( merged.get(fieldId).getNodeType().equals(JsonNodeType.ARRAY)) { + merged.put(fieldId, mergeArrays(((ArrayNode)field.getValue()), ((ArrayNode)merged.get(fieldId; +} else { + merged.put(fieldId, content.get(fieldId)); } } else { merged.put(fieldId, content.get(fieldId)); @@ -136,4 +140,14 @@ public static ObjectNode mergeProperties(ObjectNode content, ObjectNode parent) return merged; } + /** + * merge two arrays. + * @param content ArrayNode + * @param parent ArrayNode + * @return merged ArrayNode + */ + private static ArrayNode mergeArrays(ArrayNode content, ArrayNode parent) { +return parent.deepCopy().addAll(content); + } + } diff --git a/streams-util/src/test/java/org/apache/streams/util/schema/test/PropertyUtilTest.java b/streams-util/src/test/java/org/apache/streams/util/test/PropertyUtilTest.java similarity index 84% rename from streams-util/src/test/java/org/apache/streams/util/schema/test/PropertyUtilTest.java rename to streams-util/src/test/java/org/apache/streams/util/test/PropertyUtilTest.java index 062d719e1..fdba3f029 100644 --- a/streams-util/src/test/java/org/apache/streams/util/schema/test/PropertyUtilTest.java +++ b/streams-util/src/test/java/org/apache/streams/util/test/PropertyUtilTest.java @@ -16,13 +16,14 @@ * under the License. */ -package org.apache.streams.util.schema.test; +package org.apache.streams.util.test; import org.apache.streams.util.PropertyUtil; import org.apache.streams.util.schema.SchemaUtil; import com.fasterxml.jackson.databind.JsonNode; import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.node.ArrayNode; import com.fasterxml.jackson.databind.node.ObjectNode; import org.junit.Test; @@ -56,13 +57,19 @@ public void testUnflattenObjectNode() throws Exception { public void testMergeProperties() throws Exception { ObjectMapper mapper = new ObjectMapper(); ObjectNode left = mapper.createObjectNode(); -left.put("objectType", "post").putObject("actor").putObject("author").put("id", "authorId"); +left.put("objectType", "post"); +left.putArray("links").add("link1"); +left.putObject("actor").put("id", "leftId").putObject("author").put("id", "authorId"); ObjectNode right = mapper.createObjectNode(); -right.putObject("actor").put("id", "actorId").putObject("image").put("url", "http://url.com";); +right.putArray("links").add("link2"); +right.putObject("actor").put("id", "rightId").putObject("image").put("url", "http://url.com";); JsonNode merged = PropertyUtil.mergeProperties(left, right); assert( merged.has("objectType")); assert( merged.has("actor")); +assert( merged.has("links")); +assert( (merged.get("links")).size() == 2); assert( merged.get("actor").has("id")); +assert( merged.get("actor").get("id").asText().equals("leftId")); assert( merged.get("actor").has("author")); assert( merged.get("actor").get("author").has("id")); assert( merged.get("actor").has("image")); This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Bug in PropertyUtil.mergeProperties > --- > > Key: STREAMS-549 > URL: https://issues.apache.org/jira/browse/STREAMS-549 > Project: Streams
[jira] [Commented] (STREAMS-552) Disable .debug() by default in org.apache.streams.twitter.api.Twitter
[ https://issues.apache.org/jira/browse/STREAMS-552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277481#comment-16277481 ] ASF GitHub Bot commented on STREAMS-552: steveblackmon opened a new pull request #404: STREAMS-552 URL: https://github.com/apache/streams/pull/404 STREAMS-552: Disable .debug() by default in org.apache.streams.twitter.api.Twitter (https://issues.apache.org/jira/browse/STREAMS-552) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Disable .debug() by default in org.apache.streams.twitter.api.Twitter > - > > Key: STREAMS-552 > URL: https://issues.apache.org/jira/browse/STREAMS-552 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Debug logging is nice when running integration tests but should not be > enabled by default. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-552) Disable .debug() by default in org.apache.streams.twitter.api.Twitter
[ https://issues.apache.org/jira/browse/STREAMS-552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277564#comment-16277564 ] ASF GitHub Bot commented on STREAMS-552: steveblackmon closed pull request #404: STREAMS-552 URL: https://github.com/apache/streams/pull/404 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Disable .debug() by default in org.apache.streams.twitter.api.Twitter > - > > Key: STREAMS-552 > URL: https://issues.apache.org/jira/browse/STREAMS-552 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Debug logging is nice when running integration tests but should not be > enabled by default. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-553) Implement Premium Search in streams-provider-twitter
[ https://issues.apache.org/jira/browse/STREAMS-553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277566#comment-16277566 ] ASF GitHub Bot commented on STREAMS-553: steveblackmon opened a new pull request #405: STREAMS-553: Implement Premium Search in streams-provider-twitter URL: https://github.com/apache/streams/pull/405 STREAMS-553: Implement Premium Search in streams-provider-twitter (https://issues.apache.org/jira/browse/STREAMS-553) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Implement Premium Search in streams-provider-twitter > > > Key: STREAMS-553 > URL: https://issues.apache.org/jira/browse/STREAMS-553 > Project: Streams > Issue Type: New Feature >Reporter: Steve Blackmon > > https://developer.twitter.com/en/docs/tweets/search/api-reference/premium-search -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-553) Implement Premium Search in streams-provider-twitter
[ https://issues.apache.org/jira/browse/STREAMS-553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277885#comment-16277885 ] ASF GitHub Bot commented on STREAMS-553: steveblackmon closed pull request #405: STREAMS-553: Implement Premium Search in streams-provider-twitter URL: https://github.com/apache/streams/pull/405 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java new file mode 100644 index 0..313832b9a --- /dev/null +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java @@ -0,0 +1,48 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.api; + +import org.apache.juneau.remoteable.Body; +import org.apache.juneau.remoteable.Path; +import org.apache.juneau.remoteable.QueryIfNE; +import org.apache.juneau.remoteable.RemoteMethod; +import org.apache.juneau.remoteable.Remoteable; + + +/** + * Interface for /search methods. + */ +@Remoteable(path = "https://api.twitter.com/1.1/"+ThirtyDaySearch.path) +public interface ThirtyDaySearch { + + String path = "tweets/search/30day"; + + /** + * Returns a collection of relevant Tweets matching a specified query. + * + * @param environment "environment to use" + * @param searchRequest {@link org.apache.streams.twitter.api.ThirtyDaySearchRequest} + * @return {@link ThirtyDaySearchResponse} + * @see https://developer.twitter.com/en/docs/tweets/search/api-reference/30-day-search";>https://developer.twitter.com/en/docs/tweets/search/api-reference/30-day-search + * + */ + @RemoteMethod(httpMethod = "POST", path = "/{environment}.json") + public ThirtyDaySearchResponse thirtyDaySearch(@Path("environment") String environment, @Body ThirtyDaySearchRequest searchRequest); + +} diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearchCounts.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearchCounts.java new file mode 100644 index 0..cb89155d5 --- /dev/null +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearchCounts.java @@ -0,0 +1,45 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.api; + +import org.apache.juneau.remoteable.Body; +import org.apache.juneau.remoteable.Path; +import org.apache.juneau.remoteable.RemoteMethod; +import org.apache.juneau.remoteable.Remoteable; + + +/** + * Interface for /search methods. + */ +@Remoteable(path = "https://api.twitter.com/1.1/"+ ThirtyDaySearch.path) +public interface ThirtyDaySearchCounts { + + /** + * Returns counts of relevant Tweets matching a specified query. + * + * @param environment "environment to use" + * @param searchCountsRequest {@link org.apache.streams.twitter.api.ThirtyDaySearchCountsRequest} + * @return {@link ThirtyDaySearchCountsResponse} + * @see https://developer.twitter.com/en/docs/tweets/search/api-reference/30-day-searc
[jira] [Commented] (STREAMS-555) Test in streams-util failing - because http://json-schema.org/schema has changed
[ https://issues.apache.org/jira/browse/STREAMS-555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277887#comment-16277887 ] ASF GitHub Bot commented on STREAMS-555: steveblackmon opened a new pull request #406: STREAMS-555: Test in streams-util failing URL: https://github.com/apache/streams/pull/406 STREAMS-555: Test in streams-util failing - because http://json-schema.org/schema has changed (https://issues.apache.org/jira/browse/STREAMS-555) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Test in streams-util failing - because http://json-schema.org/schema has > changed > > > Key: STREAMS-555 > URL: https://issues.apache.org/jira/browse/STREAMS-555 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > The content behind http://json-schema.org/schema has changed to a newer draft > that is not compatible with the jsonschema v3 used by Apache Streams. There > are a few places where that links is used in the project, which we need to > change to reference http://json-schema.org/draft-03/schema to fix the test. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-555) Test in streams-util failing - because http://json-schema.org/schema has changed
[ https://issues.apache.org/jira/browse/STREAMS-555?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16277888#comment-16277888 ] ASF GitHub Bot commented on STREAMS-555: steveblackmon closed pull request #406: STREAMS-555: Test in streams-util failing URL: https://github.com/apache/streams/pull/406 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-schemas/streams-schema-activitystreams/src/main/jsonschema/collection.json b/streams-schemas/streams-schema-activitystreams/src/main/jsonschema/collection.json index 5606c0d4b..6089bd06d 100644 --- a/streams-schemas/streams-schema-activitystreams/src/main/jsonschema/collection.json +++ b/streams-schemas/streams-schema-activitystreams/src/main/jsonschema/collection.json @@ -34,14 +34,14 @@ "optional": true, "description": "Links between an this object and other resources as defined in Web Linking", "extends": { -"$ref": "http://json-schema.org/links#properties"; +"$ref": "http://json-schema.org/draft-03/links#properties"; } } }, "links": [ { "rel": "describedby", -"href": "http://json-schema.org/schema"; +"href": "http://json-schema.org/draft-03/schema"; } ] } \ No newline at end of file This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Test in streams-util failing - because http://json-schema.org/schema has > changed > > > Key: STREAMS-555 > URL: https://issues.apache.org/jira/browse/STREAMS-555 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > The content behind http://json-schema.org/schema has changed to a newer draft > that is not compatible with the jsonschema v3 used by Apache Streams. There > are a few places where that links is used in the project, which we need to > change to reference http://json-schema.org/draft-03/schema to fix the test. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-563) Remove "incubating" from all pom.xmls
[ https://issues.apache.org/jira/browse/STREAMS-563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16300671#comment-16300671 ] ASF GitHub Bot commented on STREAMS-563: rawkintrevo opened a new pull request #408: STREAMS-563 Remove Incubating from all pom.xml URL: https://github.com/apache/streams/pull/408 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove "incubating" from all pom.xmls > - > > Key: STREAMS-563 > URL: https://issues.apache.org/jira/browse/STREAMS-563 > Project: Streams > Issue Type: Access >Reporter: Trevor Grant > -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-557) RiakHttpClient would be more useful if it exposed baseUrl publically
[ https://issues.apache.org/jira/browse/STREAMS-557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16301980#comment-16301980 ] ASF GitHub Bot commented on STREAMS-557: steveblackmon opened a new pull request #409: STREAMS-557 URL: https://github.com/apache/streams/pull/409 STREAMS-557: RiakHttpClient would be more useful if it exposed baseUrl publically (https://issues.apache.org/jira/browse/STREAMS-557) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > RiakHttpClient would be more useful if it exposed baseUrl publically > > > Key: STREAMS-557 > URL: https://issues.apache.org/jira/browse/STREAMS-557 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon > > CloseableHttpClient doesn't handle appending path fragments to base paths, so > RiakHttpClient would be more useful if it exposed baseUrl publicly so the > library using it could do so -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-563) Remove "incubating" from all pom.xmls
[ https://issues.apache.org/jira/browse/STREAMS-563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302021#comment-16302021 ] ASF GitHub Bot commented on STREAMS-563: rawkintrevo commented on issue #408: STREAMS-563 Remove Incubating from all pom.xml URL: https://github.com/apache/streams/pull/408#issuecomment-353685016 ah- my bad. Thanks @steveblackmon This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove "incubating" from all pom.xmls > - > > Key: STREAMS-563 > URL: https://issues.apache.org/jira/browse/STREAMS-563 > Project: Streams > Issue Type: Access >Reporter: Trevor Grant > -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-557) RiakHttpClient would be more useful if it exposed baseUrl publically
[ https://issues.apache.org/jira/browse/STREAMS-557?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302240#comment-16302240 ] ASF GitHub Bot commented on STREAMS-557: steveblackmon closed pull request #409: STREAMS-557 URL: https://github.com/apache/streams/pull/409 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-persist-riak/src/main/java/org/apache/streams/riak/http/RiakHttpClient.java b/streams-contrib/streams-persist-riak/src/main/java/org/apache/streams/riak/http/RiakHttpClient.java index 7155b52b4..c5a64814b 100644 --- a/streams-contrib/streams-persist-riak/src/main/java/org/apache/streams/riak/http/RiakHttpClient.java +++ b/streams-contrib/streams-persist-riak/src/main/java/org/apache/streams/riak/http/RiakHttpClient.java @@ -102,4 +102,7 @@ public HttpClient client() { return client; } + public URI getBaseURI() { +return baseURI; + } } This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > RiakHttpClient would be more useful if it exposed baseUrl publically > > > Key: STREAMS-557 > URL: https://issues.apache.org/jira/browse/STREAMS-557 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon > > CloseableHttpClient doesn't handle appending path fragments to base paths, so > RiakHttpClient would be more useful if it exposed baseUrl publicly so the > library using it could do so -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-565) Permit access to StreamsConfigurator underlying Config without resolve()
[ https://issues.apache.org/jira/browse/STREAMS-565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302241#comment-16302241 ] ASF GitHub Bot commented on STREAMS-565: steveblackmon opened a new pull request #410: STREAMS-565 URL: https://github.com/apache/streams/pull/410 STREAMS-565: Permit access to StreamsConfigurator underlying Config without resolve() (https://issues.apache.org/jira/browse/STREAMS-565) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Permit access to StreamsConfigurator underlying Config without resolve() > > > Key: STREAMS-565 > URL: https://issues.apache.org/jira/browse/STREAMS-565 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Currently the only way to get access to the underlying Config is with > getConfig, which calls resolve() > Add a new public method that allows access without resolve() -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-565) Permit access to StreamsConfigurator underlying Config without resolve()
[ https://issues.apache.org/jira/browse/STREAMS-565?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302519#comment-16302519 ] ASF GitHub Bot commented on STREAMS-565: steveblackmon closed pull request #410: STREAMS-565 URL: https://github.com/apache/streams/pull/410 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-config/src/main/java/org/apache/streams/config/StreamsConfigurator.java b/streams-config/src/main/java/org/apache/streams/config/StreamsConfigurator.java index 4c1c14bf7..d7f7805b6 100644 --- a/streams-config/src/main/java/org/apache/streams/config/StreamsConfigurator.java +++ b/streams-config/src/main/java/org/apache/streams/config/StreamsConfigurator.java @@ -22,6 +22,7 @@ import com.typesafe.config.Config; import com.typesafe.config.ConfigFactory; import com.typesafe.config.ConfigRenderOptions; +import com.typesafe.config.ConfigResolveOptions; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -49,6 +50,10 @@ public static Config getConfig() { return config.resolve(); } + public static Config rawConfig() { +return config; + } + public static void addConfig(Config newConfig) { config = newConfig.withFallback(config); } This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Permit access to StreamsConfigurator underlying Config without resolve() > > > Key: STREAMS-565 > URL: https://issues.apache.org/jira/browse/STREAMS-565 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Currently the only way to get access to the underlying Config is with > getConfig, which calls resolve() > Add a new public method that allows access without resolve() -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-556) Twitter: SevenDaySearchProvider
[ https://issues.apache.org/jira/browse/STREAMS-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302536#comment-16302536 ] ASF GitHub Bot commented on STREAMS-556: steveblackmon opened a new pull request #411: STREAMS-556: Twitter: SevenDaySearchProvider URL: https://github.com/apache/streams/pull/411 (https://issues.apache.org/jira/browse/STREAMS-556) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Twitter: SevenDaySearchProvider > --- > > Key: STREAMS-556 > URL: https://issues.apache.org/jira/browse/STREAMS-556 > Project: Streams > Issue Type: New Feature >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Provider that pages through SevenDaySearch results collecting Tweets until a > limit is hit or there are no more posts. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-556) Twitter: SevenDaySearchProvider
[ https://issues.apache.org/jira/browse/STREAMS-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302602#comment-16302602 ] ASF GitHub Bot commented on STREAMS-556: steveblackmon closed pull request #411: STREAMS-556: Twitter: SevenDaySearchProvider URL: https://github.com/apache/streams/pull/411 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/SevenDaySearch.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/SevenDaySearch.java index 92bdf4def..696cd5317 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/SevenDaySearch.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/SevenDaySearch.java @@ -34,12 +34,12 @@ /** * Returns a collection of relevant Tweets matching a specified query. * - * @return {@link AccountSettings} - * @see https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-settings";>https://developer.twitter.com/en/docs/accounts-and-users/manage-account-settings/api-reference/get-account-settings + * @return {@link SevenDaySearchResponse} + * @see https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-tweets";>https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-tweets * */ @RemoteMethod(httpMethod = "GET", path = "/tweets.json") - public SevenDaySearchResponse sevenDaySearch(@QueryIfNE SevenDaySearchRequest parameters); + public SevenDaySearchResponse sevenDaySearch(@QueryIfNE("*") SevenDaySearchRequest parameters); } diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/SevenDaySearchProvider.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/SevenDaySearchProvider.java new file mode 100644 index 0..fe4fdcc57 --- /dev/null +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/SevenDaySearchProvider.java @@ -0,0 +1,332 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.provider; + +import org.apache.streams.config.ComponentConfigurator; +import org.apache.streams.config.StreamsConfiguration; +import org.apache.streams.config.StreamsConfigurator; +import org.apache.streams.core.DatumStatusCounter; +import org.apache.streams.core.StreamsDatum; +import org.apache.streams.core.StreamsProvider; +import org.apache.streams.core.StreamsResultSet; +import org.apache.streams.jackson.StreamsJacksonMapper; +import org.apache.streams.twitter.SevenDaySearchProviderConfiguration; +import org.apache.streams.twitter.TwitterTimelineProviderConfiguration; +import org.apache.streams.twitter.api.SevenDaySearchRequest; +import org.apache.streams.twitter.api.StatusesUserTimelineRequest; +import org.apache.streams.twitter.api.Twitter; +import org.apache.streams.twitter.converter.TwitterDateTimeFormat; + +import com.fasterxml.jackson.core.JsonProcessingException; +import com.fasterxml.jackson.databind.ObjectMapper; +import com.google.common.base.Preconditions; +import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.ListenableFuture; +import com.google.common.util.concurrent.ListeningExecutorService; +import com.google.common.util.concurrent.MoreExecutors; +import com.google.common.util.concurrent.Uninterruptibles; +import com.typesafe.config.Config; +import com.typesafe.config.ConfigFactory; +import com.typesafe.config.ConfigParseOptions; +import org.apache.commons.lang.NotImplementedException; +import org.joda.time.DateTime; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + +import java.io.BufferedOutputStream; +import java.io.File; +import java.io.FileOutputStream; +import java.io.PrintStream; +im
[jira] [Commented] (STREAMS-566) Twitter: ThirtyDaySearchProvider
[ https://issues.apache.org/jira/browse/STREAMS-566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302603#comment-16302603 ] ASF GitHub Bot commented on STREAMS-566: steveblackmon opened a new pull request #412: STREAMS-566: Twitter: ThirtyDaySearchProvider URL: https://github.com/apache/streams/pull/412 STREAMS-566: Twitter: ThirtyDaySearchProvider (https://issues.apache.org/jira/browse/STREAMS-566) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Twitter: ThirtyDaySearchProvider > > > Key: STREAMS-566 > URL: https://issues.apache.org/jira/browse/STREAMS-566 > Project: Streams > Issue Type: New Feature >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Provider that pages through ThirtyDaySearch results collecting Tweets until a > limit is hit or there are no more posts. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-566) Twitter: ThirtyDaySearchProvider
[ https://issues.apache.org/jira/browse/STREAMS-566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302739#comment-16302739 ] ASF GitHub Bot commented on STREAMS-566: steveblackmon closed pull request #412: STREAMS-566: Twitter: ThirtyDaySearchProvider URL: https://github.com/apache/streams/pull/412 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java index 313832b9a..e909fa76c 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/ThirtyDaySearch.java @@ -39,7 +39,7 @@ * @param environment "environment to use" * @param searchRequest {@link org.apache.streams.twitter.api.ThirtyDaySearchRequest} * @return {@link ThirtyDaySearchResponse} - * @see https://developer.twitter.com/en/docs/tweets/search/api-reference/30-day-search";>https://developer.twitter.com/en/docs/tweets/search/api-reference/30-day-search + * @see https://developer.twitter.com/en/docs/tweets/search/overview/premium";>https://developer.twitter.com/en/docs/tweets/search/overview/premium * */ @RemoteMethod(httpMethod = "POST", path = "/{environment}.json") diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/ThirtyDaySearchProvider.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/ThirtyDaySearchProvider.java new file mode 100644 index 0..bf67fb2c1 --- /dev/null +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/provider/ThirtyDaySearchProvider.java @@ -0,0 +1,330 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.twitter.provider; + +import org.apache.streams.config.ComponentConfigurator; +import org.apache.streams.config.StreamsConfiguration; +import org.apache.streams.config.StreamsConfigurator; +import org.apache.streams.core.DatumStatusCounter; +import org.apache.streams.core.StreamsDatum; +import org.apache.streams.core.StreamsProvider; +import org.apache.streams.core.StreamsResultSet; +import org.apache.streams.jackson.StreamsJacksonMapper; +import org.apache.streams.twitter.ThirtyDaySearchProviderConfiguration; +import org.apache.streams.twitter.api.ThirtyDaySearchRequest; +import org.apache.streams.twitter.api.Twitter; +import org.apache.streams.twitter.converter.TwitterDateTimeFormat; + +import com.fasterxml.jackson.core.JsonProcessingException; +import com.fasterxml.jackson.databind.ObjectMapper; +import com.google.common.base.Preconditions; +import com.google.common.util.concurrent.Futures; +import com.google.common.util.concurrent.ListenableFuture; +import com.google.common.util.concurrent.ListeningExecutorService; +import com.google.common.util.concurrent.MoreExecutors; +import com.google.common.util.concurrent.Uninterruptibles; +import com.typesafe.config.Config; +import com.typesafe.config.ConfigFactory; +import com.typesafe.config.ConfigParseOptions; +import org.apache.commons.lang.NotImplementedException; +import org.joda.time.DateTime; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + +import java.io.BufferedOutputStream; +import java.io.File; +import java.io.FileOutputStream; +import java.io.PrintStream; +import java.io.Serializable; +import java.math.BigInteger; +import java.util.ArrayList; +import java.util.List; +import java.util.Objects; +import java.util.Queue; +import java.util.concurrent.ExecutorService; +import java.util.concurrent.LinkedBlockingQueue; +import java.util.concurrent.TimeUnit; +import java.util.concurrent.atomic.AtomicBoolean; +import java.util.concurrent.locks.ReadWriteLock; +import java.util.c
[jira] [Commented] (STREAMS-564) Update dependency versions on juneau and httpclient
[ https://issues.apache.org/jira/browse/STREAMS-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16302740#comment-16302740 ] ASF GitHub Bot commented on STREAMS-564: steveblackmon opened a new pull request #413: STREAMS-564 URL: https://github.com/apache/streams/pull/413 STREAMS-564: Update dependency versions on juneau and httpclient (https://issues.apache.org/jira/browse/STREAMS-564) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update dependency versions on juneau and httpclient > --- > > Key: STREAMS-564 > URL: https://issues.apache.org/jira/browse/STREAMS-564 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Juneau is about to release 7.0.1 with newer httpclient and jetty dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-564) Update dependency versions on juneau and httpclient
[ https://issues.apache.org/jira/browse/STREAMS-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16303397#comment-16303397 ] ASF GitHub Bot commented on STREAMS-564: steveblackmon closed pull request #413: STREAMS-564 URL: https://github.com/apache/streams/pull/413 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index c7d17669c..713885fd9 100644 --- a/pom.xml +++ b/pom.xml @@ -361,7 +361,7 @@ 1.8.1 0.4.1 0.1.10 -7.0.0 +7.0.1 0.4.10 0.13.1 0.11.1 @@ -377,7 +377,7 @@ 2.5 3.6 1.6 -1.3.1 +1.3.2 0.9.11 20.0 1.4.0 @@ -387,8 +387,8 @@ 2.4.7 1.10.19 1.7.1 -4.4.6 -4.5.3 +4.4.8 +4.5.4 1.7 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update dependency versions on juneau and httpclient > --- > > Key: STREAMS-564 > URL: https://issues.apache.org/jira/browse/STREAMS-564 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Juneau is about to release 7.0.1 with newer httpclient and jetty dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-564) Update dependency versions on juneau and httpclient
[ https://issues.apache.org/jira/browse/STREAMS-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16303398#comment-16303398 ] ASF GitHub Bot commented on STREAMS-564: steveblackmon opened a new pull request #413: STREAMS-564 URL: https://github.com/apache/streams/pull/413 STREAMS-564: Update dependency versions on juneau and httpclient (https://issues.apache.org/jira/browse/STREAMS-564) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update dependency versions on juneau and httpclient > --- > > Key: STREAMS-564 > URL: https://issues.apache.org/jira/browse/STREAMS-564 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Juneau is about to release 7.0.1 with newer httpclient and jetty dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-564) Update dependency versions on juneau and httpclient
[ https://issues.apache.org/jira/browse/STREAMS-564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16304984#comment-16304984 ] ASF GitHub Bot commented on STREAMS-564: steveblackmon closed pull request #413: STREAMS-564 URL: https://github.com/apache/streams/pull/413 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index c7d17669c..713885fd9 100644 --- a/pom.xml +++ b/pom.xml @@ -361,7 +361,7 @@ 1.8.1 0.4.1 0.1.10 -7.0.0 +7.0.1 0.4.10 0.13.1 0.11.1 @@ -377,7 +377,7 @@ 2.5 3.6 1.6 -1.3.1 +1.3.2 0.9.11 20.0 1.4.0 @@ -387,8 +387,8 @@ 2.4.7 1.10.19 1.7.1 -4.4.6 -4.5.3 +4.4.8 +4.5.4 1.7 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update dependency versions on juneau and httpclient > --- > > Key: STREAMS-564 > URL: https://issues.apache.org/jira/browse/STREAMS-564 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Juneau is about to release 7.0.1 with newer httpclient and jetty dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-567) failing IT in streams-plugin-pojo
[ https://issues.apache.org/jira/browse/STREAMS-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16305640#comment-16305640 ] ASF GitHub Bot commented on STREAMS-567: steveblackmon opened a new pull request #414: STREAMS-567 URL: https://github.com/apache/streams/pull/414 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > failing IT in streams-plugin-pojo > - > > Key: STREAMS-567 > URL: https://issues.apache.org/jira/browse/STREAMS-567 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > I found a failing IT in streams-plugin-pojo - juneau annotations are added to > generated classes but the integration test for the process does not have > access to juneau dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-567) failing IT in streams-plugin-pojo
[ https://issues.apache.org/jira/browse/STREAMS-567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16309856#comment-16309856 ] ASF GitHub Bot commented on STREAMS-567: steveblackmon closed pull request #414: STREAMS-567 URL: https://github.com/apache/streams/pull/414 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-plugins/streams-plugin-pojo/src/test/resources/streams-plugin-pojo/pom.xml b/streams-plugins/streams-plugin-pojo/src/test/resources/streams-plugin-pojo/pom.xml index 7b0124242..337168d59 100644 --- a/streams-plugins/streams-plugin-pojo/src/test/resources/streams-plugin-pojo/pom.xml +++ b/streams-plugins/streams-plugin-pojo/src/test/resources/streams-plugin-pojo/pom.xml @@ -30,6 +30,7 @@ 2.6.1 +7.0.1 3.4 @@ -122,6 +123,11 @@ jackson-datatype-joda ${jackson.version} + +org.apache.juneau +juneau-marshall +${juneau.version} + org.apache.commons commons-lang3 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > failing IT in streams-plugin-pojo > - > > Key: STREAMS-567 > URL: https://issues.apache.org/jira/browse/STREAMS-567 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > I found a failing IT in streams-plugin-pojo - juneau annotations are added to > generated classes but the integration test for the process does not have > access to juneau dependencies. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-563) Remove "incubating" from all pom.xmls
[ https://issues.apache.org/jira/browse/STREAMS-563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16311494#comment-16311494 ] ASF GitHub Bot commented on STREAMS-563: rawkintrevo closed pull request #408: STREAMS-563 Remove Incubating from all pom.xml URL: https://github.com/apache/streams/pull/408 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml b/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml index 89056fa96..16730c4b9 100644 --- a/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml +++ b/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml @@ -23,7 +23,7 @@ org.apache.streams.plugins streams-plugin-cassandra-test -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT jar Test StreamsCassandraResourceGeneratorMojo @@ -98,7 +98,7 @@ org.apache.streams.plugins streams-plugin-cassandra -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT target/test-classes/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml b/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml index 87b536dbe..cd38d7a23 100644 --- a/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml +++ b/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml @@ -23,7 +23,7 @@ org.apache.streams.plugins streams-plugin-elasticsearch-test -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT jar Test StreamsElasticsearchResourceGeneratorMojo @@ -98,7 +98,7 @@ org.apache.streams.plugins streams-plugin-elasticsearch -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT target/test-classes/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml b/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml index 67799a724..78e83f16f 100644 --- a/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml +++ b/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml @@ -24,7 +24,7 @@ org.apache.streams.plugins streams-plugin-hbase-test -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT jar Test StreamsHbaseResourceGeneratorMojo @@ -99,7 +99,7 @@ org.apache.streams.plugins streams-plugin-hbase -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT target/test-classes/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml b/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml index 5e338e7df..6d0b949e4 100644 --- a/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml +++ b/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml @@ -24,7 +24,7 @@ org.apache.streams.plugins streams-plugin-hive-test -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT jar Test StreamsPojoHiveMojo @@ -99,7 +99,7 @@ org.apache.streams.plugins streams-plugin-hive -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT target/test-classes/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-pig/src/test/resources/streams-plugin-pig/pom.xml b/streams-plugins/streams-plugin-pig/src/test/resources/streams-plugin-pig/pom.xml index 4fe6f49d2..6c3cc5a19 100644 --- a/streams-plugins/streams-plugin-pig/src/test/resources/streams-plugin-pig/pom.xml +++ b/streams-plugins/streams-plugin-pig/src/test/resources/streams-plugin-pig/pom.xml @@ -24,7 +24,7 @@ org.apache.streams.plugins streams-plugin-pig-test -0.5-incubating-SNAPSHOT +0.5.1-SNAPSHOT jar Test StreamsPigResourceGeneratorMojo @@ -99,7 +99,7 @@
[jira] [Commented] (STREAMS-570) Update enabled=false on TwitterThirtyDaySearchProviderIT
[ https://issues.apache.org/jira/browse/STREAMS-570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16313375#comment-16313375 ] ASF GitHub Bot commented on STREAMS-570: rawkintrevo opened a new pull request #416: STREAMS-570 Disable TwitterThirtyDaySearchProviderIT URL: https://github.com/apache/streams/pull/416 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update enabled=false on TwitterThirtyDaySearchProviderIT > > > Key: STREAMS-570 > URL: https://issues.apache.org/jira/browse/STREAMS-570 > Project: Streams > Issue Type: Bug > Components: Tests >Reporter: Trevor Grant >Assignee: Trevor Grant > Fix For: 0.6 > > > The thirty day provider is only available on Enterprise creds (which is why > it is disabled), however test fails on `mvn clean verify $REPO > -DskipTests=true -DskipITs=false` (which is part of `release.sh`) > The reason is basically described here: > https://github.com/cbeust/testng/issues/151 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-570) Update enabled=false on TwitterThirtyDaySearchProviderIT
[ https://issues.apache.org/jira/browse/STREAMS-570?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16313418#comment-16313418 ] ASF GitHub Bot commented on STREAMS-570: rawkintrevo closed pull request #416: STREAMS-570 Disable TwitterThirtyDaySearchProviderIT URL: https://github.com/apache/streams/pull/416 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/providers/TwitterThirtyDaySearchProviderIT.java b/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/providers/TwitterThirtyDaySearchProviderIT.java index b4fd3ce51..c6609af80 100644 --- a/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/providers/TwitterThirtyDaySearchProviderIT.java +++ b/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/providers/TwitterThirtyDaySearchProviderIT.java @@ -36,8 +36,10 @@ public class TwitterThirtyDaySearchProviderIT { private static final Logger LOGGER = LoggerFactory.getLogger(TwitterThirtyDaySearchProviderIT.class); - - @Test(groups = "TwitterThirtyDaySearchProviderIT") + /* + Must disble methods individually if using Test annotation, see STREAMS-570 + */ + @Test(groups = "TwitterThirtyDaySearchProviderIT", enabled = false) public void testTwitterThirtyDaySearchProvider() throws Exception { String configfile = "./target/test-classes/TwitterThirtyDaySearchProviderIT.conf"; This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update enabled=false on TwitterThirtyDaySearchProviderIT > > > Key: STREAMS-570 > URL: https://issues.apache.org/jira/browse/STREAMS-570 > Project: Streams > Issue Type: Bug > Components: Tests >Reporter: Trevor Grant >Assignee: Trevor Grant > Fix For: 0.6 > > > The thirty day provider is only available on Enterprise creds (which is why > it is disabled), however test fails on `mvn clean verify $REPO > -DskipTests=true -DskipITs=false` (which is part of `release.sh`) > The reason is basically described here: > https://github.com/cbeust/testng/issues/151 -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-574) Merge streams-examples into streams repo
[ https://issues.apache.org/jira/browse/STREAMS-574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16319738#comment-16319738 ] ASF GitHub Bot commented on STREAMS-574: steveblackmon opened a new pull request #417: STREAMS-574: Merge streams-examples into streams repo URL: https://github.com/apache/streams/pull/417 STREAMS-574: Merge streams-examples into streams repo (https://issues.apache.org/jira/browse/STREAMS-574) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Merge streams-examples into streams repo > > > Key: STREAMS-574 > URL: https://issues.apache.org/jira/browse/STREAMS-574 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Doing this should simplify many aspects of developing the project. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-550) Twitter: Retweets Provider
[ https://issues.apache.org/jira/browse/STREAMS-550?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16320461#comment-16320461 ] ASF GitHub Bot commented on STREAMS-550: steveblackmon opened a new pull request #418: STREAMS-550: Twitter: Retweets Provider URL: https://github.com/apache/streams/pull/418 Twitter.java: Support for collecting retweets and retweeters of a post. TwitterEngagersProvider.java: A new provider that collects all available retweeters across a full timeline for specified user(s) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Twitter: Retweets Provider > -- > > Key: STREAMS-550 > URL: https://issues.apache.org/jira/browse/STREAMS-550 > Project: Streams > Issue Type: New Feature >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 4h > Remaining Estimate: 4h > > Build a provider that collects all available retweets given a list of post > ids. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-560) Instagram: Likes Provider
[ https://issues.apache.org/jira/browse/STREAMS-560?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16320523#comment-16320523 ] ASF GitHub Bot commented on STREAMS-560: steveblackmon opened a new pull request #419: STREAMS-560: Instagram: Likes Provider URL: https://github.com/apache/streams/pull/419 InstagramLikers[Collector,Provider]: Likers from a set of post ids InstagramCollects[Collector,Provider]: Comments for a set of post ids InstagramEngagers[Collector,Provider]: Likers and Commenters for a userās recent posts This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Instagram: Likes Provider > - > > Key: STREAMS-560 > URL: https://issues.apache.org/jira/browse/STREAMS-560 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon > Original Estimate: 1h > Remaining Estimate: 1h > > Provider that generates 'like' activities given a list of post ids. -- This message was sent by Atlassian JIRA (v6.4.14#64029)
[jira] [Commented] (STREAMS-574) Merge streams-examples into streams repo
[ https://issues.apache.org/jira/browse/STREAMS-574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16330817#comment-16330817 ] ASF GitHub Bot commented on STREAMS-574: steveblackmon closed pull request #417: STREAMS-574: Merge streams-examples into streams repo URL: https://github.com/apache/streams/pull/417 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Merge streams-examples into streams repo > > > Key: STREAMS-574 > URL: https://issues.apache.org/jira/browse/STREAMS-574 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > Doing this should simplify many aspects of developing the project. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-580) Namespacing with fallbacks in ComponentConfigurator
[ https://issues.apache.org/jira/browse/STREAMS-580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16349018#comment-16349018 ] ASF GitHub Bot commented on STREAMS-580: steveblackmon opened a new pull request #421: STREAMS-580: Namespacing with fallbacks in ComponentConfigurator URL: https://github.com/apache/streams/pull/421 STREAMS-580: Namespacing with fallbacks in ComponentConfigurator (https://issues.apache.org/jira/browse/STREAMS-580) This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Namespacing with fallbacks in ComponentConfigurator > --- > > Key: STREAMS-580 > URL: https://issues.apache.org/jira/browse/STREAMS-580 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > Add support for class and package based namespacing in ComponentConfigurator: > - Add a new method to ComponentConfigurator - detectConfiguration() > - When the caller does not provide a Config or a path to detectConfiguration, > get fancy: > - Look for the fields declared by the component POJO class on each of the > following: > - the SimpleClassName > - the CanonicalClassName > - each ancestor package of the CanonicalClassName, longest to shortest > - if a field is specified at more than one package/class level, the class or > longest package ancestor takes precedence. > Write sufficient unit tests to demonstrate that this is working as intended. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-580) Namespacing with fallbacks in ComponentConfigurator
[ https://issues.apache.org/jira/browse/STREAMS-580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16354526#comment-16354526 ] ASF GitHub Bot commented on STREAMS-580: steveblackmon commented on issue #421: STREAMS-580: Namespacing with fallbacks in ComponentConfigurator URL: https://github.com/apache/streams/pull/421#issuecomment-363562594 Basically this changeset unlocks the ability to make changes like this one: https://github.com/steveblackmon/incubator-streams/commit/73240167909816be8fa464857240d140a4289432#diff-bac50d9f2ed1a154a60243a9511b6fb5 see especially streams-contrib/streams-provider-twitter/src/main/resources/reference.conf within the streams repository, and in any code built with streams-config or using streams-contrib This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Namespacing with fallbacks in ComponentConfigurator > --- > > Key: STREAMS-580 > URL: https://issues.apache.org/jira/browse/STREAMS-580 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > Add support for class and package based namespacing in ComponentConfigurator: > - Add a new method to ComponentConfigurator - detectConfiguration() > - When the caller does not provide a Config or a path to detectConfiguration, > get fancy: > - Look for the fields declared by the component POJO class on each of the > following: > - the SimpleClassName > - the CanonicalClassName > - each ancestor package of the CanonicalClassName, longest to shortest > - if a field is specified at more than one package/class level, the class or > longest package ancestor takes precedence. > Write sufficient unit tests to demonstrate that this is working as intended. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-580) Namespacing with fallbacks in ComponentConfigurator
[ https://issues.apache.org/jira/browse/STREAMS-580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16358966#comment-16358966 ] ASF GitHub Bot commented on STREAMS-580: jfrazee commented on issue #421: STREAMS-580: Namespacing with fallbacks in ComponentConfigurator URL: https://github.com/apache/streams/pull/421#issuecomment-364570699 +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Namespacing with fallbacks in ComponentConfigurator > --- > > Key: STREAMS-580 > URL: https://issues.apache.org/jira/browse/STREAMS-580 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > Add support for class and package based namespacing in ComponentConfigurator: > - Add a new method to ComponentConfigurator - detectConfiguration() > - When the caller does not provide a Config or a path to detectConfiguration, > get fancy: > - Look for the fields declared by the component POJO class on each of the > following: > - the SimpleClassName > - the CanonicalClassName > - each ancestor package of the CanonicalClassName, longest to shortest > - if a field is specified at more than one package/class level, the class or > longest package ancestor takes precedence. > Write sufficient unit tests to demonstrate that this is working as intended. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-580) Namespacing with fallbacks in ComponentConfigurator
[ https://issues.apache.org/jira/browse/STREAMS-580?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16358970#comment-16358970 ] ASF GitHub Bot commented on STREAMS-580: jfrazee closed pull request #421: STREAMS-580: Namespacing with fallbacks in ComponentConfigurator URL: https://github.com/apache/streams/pull/421 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Namespacing with fallbacks in ComponentConfigurator > --- > > Key: STREAMS-580 > URL: https://issues.apache.org/jira/browse/STREAMS-580 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > Add support for class and package based namespacing in ComponentConfigurator: > - Add a new method to ComponentConfigurator - detectConfiguration() > - When the caller does not provide a Config or a path to detectConfiguration, > get fancy: > - Look for the fields declared by the component POJO class on each of the > following: > - the SimpleClassName > - the CanonicalClassName > - each ancestor package of the CanonicalClassName, longest to shortest > - if a field is specified at more than one package/class level, the class or > longest package ancestor takes precedence. > Write sufficient unit tests to demonstrate that this is working as intended. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-582) Apply canonical classname name-spacing to all modules
[ https://issues.apache.org/jira/browse/STREAMS-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364572#comment-16364572 ] ASF GitHub Bot commented on STREAMS-582: steveblackmon opened a new pull request #422: STREAMS-582: Apply streams-config updated features to all modules URL: https://github.com/apache/streams/pull/422 STREAMS-582: Apply canonical classname name-spacing to all modules (https://issues.apache.org/jira/browse/STREAMS-582) This changeset fixes up all classes and tests to use ComponentConfigurator.detectConfiguration() and all examples to use StreamsConfigurator.detectCustomConfiguration() This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Apply canonical classname name-spacing to all modules > - > > Key: STREAMS-582 > URL: https://issues.apache.org/jira/browse/STREAMS-582 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > - Apply the new canonical classname name-spacing technique to the > reference.conf of all modules. > - Update all components to call the smarter detectConfiguration in their > default constructor > - Update unit and integration tests to pass with these changes incorporated -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-582) Apply canonical classname name-spacing to all modules
[ https://issues.apache.org/jira/browse/STREAMS-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16364582#comment-16364582 ] ASF GitHub Bot commented on STREAMS-582: steveblackmon commented on issue #422: STREAMS-582: Apply streams-config updated features to all modules URL: https://github.com/apache/streams/pull/422#issuecomment-365703821 i messed up the git history with this PR - i'm going to re-open with a squash commit. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Apply canonical classname name-spacing to all modules > - > > Key: STREAMS-582 > URL: https://issues.apache.org/jira/browse/STREAMS-582 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > - Apply the new canonical classname name-spacing technique to the > reference.conf of all modules. > - Update all components to call the smarter detectConfiguration in their > default constructor > - Update unit and integration tests to pass with these changes incorporated -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-582) Apply canonical classname name-spacing to all modules
[ https://issues.apache.org/jira/browse/STREAMS-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16367518#comment-16367518 ] ASF GitHub Bot commented on STREAMS-582: steveblackmon opened a new pull request #423: STREAMS-582,583 URL: https://github.com/apache/streams/pull/423 This changeset fixes up all classes and tests to use + improve documentation of classes and tests in streams-config and replaces the hard-coded shorthand config pattern with a canonical name config pattern throughout the project. All unit and integrations tests passed for me locally. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Apply canonical classname name-spacing to all modules > - > > Key: STREAMS-582 > URL: https://issues.apache.org/jira/browse/STREAMS-582 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > - Apply the new canonical classname name-spacing technique to the > reference.conf of all modules. > - Update all components to call the smarter detectConfiguration in their > default constructor > - Update unit and integration tests to pass with these changes incorporated -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-582) Apply canonical classname name-spacing to all modules
[ https://issues.apache.org/jira/browse/STREAMS-582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16415702#comment-16415702 ] ASF GitHub Bot commented on STREAMS-582: steveblackmon commented on issue #423: STREAMS-582,583 URL: https://github.com/apache/streams/pull/423#issuecomment-376543191 I will lazy consensus merge this after 24 hours. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Apply canonical classname name-spacing to all modules > - > > Key: STREAMS-582 > URL: https://issues.apache.org/jira/browse/STREAMS-582 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 1h > Remaining Estimate: 1h > > - Apply the new canonical classname name-spacing technique to the > reference.conf of all modules. > - Update all components to call the smarter detectConfiguration in their > default constructor > - Update unit and integration tests to pass with these changes incorporated -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417405#comment-16417405 ] ASF GitHub Bot commented on STREAMS-586: steveblackmon opened a new pull request #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424 now the docker containers lifetime is just the lifecycle of the integration-tests phase for one module. and integration testing should work regardless of which level of the maven hierarchy it is launched from. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Run docker containers supporting persist ITs one at a time > -- > > Key: STREAMS-586 > URL: https://issues.apache.org/jira/browse/STREAMS-586 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > The release process as currently documented requires theĀ release manager to > start a docker container for every persist module that uses docker, all at > once. > A more sensible strategy would be for each module to start it's own container > in pre-integration-test and stop it in post-integration-test. > It would be niceĀ to have a way keep the containers built during ITs running > once testing has concluded. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417518#comment-16417518 ] ASF GitHub Bot commented on STREAMS-586: smarthi commented on a change in pull request #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424#discussion_r177785989 ## File path: streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java ## @@ -25,8 +25,11 @@ import org.apache.streams.util.PropertyUtil; import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.SerializationFeature; import com.fasterxml.jackson.databind.node.ObjectNode; +import com.google.common.base.Predicate; +import com.google.common.collect.Maps; Review comment: Guava API still ? This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Run docker containers supporting persist ITs one at a time > -- > > Key: STREAMS-586 > URL: https://issues.apache.org/jira/browse/STREAMS-586 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > The release process as currently documented requires theĀ release manager to > start a docker container for every persist module that uses docker, all at > once. > A more sensible strategy would be for each module to start it's own container > in pre-integration-test and stop it in post-integration-test. > It would be niceĀ to have a way keep the containers built during ITs running > once testing has concluded. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417532#comment-16417532 ] ASF GitHub Bot commented on STREAMS-586: steveblackmon commented on a change in pull request #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424#discussion_r177788237 ## File path: streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java ## @@ -25,8 +25,11 @@ import org.apache.streams.util.PropertyUtil; import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.databind.SerializationFeature; import com.fasterxml.jackson.databind.node.ObjectNode; +import com.google.common.base.Predicate; +import com.google.common.collect.Maps; Review comment: these two imports aren't even used. i'll remove them This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Run docker containers supporting persist ITs one at a time > -- > > Key: STREAMS-586 > URL: https://issues.apache.org/jira/browse/STREAMS-586 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > The release process as currently documented requires theĀ release manager to > start a docker container for every persist module that uses docker, all at > once. > A more sensible strategy would be for each module to start it's own container > in pre-integration-test and stop it in post-integration-test. > It would be niceĀ to have a way keep the containers built during ITs running > once testing has concluded. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417549#comment-16417549 ] ASF GitHub Bot commented on STREAMS-586: steveblackmon commented on issue #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424#issuecomment-376928279 I realized that the way these poms are set up, docker:start and docker:stop are activated even when dockerITs is disabled. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Run docker containers supporting persist ITs one at a time > -- > > Key: STREAMS-586 > URL: https://issues.apache.org/jira/browse/STREAMS-586 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > The release process as currently documented requires theĀ release manager to > start a docker container for every persist module that uses docker, all at > once. > A more sensible strategy would be for each module to start it's own container > in pre-integration-test and stop it in post-integration-test. > It would be niceĀ to have a way keep the containers built during ITs running > once testing has concluded. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417559#comment-16417559 ] ASF GitHub Bot commented on STREAMS-586: smarthi commented on issue #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424#issuecomment-376930566 +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Run docker containers supporting persist ITs one at a time > -- > > Key: STREAMS-586 > URL: https://issues.apache.org/jira/browse/STREAMS-586 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > The release process as currently documented requires theĀ release manager to > start a docker container for every persist module that uses docker, all at > once. > A more sensible strategy would be for each module to start it's own container > in pre-integration-test and stop it in post-integration-test. > It would be niceĀ to have a way keep the containers built during ITs running > once testing has concluded. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-586) Run docker containers supporting persist ITs one at a time
[ https://issues.apache.org/jira/browse/STREAMS-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16417623#comment-16417623 ] ASF GitHub Bot commented on STREAMS-586: steveblackmon closed pull request #424: resolves STREAMS-586 URL: https://github.com/apache/streams/pull/424 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/.gitignore b/.gitignore index 57f210bcc..aa5412884 100644 --- a/.gitignore +++ b/.gitignore @@ -23,6 +23,14 @@ NOTICE /*.conf /*.properties +# Docker Maven Plugin property files +cassandra.properties +elasticsearch.properties +hbase.properties +mongo.properties +neo4j.properties +riak.properties + # OS generated files # .DS_Store .DS_Store? diff --git a/pom.xml b/pom.xml index 949a1033c..984711ff0 100644 --- a/pom.xml +++ b/pom.xml @@ -332,7 +332,7 @@ 2.6 2.10 1.0.3 -0.15.0 +0.24.0 1.2.1 1.4.1 2.19.1 diff --git a/streams-contrib/streams-persist-cassandra/pom.xml b/streams-contrib/streams-persist-cassandra/pom.xml index cc9c178df..5df759611 100644 --- a/streams-contrib/streams-persist-cassandra/pom.xml +++ b/streams-contrib/streams-persist-cassandra/pom.xml @@ -198,16 +198,21 @@ default true on + ${project.build.testOutputDirectory} -cassandra:3.9 +cassandra:3.11 cassandra none - ${cassandra.tcp.host}:${cassandra.tcp.port}:9042 + ${tcp.host}:${tcp.port}:9042 cassandra.properties + +Starting listening for CQL clients on /0.0.0.0:9042 +6 + true default @@ -220,7 +225,26 @@ - + + +start + +start + + +pre-integration-test + + + +stop + +stop + + +post-integration-test + + + diff --git a/streams-contrib/streams-persist-cassandra/src/test/java/org/apache/streams/cassandra/test/CassandraPersistIT.java b/streams-contrib/streams-persist-cassandra/src/test/java/org/apache/streams/cassandra/test/CassandraPersistIT.java index aa2f16d72..8a7c924ec 100644 --- a/streams-contrib/streams-persist-cassandra/src/test/java/org/apache/streams/cassandra/test/CassandraPersistIT.java +++ b/streams-contrib/streams-persist-cassandra/src/test/java/org/apache/streams/cassandra/test/CassandraPersistIT.java @@ -22,6 +22,7 @@ import org.apache.streams.cassandra.CassandraPersistReader; import org.apache.streams.cassandra.CassandraPersistWriter; import org.apache.streams.config.ComponentConfigurator; +import org.apache.streams.config.StreamsConfigurator; import org.apache.streams.core.StreamsDatum; import org.apache.streams.core.StreamsResultSet; import org.apache.streams.jackson.StreamsJacksonMapper; @@ -43,6 +44,8 @@ import java.nio.charset.StandardCharsets; import java.util.List; +import static org.testng.Assert.assertTrue; + /** * Test writing documents */ @@ -58,19 +61,15 @@ @BeforeClass public void setup() throws Exception { -Config reference = ConfigFactory.load(); -File conf_file = new File("target/test-classes/CassandraPersistIT.conf"); -assert(conf_file.exists()); -Config testResourceConfig = ConfigFactory.parseFileAnySyntax(conf_file, ConfigParseOptions.
[jira] [Commented] (STREAMS-588) Re-enable twitter-follow-neo4j example
[ https://issues.apache.org/jira/browse/STREAMS-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419221#comment-16419221 ] ASF GitHub Bot commented on STREAMS-588: steveblackmon opened a new pull request #425: resolves STREAMS-588 URL: https://github.com/apache/streams/pull/425 fixes streams-examples / .. / TwitterFollowNeo4j by adding a way to clear zero-size objects, empty arrays, and null / empty values out of objects en-route to neo4j and improving testing in PropertyUtil This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Re-enable twitter-follow-neo4j example > -- > > Key: STREAMS-588 > URL: https://issues.apache.org/jira/browse/STREAMS-588 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > twitter-follow-neo4j was disabled when streams-examples was merged into this > repo because the tests were no longer passing. > Figure out why, fix if necessary, and re-enable it. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-588) Re-enable twitter-follow-neo4j example
[ https://issues.apache.org/jira/browse/STREAMS-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419341#comment-16419341 ] ASF GitHub Bot commented on STREAMS-588: rawkintrevo commented on issue #425: resolves STREAMS-588 URL: https://github.com/apache/streams/pull/425#issuecomment-377299120 lgtm +1 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Re-enable twitter-follow-neo4j example > -- > > Key: STREAMS-588 > URL: https://issues.apache.org/jira/browse/STREAMS-588 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > twitter-follow-neo4j was disabled when streams-examples was merged into this > repo because the tests were no longer passing. > Figure out why, fix if necessary, and re-enable it. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-588) Re-enable twitter-follow-neo4j example
[ https://issues.apache.org/jira/browse/STREAMS-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419584#comment-16419584 ] ASF GitHub Bot commented on STREAMS-588: steveblackmon closed pull request #425: resolves STREAMS-588 URL: https://github.com/apache/streams/pull/425 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java b/streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java index 325669139..dc6efeb49 100644 --- a/streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java +++ b/streams-contrib/streams-persist-neo4j/src/main/java/org/apache/streams/neo4j/CypherQueryGraphHelper.java @@ -124,6 +124,7 @@ String query = createVertex.render(); ObjectNode object = MAPPER.convertValue(activityObject, ObjectNode.class); +object = PropertyUtil.getInstance(MAPPER).cleanProperties(object); Map props = PropertyUtil.getInstance(MAPPER).flattenToMap(object); Pair> queryPlusParameters = new Pair(createVertex.render(), props); @@ -171,6 +172,7 @@ String query = mergeVertex.render(); ObjectNode object = MAPPER.convertValue(activityObject, ObjectNode.class); +object = PropertyUtil.getInstance(MAPPER).cleanProperties(object); Map props = PropertyUtil.getInstance(MAPPER).flattenToMap(object); LOGGER.debug("mergeVertexRequest: ({},{})", query, props); @@ -191,6 +193,7 @@ Pair queryPlusParameters = new Pair(null, new HashMap<>()); ObjectNode object = MAPPER.convertValue(activity, ObjectNode.class); +object = PropertyUtil.getInstance(MAPPER).cleanProperties(object); Map props = PropertyUtil.getInstance(MAPPER).flattenToMap(object); ST mergeEdge = new ST(createEdgeStatementTemplate); @@ -221,6 +224,7 @@ Pair queryPlusParameters = new Pair(null, new HashMap<>()); ObjectNode object = MAPPER.convertValue(activity, ObjectNode.class); +object = PropertyUtil.getInstance(MAPPER).cleanProperties(object); Map props = PropertyUtil.getInstance(MAPPER).flattenToMap(object); ST mergeEdge = new ST(createEdgeStatementTemplate); diff --git a/streams-examples/streams-examples-local/pom.xml b/streams-examples/streams-examples-local/pom.xml index 11bd32156..9987df3a6 100644 --- a/streams-examples/streams-examples-local/pom.xml +++ b/streams-examples/streams-examples-local/pom.xml @@ -41,7 +41,7 @@ elasticsearch-hdfs elasticsearch-reindex mongo-elasticsearch-sync - +twitter-follow-neo4j twitter-history-elasticsearch twitter-userstream-elasticsearch diff --git a/streams-examples/streams-examples-local/twitter-follow-neo4j/pom.xml b/streams-examples/streams-examples-local/twitter-follow-neo4j/pom.xml index 2e47cf67d..366343d40 100644 --- a/streams-examples/streams-examples-local/twitter-follow-neo4j/pom.xml +++ b/streams-examples/streams-examples-local/twitter-follow-neo4j/pom.xml @@ -26,8 +26,8 @@ 4.0.0 -streams-twitter-follow-neo4j -twitter-follow-neo4j +streams-example-twitter-follow-neo4j +streams-example-twitter-follow-neo4j Collects friend or follower connections for a set of twitter users to build a graph database in neo4j. @@ -186,6 +186,16 @@ + +org.apache.streams +streams-config +${project.version} + + +org.apache.streams +streams-runtime-local +${project.version} + org.apache.streams streams-persist-neo4j @@ -253,4 +263,90 @@ + + +dockerITs + +false + +skipITs +false + + + + + +io.fabric8 +docker-maven-plugin + +500 +default +true +on + + +neo4j +neo4j + +
[jira] [Commented] (STREAMS-587) Ensure all files pass apache-rat before 0.6 release
[ https://issues.apache.org/jira/browse/STREAMS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419590#comment-16419590 ] ASF GitHub Bot commented on STREAMS-587: steveblackmon opened a new pull request #426: resolves STREAMS-587 URL: https://github.com/apache/streams/pull/426 missing apache license headers This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Ensure all files pass apache-rat before 0.6 release > --- > > Key: STREAMS-587 > URL: https://issues.apache.org/jira/browse/STREAMS-587 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-587) Ensure all files pass apache-rat before 0.6 release
[ https://issues.apache.org/jira/browse/STREAMS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16419591#comment-16419591 ] ASF GitHub Bot commented on STREAMS-587: steveblackmon closed pull request #426: resolves STREAMS-587 URL: https://github.com/apache/streams/pull/426 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberOne.java b/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberOne.java index fa073d722..6e3a53eb0 100644 --- a/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberOne.java +++ b/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberOne.java @@ -1,3 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.config.test; import org.apache.streams.config.ComponentConfiguration; diff --git a/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberTwo.java b/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberTwo.java index ba98eee5e..8f69af2df 100644 --- a/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberTwo.java +++ b/streams-config/src/test/java/org/apache/streams/config/test/ComponentConfigurationForTestingNumberTwo.java @@ -1,3 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.config.test; import org.apache.streams.config.ComponentConfiguration; diff --git a/streams-config/src/test/java/org/apache/streams/config/test/StreamsConfigurationForTesting.java b/streams-config/src/test/java/org/apache/streams/config/test/StreamsConfigurationForTesting.java index 0e9c0422c..dab82d659 100644 --- a/streams-config/src/test/java/org/apache/streams/config/test/StreamsConfigurationForTesting.java +++ b/streams-config/src/test/java/org/apache/streams/config/test/StreamsConfigurationForTesting.java @@ -1,3 +1,21 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * + * 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.streams.config.test; import org.apache.streams.config.ComponentConfiguration; diff --git a/streams-config/src/test/resources/canonicalClassName.conf b/streams-config/src/test/resources/canonicalClassName.conf index 4cbcd34f9..95b56bdba 100644 --- a/streams-config/src/test/resources/canonicalClassName.conf +++ b/streams-config/src/test/resourc
[jira] [Commented] (STREAMS-551) Add a distribution binary and container
[ https://issues.apache.org/jira/browse/STREAMS-551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16420675#comment-16420675 ] ASF GitHub Bot commented on STREAMS-551: steveblackmon opened a new pull request #428: resolves STREAMS-551 URL: https://github.com/apache/streams/pull/428 build an uber-jar by enabling the dist profile This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Add a distribution binary and container > --- > > Key: STREAMS-551 > URL: https://issues.apache.org/jira/browse/STREAMS-551 > Project: Streams > Issue Type: New Feature >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Original Estimate: 4h > Remaining Estimate: 4h > > Create a maven module that packages an uber-jar with all dependencies needed > to run any streams provider with a main method. > Add an optional maven profile that puts that uber-jar in a docker container > with an appropriate cmd/entrypoint. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-551) Add a distribution binary and container
[ https://issues.apache.org/jira/browse/STREAMS-551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16420897#comment-16420897 ] ASF GitHub Bot commented on STREAMS-551: steveblackmon closed pull request #428: resolves STREAMS-551 URL: https://github.com/apache/streams/pull/428 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index df2ede6c3a..b3b0be0aba 100644 --- a/pom.xml +++ b/pom.xml @@ -410,6 +410,7 @@ streams-config streams-contrib streams-components +streams-dist streams-examples streams-monitoring streams-pojo diff --git a/streams-dist/pom.xml b/streams-dist/pom.xml new file mode 100644 index 00..50c886947d --- /dev/null +++ b/streams-dist/pom.xml @@ -0,0 +1,293 @@ + + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd";> + +4.0.0 + + +org.apache.streams +apache-streams +0.5.2-SNAPSHOT +.. + + +streams-dist +streams-dist +jar + + + + +org.apache.streams +streams-config +${project.version} + + + +org.apache.streams +streams-converters +${project.version} + + + +org.apache.streams +streams-core +${project.version} + + + +org.apache.streams +streams-filters +${project.version} + + + +org.apache.streams +streams-http +${project.version} + + + +org.apache.streams +streams-monitoring +${project.version} + + + +org.apache.streams +streams-persist-cassandra +${project.version} + + + +org.apache.streams +streams-persist-console +${project.version} + + + +org.apache.streams +streams-persist-elasticsearch +${project.version} + + + +org.apache.streams +streams-persist-filebuffer +${project.version} + + + +org.apache.streams +streams-persist-graph +${project.version} + + + +org.apache.streams +streams-persist-hbase +${project.version} + + + +org.apache.streams +streams-persist-hdfs +${project.version} + + + +org.apache.streams +streams-persist-kafka +${project.version} + + + +org.apache.streams +streams-persist-kinesis +${project.version} + + + +org.apache.streams +streams-persist-mongo +${project.version} + + + +org.apache.streams +streams-persist-neo4j +${project.version} + + + +org.apache.streams +streams-persist-riak +${project.version} + + + +org.apache.streams +streams-persist-s3 +${project.version} + + + +org.apache.streams +streams-pojo +${project.version} + + + +org.apache.streams +streams-pojo-extensions +${project.version} + + + +org.apache.streams +streams-processor-jackson +${project.version} + + + +org.apache.streams +streams-processor-json +${project.version} + + + +org.apache.streams +streams-processor-peoplepattern +${project.version} + + + +org.apache.streams +streams-processor-regex +${project.version} + + + +org.apache.streams +streams-processor-urls +${project.version} + + + +org.apache.streams +streams-provider-facebook +${project.version} + + + +org.apache.streams +streams-provider-google-gmail +${project.version} + + + +org.apache.streams +streams-provider-google-gplus +${project.version} + + +
[jira] [Commented] (STREAMS-487) Test data originating within modules, used in other modules, should be a stand-alone artifact.
[ https://issues.apache.org/jira/browse/STREAMS-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423225#comment-16423225 ] ASF GitHub Bot commented on STREAMS-487: steveblackmon opened a new pull request #430: STREAMS-487: zip resource artifacts URL: https://github.com/apache/streams/pull/430 resolves STREAMS-487 Test data originating within modules, used in other modules, should be a stand-alone artifact. Since schemas are used as test data also, added a zip artifact containing schemas also. Required refactoring on all tests that were previously accessing files in the test classpath (target/test-classes) Now when test data is sourced through dependency on an archive zip, that data is unpacked in target/dependency and accessed as a file rather than a resource. This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Test data originating within modules, used in other modules, should be a > stand-alone artifact. > -- > > Key: STREAMS-487 > URL: https://issues.apache.org/jira/browse/STREAMS-487 > Project: Streams > Issue Type: Task > Components: Tests >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > Currently test data get compressed into the test-jar. It would make more > sense to create stand-alone zip artifacts, rather than confusing the data > with the test classes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-487) Test data originating within modules, used in other modules, should be a stand-alone artifact.
[ https://issues.apache.org/jira/browse/STREAMS-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423391#comment-16423391 ] ASF GitHub Bot commented on STREAMS-487: steveblackmon commented on issue #430: STREAMS-487: zip resource artifacts URL: https://github.com/apache/streams/pull/430#issuecomment-378105530 for some reason jenkins is running the unit tests but not moving resources into target/test-classes This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Test data originating within modules, used in other modules, should be a > stand-alone artifact. > -- > > Key: STREAMS-487 > URL: https://issues.apache.org/jira/browse/STREAMS-487 > Project: Streams > Issue Type: Task > Components: Tests >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > Currently test data get compressed into the test-jar. It would make more > sense to create stand-alone zip artifacts, rather than confusing the data > with the test classes. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-487) Test data originating within modules, used in other modules, should be a stand-alone artifact.
[ https://issues.apache.org/jira/browse/STREAMS-487?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423396#comment-16423396 ] ASF GitHub Bot commented on STREAMS-487: steveblackmon closed pull request #430: STREAMS-487: zip resource artifacts URL: https://github.com/apache/streams/pull/430 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 372a496c2c..4d3ad57a37 100644 --- a/pom.xml +++ b/pom.xml @@ -864,6 +864,34 @@ org.apache.maven.plugins maven-dependency-plugin ${dependency.plugin.version} + + +unpack-schemas +generate-sources + +unpack-dependencies + + +true + org.apache.streams +compile +zip + + + +unpack-test-resources +process-test-resources + +unpack-dependencies + + +true + org.apache.streams +test +zip + + + org.apache.maven.plugins @@ -880,60 +908,6 @@ docker-maven-plugin ${docker.plugin.version} - -org.apache.maven.plugins -maven-checkstyle-plugin -${checkstyle.plugin.version} - - -com.puppycrawl.tools -checkstyle -7.2 - - - - -validate -validate - - src/main/resources/streams-java-checkstyle.xml -UTF-8 -true - true - ${project.basedir}/src/test/java -false - - -check - - - - - -org.scalastyle -scalastyle-maven-plugin -${scalastyle.plugin.version} - - -validate -validate - -false -false - true -false - ${project.basedir}/src/main/scala - ${project.basedir}/src/test/scala - - ${project.build.directory}/scalastyle-output.xml -UTF-8 - - -check - - - - net.alchim31.maven scala-maven-plugin diff --git a/streams-contrib/streams-persist-cassandra/pom.xml b/streams-contrib/streams-persist-cassandra/pom.xml index 5df7596113..56a0c16d67 100644 --- a/streams-contrib/streams-persist-cassandra/pom.xml +++ b/streams-contrib/streams-persist-cassandra/pom.xml @@ -127,20 +127,6 @@ org.apache.maven.plugins maven-dependency-plugin - - -resource-dependencies -process-test-resources - -unpack-dependencies - - - streams-schema-activitystreams -**/*.json - ${p
[jira] [Commented] (STREAMS-591) Exclude identified Cat-X dependencies from reactor
[ https://issues.apache.org/jira/browse/STREAMS-591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423405#comment-16423405 ] ASF GitHub Bot commented on STREAMS-591: steveblackmon opened a new pull request #431: resolves STREAMS-591 URL: https://github.com/apache/streams/pull/431 excludes all identified problematic third-party dependencies disables one test which began failing with org.jboss.logging:jboss-logging excluded This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Exclude identified Cat-X dependencies from reactor > -- > > Key: STREAMS-591 > URL: https://issues.apache.org/jira/browse/STREAMS-591 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > The following dependencies have been identified as problematic and flagged > for exclusion. > (BEA Systems, Inc. ("BEA") Binary Code License Agreement) jsr173_api > (javax.xml.bind:jsr173_api:1.0 - no url defined) > (GNU Lesser General Public License, version 2.1) JBoss Logging 3 > (org.jboss.logging:jboss-logging:3.1.0.CR2 - http://www.jboss.org) > (GNU LESSER GENERAL PUBLIC LICENSE) BeanShell (org.beanshell:bsh:2.0b4 - > no url defined) > (GNU Lesser Public License) FindBugs-Annotations > (com.google.code.findbugs:annotations:1.3.9 - > http://findbugs.sourceforge.net/) > (GNU Lesser Public License) FindBugs-Annotations > (com.google.code.findbugs:annotations:3.0.0 - > http://findbugs.sourceforge.net/) -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-591) Exclude identified Cat-X dependencies from reactor
[ https://issues.apache.org/jira/browse/STREAMS-591?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423447#comment-16423447 ] ASF GitHub Bot commented on STREAMS-591: steveblackmon closed pull request #431: resolves STREAMS-591 URL: https://github.com/apache/streams/pull/431 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 4d3ad57a37..d2af9c095c 100644 --- a/pom.xml +++ b/pom.xml @@ -1034,6 +1034,12 @@ testng ${testng.version} test + + +org.beanshell +bsh + + org.mockito @@ -1378,7 +1384,7 @@ org.apache.apache.resources apache-source-release-assembly-descriptor -1.0.5 +1.0.6 diff --git a/src/license/THIRD-PARTY.properties b/src/license/THIRD-PARTY.properties index beb0b5ecdb..b45ac31289 100644 --- a/src/license/THIRD-PARTY.properties +++ b/src/license/THIRD-PARTY.properties @@ -7,7 +7,6 @@ commons-codec--commons-codec--1.2=Apache License, Version 2.0 dom4j--dom4j--1.1=Copyright 2001-2016 (C) MetaStuff, Ltd. and DOM4J contributors. All Rights Reserved. javax.servlet--servlet-api--2.5=Common Development and Distribution License javax.servlet.jsp--jsp-api--2.1=Copyright (c) 1997-2010 Oracle and/or its affiliates. All rights reserved. -javax.xml.bind--jsr173_api--1.0=BEA Systems, Inc. ("BEA") Binary Code License Agreement jdom--jdom--1.0=Copyright (C) 2000-2012 Jason Hunter & Brett McLaughlin. All rights reserved. org.antlr--antlr-runtime--3.3=BSD Licence org.apache.ant--ant--1.7.0=Apache License, Version 2.0 diff --git a/streams-plugins/streams-plugin-pojo/pom.xml b/streams-plugins/streams-plugin-pojo/pom.xml index f03a794e57..3d3fda7316 100644 --- a/streams-plugins/streams-plugin-pojo/pom.xml +++ b/streams-plugins/streams-plugin-pojo/pom.xml @@ -57,6 +57,10 @@ commons-logging commons-logging + +com.google.code.findbugs +annotations + diff --git a/streams-pojo/pom.xml b/streams-pojo/pom.xml index b474f550e7..0f1909c38c 100644 --- a/streams-pojo/pom.xml +++ b/streams-pojo/pom.xml @@ -77,12 +77,24 @@ com.fasterxml.jackson.dataformat jackson-dataformat-xml + + +stax2-api +org.codehaus.woodstox + + com.fasterxml jackson-xml-databind 0.6.2 + + +stax2-api +org.codehaus.woodstox + + @@ -98,6 +110,12 @@ nz.net.ultraq.jaxb jaxb-utilities + + +jsr173_api +javax.xml.bind + + diff --git a/streams-runtimes/streams-runtime-dropwizard/pom.xml b/streams-runtimes/streams-runtime-dropwizard/pom.xml index 8fec515e3a..13f9e4b2d3 100644 --- a/streams-runtimes/streams-runtime-dropwizard/pom.xml +++ b/streams-runtimes/streams-runtime-dropwizard/pom.xml @@ -64,6 +64,12 @@ io.dropwizard dropwizard-validation ${dropwizard.version} + + +org.jboss.logging +jboss-logging + + @@ -99,6 +105,12 @@ com.hubspot.dropwizard dropwizard-guice 0.7.1 + + +com.google.code.findbugs +annotations + + com.google.inject @@ -252,11 +264,6 @@ org.apache.maven.plugins maven-failsafe-plugin - - - - - \ No newline at end of file diff --git a/streams-runtimes/streams-runtime-dropwizard/src/test/java/org/apache/streams/dropwizard/test/GenericWebhookResourceTest.java b/streams-runtimes/streams-runtime-dropwizard/src/test/java/org/apache/streams/dropwizard/test/GenericWebhookResourceTest.java index 68c2ec7482..fee77d4e21
[jira] [Commented] (STREAMS-592) Delete streams-runtime-dropwizard
[ https://issues.apache.org/jira/browse/STREAMS-592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16424115#comment-16424115 ] ASF GitHub Bot commented on STREAMS-592: steveblackmon opened a new pull request #432: resolves STREAMS-592 URL: https://github.com/apache/streams/pull/432 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Delete streams-runtime-dropwizard > - > > Key: STREAMS-592 > URL: https://issues.apache.org/jira/browse/STREAMS-592 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > It's bringing in unwanted dependencies and isn't required by the project > roadmap -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-592) Delete streams-runtime-dropwizard
[ https://issues.apache.org/jira/browse/STREAMS-592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16424131#comment-16424131 ] ASF GitHub Bot commented on STREAMS-592: mfranklin commented on issue #432: resolves STREAMS-592 URL: https://github.com/apache/streams/pull/432#issuecomment-378278739 š This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Delete streams-runtime-dropwizard > - > > Key: STREAMS-592 > URL: https://issues.apache.org/jira/browse/STREAMS-592 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > It's bringing in unwanted dependencies and isn't required by the project > roadmap -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-592) Delete streams-runtime-dropwizard
[ https://issues.apache.org/jira/browse/STREAMS-592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16424146#comment-16424146 ] ASF GitHub Bot commented on STREAMS-592: steveblackmon closed pull request #432: resolves STREAMS-592 URL: https://github.com/apache/streams/pull/432 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-runtimes/pom.xml b/streams-runtimes/pom.xml index 49bc801491..3d54be4c5c 100644 --- a/streams-runtimes/pom.xml +++ b/streams-runtimes/pom.xml @@ -34,7 +34,6 @@ pom -streams-runtime-dropwizard streams-runtime-local diff --git a/streams-runtimes/streams-runtime-dropwizard/README.md b/streams-runtimes/streams-runtime-dropwizard/README.md deleted file mode 100644 index 52dab1da68..00 --- a/streams-runtimes/streams-runtime-dropwizard/README.md +++ /dev/null @@ -1,8 +0,0 @@ -Apache Streams -Licensed under Apache License 2.0 - http://www.apache.org/licenses/LICENSE-2.0 - - -org.apache.streams:streams-runtime-dropwizard -= - -[README.md](src/site/markdown/index.md "README") diff --git a/streams-runtimes/streams-runtime-dropwizard/pom.xml b/streams-runtimes/streams-runtime-dropwizard/pom.xml deleted file mode 100644 index 13f9e4b2d3..00 --- a/streams-runtimes/streams-runtime-dropwizard/pom.xml +++ /dev/null @@ -1,269 +0,0 @@ - - - -http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd";> -4.0.0 - - -org.apache.streams -streams-runtimes -0.5.2-SNAPSHOT - - -streams-runtime-dropwizard - - -0.7.1 - - - - - -io.dropwizard -dropwizard-core -${dropwizard.version} - - -com.fasterxml.jackson.core -jackson-annotations - - -com.fasterxml.jackson.module -jackson-module-afterburner - - -com.fasterxml.jackson.datatype -jackson-datatype-guava - - - - - -io.dropwizard -dropwizard-configuration -${dropwizard.version} - - - -io.dropwizard -dropwizard-validation -${dropwizard.version} - - -org.jboss.logging -jboss-logging - - - - - -io.dropwizard -dropwizard-metrics -${dropwizard.version} - - - -io.dropwizard -dropwizard-testing -${dropwizard.version} -test - - - -com.fasterxml.jackson.core -jackson-annotations -${jackson.version} - - -com.fasterxml.jackson.module -jackson-module-afterburner -${jackson.version} - - -com.fasterxml.jackson.datatype -jackson-datatype-guava -${jackson.version} - - - -com.hubspot.dropwizard -dropwizard-guice -0.7.1 - - -com.google.code.findbugs -annotations - - - - -com.google.inject -guice -3.0 - - - -com.fasterxml.jackson.core -jackson-core - - - -org.apache.streams -streams-config -${project.version} - - -org.apache.streams -streams-core -${project.version} - - -org.apache.streams -streams-util -${project.version} - - -org.apache.streams -streams-pojo -${project.version} - - -org.apache.streams -streams-runtime-local -${project.version} - - - -ch.qos.logback -logback-core - - -ch.qos.logback -logback-classic - - -org.slf4j -log4j-over-slf4j - - - -org.hamcrest -hamcrest-all -1.
[jira] [Commented] (STREAMS-595) some streams-plugins ITs are failing
[ https://issues.apache.org/jira/browse/STREAMS-595?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16427414#comment-16427414 ] ASF GitHub Bot commented on STREAMS-595: steveblackmon opened a new pull request #443: resolves STREAMS-595 URL: https://github.com/apache/streams/pull/443 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > some streams-plugins ITs are failing > > > Key: STREAMS-595 > URL: https://issues.apache.org/jira/browse/STREAMS-595 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > > looks like the test pom's need to be updated to match the project version and > to unpack test data from zip artifacts rather than the previously used > test-jar artifacts -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-595) some streams-plugins ITs are failing
[ https://issues.apache.org/jira/browse/STREAMS-595?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16428883#comment-16428883 ] ASF GitHub Bot commented on STREAMS-595: steveblackmon commented on issue #443: resolves STREAMS-595 URL: https://github.com/apache/streams/pull/443#issuecomment-379364110 merged to confirm fix via jenkins This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > some streams-plugins ITs are failing > > > Key: STREAMS-595 > URL: https://issues.apache.org/jira/browse/STREAMS-595 > Project: Streams > Issue Type: Bug >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > looks like the test pom's need to be updated to match the project version and > to unpack test data from zip artifacts rather than the previously used > test-jar artifacts -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-595) some streams-plugins ITs are failing
[ https://issues.apache.org/jira/browse/STREAMS-595?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16428882#comment-16428882 ] ASF GitHub Bot commented on STREAMS-595: steveblackmon closed pull request #443: resolves STREAMS-595 URL: https://github.com/apache/streams/pull/443 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml b/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml index ac21d5a56b..18a9c20d0a 100644 --- a/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml +++ b/streams-plugins/streams-plugin-cassandra/src/test/resources/streams-plugin-cassandra/pom.xml @@ -23,7 +23,7 @@ org.apache.streams.plugins streams-plugin-cassandra-test -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT jar Test StreamsCassandraResourceGeneratorMojo @@ -88,8 +88,9 @@ org.apache.streams streams-schema-activitystreams ${project.version} -jar -test +zip +schemas +runtime @@ -98,7 +99,7 @@ org.apache.streams.plugins streams-plugin-cassandra -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT target/dependency/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml b/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml index 84615a29e7..a15933adca 100644 --- a/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml +++ b/streams-plugins/streams-plugin-elasticsearch/src/test/resources/streams-plugin-elasticsearch/pom.xml @@ -23,7 +23,7 @@ org.apache.streams.plugins streams-plugin-elasticsearch-test -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT jar Test StreamsElasticsearchResourceGeneratorMojo @@ -88,8 +88,9 @@ org.apache.streams streams-schema-activitystreams ${project.version} -test -jar +zip +schemas +runtime @@ -98,7 +99,7 @@ org.apache.streams.plugins streams-plugin-elasticsearch -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT target/dependency/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml b/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml index ea39c15919..2b3fafeda1 100644 --- a/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml +++ b/streams-plugins/streams-plugin-hbase/src/test/resources/streams-plugin-hbase/pom.xml @@ -24,7 +24,7 @@ org.apache.streams.plugins streams-plugin-hbase-test -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT jar Test StreamsHbaseResourceGeneratorMojo @@ -89,8 +89,9 @@ org.apache.streams streams-schema-activitystreams ${project.version} -test -jar +zip +schemas +runtime @@ -99,7 +100,7 @@ org.apache.streams.plugins streams-plugin-hbase -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT target/dependency/activitystreams-schemas/activity.json diff --git a/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml b/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml index 47c9f9bc9f..999f5bcdf1 100644 --- a/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml +++ b/streams-plugins/streams-plugin-hive/src/test/resources/streams-plugin-hive/pom.xml @@ -24,7 +24,7 @@ org.apache.streams.plugins streams-plugin-hive-test -0.5.1-SNAPSHOT +0.5.2-SNAPSHOT jar Test StreamsPojoHiveMojo @@ -89,8 +89,9 @@ org.apache.streams streams-schema-activitystreams ${project.version} -test -jar +zip +schemas +runtime @@ -99,7 +100,7 @@
[jira] [Commented] (STREAMS-599) Update KEYS file for 0.6.0 release
[ https://issues.apache.org/jira/browse/STREAMS-599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16429069#comment-16429069 ] ASF GitHub Bot commented on STREAMS-599: jfrazee opened a new pull request #444: STREAMS-599 Updated KEYS file for 0.6.0 release URL: https://github.com/apache/streams/pull/444 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update KEYS file for 0.6.0 release > -- > > Key: STREAMS-599 > URL: https://issues.apache.org/jira/browse/STREAMS-599 > Project: Streams > Issue Type: Task >Reporter: Joey Frazee >Assignee: Joey Frazee >Priority: Minor > Fix For: 0.6.0 > > > The KEYS file needs updated for the 0.6.0 release -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-599) Update KEYS file for 0.6.0 release
[ https://issues.apache.org/jira/browse/STREAMS-599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16429071#comment-16429071 ] ASF GitHub Bot commented on STREAMS-599: jfrazee closed pull request #444: STREAMS-599 Updated KEYS file for 0.6.0 release URL: https://github.com/apache/streams/pull/444 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/KEYS b/KEYS index 1a3a69b8d0..1993876034 100644 --- a/KEYS +++ b/KEYS @@ -5,7 +5,7 @@ Users: pgp < KEYS Developers: pgp -kxa and append it to this file. (pgpk -ll && pgpk -xa ) >> this file. -(gpg --list-keys +(gpg --list-sigs && gpg --armor --export ) >> this file. pub 2048R/3F94A968 2013-01-07 @@ -165,79 +165,6 @@ DkFmj1upWAayCoXTpKzsHBvJZPC+Wqf9Pl3O47apelg7KxU3S011YfXpVPvCTKBv kD2o/5GKWS5QkSUEUXXY1oDiLg== =f8kJ -END PGP PUBLIC KEY BLOCK- - -pub 4096R/C5F10B99 2016-12-29 [expires: 2017-12-31] -uid [ultimate] Joey Frazee -sig 3C5F10B99 2017-01-02 Joey Frazee -uid [ultimate] Joey Frazee -sig 3C5F10B99 2017-01-02 Joey Frazee -sub 4096R/C34484EF 2016-12-29 [expires: 2017-12-31] -sig C5F10B99 2017-01-02 Joey Frazee - --BEGIN PGP PUBLIC KEY BLOCK- - -mQINBFhlO8ABEACsfCjuAT1EVcAS8Wvhxq9vadj57nQhWvZmHttS8Vuvucqot9gt -2VMbvihwWj0pEUfeGnvhdGyHvm1qHj+A4eR4PHZ6E53HEfV7IlUjoH+2gKFudYgz -/QUMgtFjhBxVxdP+xY8LSq4KtfvCCrdHFb0k+3K+FgwFeTGUrTJM8+5Lgjgrq2Zy -iICqNCdyK+rYOadCJP8CxxWjudrbkI+yXmIvYcRei8C5SyvBmfV4ozVcevhsMxOC -6uMPG7dFj6AqD7lTy7BLmNhQ36oLs3FMmevIcjzwrOe6tF9wvGR4U+5Tvxfmb5cA -8EA4wFjqPo6BxKA4ojUEhvuWlb4t6sqKDRwTm90GnBTD1jvG+uXXz3rszTb0MteY -4oCsSw0h510jJgEfY1qFmNrHvA/ULzaQAVA6x2nm1Izq25atzBdeQbI9KzlBrZ8k -PZdwQZSXFymgBGEQLoxnJQbq9uVh0cAbWUucuk7Ty0DydJWgX0M4GNOFy897euvk -K/6nEJQDXIsAuvGwaNlmlKp6t0vdpPmCi96jvmAklZr54O4K5FDK5BfMsBbJ9Kn4 -N7LIrxJ1Eblqb7cUDJKeuGoWxWgPrWaTvDe0S4BV3rqzbQ2jwgehYqJjl2jZ1zQk -ffVJOtLcblsgXjwdtoGr+xNg+Fi5ul2ZbURdxE2uOay752cGtSoNY7VaxwARAQAB -tCBKb2V5IEZyYXplZSA8amZyYXplZUBhcGFjaGUub3JnPokCQgQTAQgALAIbAwcL -CQgHAwIBBhUIAgkKCwQWAgMBAh4BAheAAhkBBQJYaskcBQkB5B3cAAoJEHjlJtLF -8QuZBHUP/A/aD4nD1tYTJj91DmFEBNm92oXvkpin1jv2uXs6EBLkHyJnpD3GGcBS -t2qsI3W7JPmxLFdQ84NwLJTUEbOxS8CgoaIYH5krv490ina9LH1B1aItY/qTh+6t -lp98+GhNr1jRaYpZwkQYQa43CTsroiJo6NHGOQ0G2Ryi+SSDHnddfT0wsO4qttcA -s9CLsmDcwJVog2vjHXBk9EdVjB5kmNNYZdd+whGnE3bl0oex+FJ7RSJu5dw/3J21 -yjU8jCI7PlZW30WTrWnqJOGpSkkONhdZQxp5FEgxefu3/E5E3lQomkaLpdHToXCc -2KwRzZ6rD/cG+yDtVhWM2mIjCGovcCO3FMHPUbW0aEhzszypSM5V/zuZMLNggUrt -er6iY63HO4KvSr6u8bLSra873I8T6VTfKTo3wjJLzTIH3v2NKRuKhKT3vuZA3Hh2 -kC267OFxrtiXAxuuvoWExP7dTWNlk+D3YrDfhZOKRpBR73R2gvn3ov59uCDEeEaR -karZVmqXVoxN/A02zn5X+xBmHMqYajUG9gq4tdWVHvg+4SfzgV0byDAy+qvTRKRa -Dc4ReeOUuOlrQZz4rjERSiRbqKFc0cN/yuoT4+big5uxEIHsbd5jZCMTILaGwb48 -gPt7/ok6WszHFJtLACmtFI7Runyz12mTHREA7E0MZGx0ywxhw0N2tCRKb2V5IEZy -YXplZSA8am9leS5mcmF6ZWVAaWNsb3VkLmNvbT6JAj8EEwEIACkCGwMHCwkIBwMC -AQYVCAIJCgsEFgIDAQIeAQIXgAUCWGrJLQUJAeQd3AAKCRB45SbSxfELmcpLEACd -4CawnyZVKLo6pFKBRVEgPkYarhuKIqrHc8o0TKS/nZC1FvFQ/4aXz/n0Sz9pPikh -9b8our3QzMB2WUPdKjCw8QT0rE/qOKy5NrYc9FXzeRgncrNsrGy/df/Tnv0h9kOg -JxkhQU6MecDVwUJ+mgA5KL7RUo41msGQqtgfPid3wCW9rT+R8+oSvtaDpNbZ++P6 -+UxMWRc286LPbTapoagRw3zh5VTwHJ5qERMrSOZe9UE43KBtPyzOZt6ZEhA9By3B -eek3haEUcYJIGrqmjlC8gNLy5LqwEmUApGJLv+5BnelaxCljkfLVR92ZRvZQX3/C -1UB24iTQn4gDkGBZCG8JxWg5ZQlqjomTsIyzGb3um931XmhIQbVfdMCk51Zx61gW -EbW10RyqnCQsCVaOeSfv5TCMkqgbGDHaQhkDA5wbOa721vOo+Jh1ZLBRMtQ0rnna -rB3PvZd/AO1xYPhSJbsQNjNxaNsyVENRn4Phpvwe7fhMKVB2bFYamsP4b5ymhRAy -480L8WnBZBRfVDr+qezk1K2Uk/cOwj9mX+3dRWTygtsGgPiRPDPbhxmxeACKZ5Vg -FpFfmyLhdVv3pehpr8Hs5741FybvdDPpS+w2/HuxBGhz4wnsag3ZIlHV1n1DKHpw -yyjdw/vVVXgolOi23u8p/OQ/86Uo+B5UTU2hanMUQ7kCDQRYZTvAARAAwglBhvUF -Nm+8c8g4mc8QX3NL4T8X07LZtDBAbUGaImYq1cRa8Nh0Z/PLvTVxO2DqEniNQcBZ -8aXhiWm0wtzyL7bRA5HADFRoB9MQacJY7INnmqq67Q7pYuLKu3/coRRNAbGERnz5 -DcY/i6vzJJ86gQmnThTq9Im4R3FM8xb5PFpaWR7l2yC2P2WNQANm0tV8EeOEdGxu -JRZyCvFje+6AxdHlF6Vs1aKNP6r4GXJ2AAlvEf0rcJ632xdh2IPQYVI8nN0Njc8w -KHZVnl2GhLq91ZOSN88U5fr8S1Qp6FzM8Ayh3hvLgKav2V+oRE4OPqFiYm+WnKqy -eBpAd0MmhVlsAuB9MatdUBMy6qbBqgElyvOCf7fDMm15zBxz9s/4VchRJGBVco7E -au98jzAv0OerdU9MYBvuHQ5LQKUnaJ9WH0c+tpFNQIQ9Cdw8VwZ3D2dPg2et1b61 -HYIAHRi/jSltJNoi9oOW9itahojcx4mPbvVkcaV0fREkItF1DVHBY/ewHBSDzfm1 -g41zzaMDQXhXYSnYpny01R+glZGmOnxkeUsh0QPtfgzZZYLDbzd8yWYMtsaIk6AR -CBmu7ql/2FlI6tqOKh/PyAh2lrrimLV7CjnoRmfUdb8t1h8Jl6uYNQ2Kv4iGokwB -q8MeNMg4gbpcXM/Lr+UQYgXEw0aATYHVsaMAEQEAAYkCJQQYAQgADwIbDAUCWGrJ -RwUJAeQeBwAKCRB45SbSxfELmSb8EACpH9F/LU9cBDlfY9f2+4VUleJGzW9IO/6G -X5skiCgjuwS8vFhmv+5nVUvvOz2K8rRzAo79OCsPaQ4bvIS92w+1hVCTn1eXpCRh -mQc8bjd6GYBtBsXnEWr1X0/zwLPCvPEQeXj3nppTPED83j+d1CjVLUa89F2vRNZV -ZHpkl7z3NzI+0ZcdTVIX5PgmCXH2NfVQ14I0ksewSIpCPOBlfRgbSoS3YCdcsbMf -J3u
[jira] [Commented] (STREAMS-600) Remove streams-runtime-dropwizard dependency from streams-dist
[ https://issues.apache.org/jira/browse/STREAMS-600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16429997#comment-16429997 ] ASF GitHub Bot commented on STREAMS-600: jfrazee opened a new pull request #445: STREAMS-600 Remove streams-runtime-dropwizard dep from streams-dist URL: https://github.com/apache/streams/pull/445 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove streams-runtime-dropwizard dependency from streams-dist > --- > > Key: STREAMS-600 > URL: https://issues.apache.org/jira/browse/STREAMS-600 > Project: Streams > Issue Type: Bug >Reporter: Joey Frazee >Assignee: Joey Frazee >Priority: Major > Fix For: 0.6.0 > > > The pom.xml in streams-dist still has a dependency onĀ > streams-runtime-dropwizard hanging around that needs to be removed. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-600) Remove streams-runtime-dropwizard dependency from streams-dist
[ https://issues.apache.org/jira/browse/STREAMS-600?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16434459#comment-16434459 ] ASF GitHub Bot commented on STREAMS-600: steveblackmon closed pull request #445: STREAMS-600 Remove streams-runtime-dropwizard dep from streams-dist URL: https://github.com/apache/streams/pull/445 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/streams-dist/pom.xml b/streams-dist/pom.xml index 6db5a1adb8..20c761ec45 100644 --- a/streams-dist/pom.xml +++ b/streams-dist/pom.xml @@ -243,12 +243,6 @@ under the License. ${project.version} - -org.apache.streams -streams-runtime-dropwizard -${project.version} - - org.apache.streams streams-runtime-local This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Remove streams-runtime-dropwizard dependency from streams-dist > --- > > Key: STREAMS-600 > URL: https://issues.apache.org/jira/browse/STREAMS-600 > Project: Streams > Issue Type: Bug >Reporter: Joey Frazee >Assignee: Joey Frazee >Priority: Major > Fix For: 0.6.0 > > > The pom.xml in streams-dist still has a dependency onĀ > streams-runtime-dropwizard hanging around that needs to be removed. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-584) Upgrade to Juneau 7.1.0
[ https://issues.apache.org/jira/browse/STREAMS-584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16440140#comment-16440140 ] ASF GitHub Bot commented on STREAMS-584: steveblackmon opened a new pull request #448: resolves STREAMS-584 URL: https://github.com/apache/streams/pull/448 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Upgrade to Juneau 7.1.0 > --- > > Key: STREAMS-584 > URL: https://issues.apache.org/jira/browse/STREAMS-584 > Project: Streams > Issue Type: Task >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > Original Estimate: 1h > Remaining Estimate: 1h > > Upgrade to Juneau 7.1.0 once the release candidate is available. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-603) Upgrade to latest Jackson 2.x
[ https://issues.apache.org/jira/browse/STREAMS-603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16440231#comment-16440231 ] ASF GitHub Bot commented on STREAMS-603: steveblackmon opened a new pull request #449: resolves STREAMS-603 URL: https://github.com/apache/streams/pull/449 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Upgrade to latest Jackson 2.x > - > > Key: STREAMS-603 > URL: https://issues.apache.org/jira/browse/STREAMS-603 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > Upgrade to latest Jackson 2.x -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-584) Upgrade to Juneau 7.1.0
[ https://issues.apache.org/jira/browse/STREAMS-584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16441012#comment-16441012 ] ASF GitHub Bot commented on STREAMS-584: steveblackmon closed pull request #448: resolves STREAMS-584 URL: https://github.com/apache/streams/pull/448 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index f46a164c2c..3b3cc079d4 100644 --- a/pom.xml +++ b/pom.xml @@ -364,7 +364,7 @@ 1.8.1 0.5.0 0.1.10 -7.0.1 +7.1.0 0.4.10 0.13.1 0.11.1 diff --git a/streams-contrib/streams-provider-instagram/src/main/java/org/apache/streams/instagram/api/Instagram.java b/streams-contrib/streams-provider-instagram/src/main/java/org/apache/streams/instagram/api/Instagram.java index e153b5fbbe..9f5cd63c2c 100644 --- a/streams-contrib/streams-provider-instagram/src/main/java/org/apache/streams/instagram/api/Instagram.java +++ b/streams-contrib/streams-provider-instagram/src/main/java/org/apache/streams/instagram/api/Instagram.java @@ -83,7 +83,7 @@ private Instagram(InstagramConfiguration configuration) throws InstantiationExce .setMaxConnPerRoute(20) .setMaxConnTotal(100) .build(); -this.restClient = new RestClientBuilder() +this.restClient = RestClient.create() .rootUrl(rootUrl) .accept(APPLICATION_JSON.getMimeType()) .httpClient(httpclient, true) @@ -99,7 +99,7 @@ private Instagram(InstagramConfiguration configuration) throws InstantiationExce .build()) .retryable( configuration.getRetryMax().intValue(), -configuration.getRetrySleepMs(), +configuration.getRetrySleepMs().intValue(), new InstagramRetryHandler()) .build(); this.mapper = StreamsJacksonMapper.getInstance(); diff --git a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Twitter.java b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Twitter.java index a34d7f3739..a91b65377a 100644 --- a/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Twitter.java +++ b/streams-contrib/streams-provider-twitter/src/main/java/org/apache/streams/twitter/api/Twitter.java @@ -121,7 +121,7 @@ private Twitter(TwitterConfiguration configuration) throws InstantiationExceptio .addInterceptorLast((HttpRequestInterceptor) (httpRequest, httpContext) -> LOGGER.debug(httpRequest.getRequestLine().getUri())) .addInterceptorLast((HttpResponseInterceptor) (httpResponse, httpContext) -> LOGGER.debug(httpResponse.getStatusLine().toString())) .build(); -this.restClientBuilder = new RestClientBuilder() +this.restClientBuilder = RestClient.create() .httpClient(httpclient, true) .parser( JsonParser.DEFAULT.builder() @@ -137,7 +137,7 @@ private Twitter(TwitterConfiguration configuration) throws InstantiationExceptio .rootUrl(rootUrl) .retryable( configuration.getRetryMax().intValue(), -configuration.getRetrySleepMs(), +configuration.getRetrySleepMs().intValue(), new TwitterRetryHandler()); if( configuration.getDebug() ) { restClientBuilder = restClientBuilder.debug(); diff --git a/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/utils/TwitterActivityConvertersTest.java b/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/utils/TwitterActivityConvertersTest.java index 7bfb8ceed5..99cdfcdac7 100644 --- a/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/utils/TwitterActivityConvertersTest.java +++ b/streams-contrib/streams-provider-twitter/src/test/java/org/apache/streams/twitter/test/utils/TwitterActivityConvertersTest.java @@ -49,6 +49,8 @@ private ActivityConverterUtil activityConverterUtil = ActivityConverterUtil.getInstance(); + private String deleteJson = "{\"delete\":{\"status\":{\"id\":12345,\"user_id\":56789}}}"; + private String followJson = "{\"follower\":{\"id\":12345},\"followee\":{\"id\":56789}}}"; private String tweetJson = "{\"created_at\":\"Wed Dec 11 22:27:34 + 2013\",\"id\":12345,\"id_str\":\"12345\",\"text\":\"text\",\"source\":\"source\",\"truncated\":false,\"in_reply_to_status_id\":null,\"in_reply_to_status_id_str\":null,\"in_reply_to_user_id\":null,\"in_reply_to_user_id_str\":null,\"in_reply_to_screen_name\":null,\"user\":{\"id\":91407775,\"id_str\":\"12345\",\"name\":\"name\",\"screen_name\":\"screen_name\",\"location\":\"\",\"url\":
[jira] [Commented] (STREAMS-603) Upgrade to latest Jackson 2.x
[ https://issues.apache.org/jira/browse/STREAMS-603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16441386#comment-16441386 ] ASF GitHub Bot commented on STREAMS-603: steveblackmon closed pull request #449: resolves STREAMS-603 URL: https://github.com/apache/streams/pull/449 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index f46a164c2c..6a802a658b 100644 --- a/pom.xml +++ b/pom.xml @@ -357,9 +357,8 @@ 2.5 -2.7.9 -0.6.2 -1.0.0 +2.9.5 +1.1.0 2.9.9 1.8.1 0.5.0 @@ -1079,11 +1078,12 @@ com.fasterxml.jackson.dataformat jackson-dataformat-xml ${jackson.version} - - -com.fasterxml -jackson-xml-databind -${jackson-xml-databind.version} + + +stax2-api +org.codehaus.woodstox + + com.fasterxml.jackson.dataformat diff --git a/streams-pojo/pom.xml b/streams-pojo/pom.xml index 0f1909c38c..ddabe697e6 100644 --- a/streams-pojo/pom.xml +++ b/streams-pojo/pom.xml @@ -85,18 +85,6 @@ - -com.fasterxml -jackson-xml-databind -0.6.2 - - -stax2-api -org.codehaus.woodstox - - - - com.fasterxml aalto-xml This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Upgrade to latest Jackson 2.x > - > > Key: STREAMS-603 > URL: https://issues.apache.org/jira/browse/STREAMS-603 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > Upgrade to latest Jackson 2.x -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-603) Upgrade to latest Jackson 2.x
[ https://issues.apache.org/jira/browse/STREAMS-603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16441394#comment-16441394 ] ASF GitHub Bot commented on STREAMS-603: steveblackmon opened a new pull request #450: resolves STREAMS-603 URL: https://github.com/apache/streams/pull/450 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Upgrade to latest Jackson 2.x > - > > Key: STREAMS-603 > URL: https://issues.apache.org/jira/browse/STREAMS-603 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > Upgrade to latest Jackson 2.x -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-603) Upgrade to latest Jackson 2.x
[ https://issues.apache.org/jira/browse/STREAMS-603?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16441511#comment-16441511 ] ASF GitHub Bot commented on STREAMS-603: steveblackmon closed pull request #450: resolves STREAMS-603 URL: https://github.com/apache/streams/pull/450 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 3b3cc079d4..7eb511ff4d 100644 --- a/pom.xml +++ b/pom.xml @@ -357,9 +357,8 @@ 2.5 -2.7.9 -0.6.2 -1.0.0 +2.9.5 +1.1.0 2.9.9 1.8.1 0.5.0 @@ -1079,11 +1078,12 @@ com.fasterxml.jackson.dataformat jackson-dataformat-xml ${jackson.version} - - -com.fasterxml -jackson-xml-databind -${jackson-xml-databind.version} + + +stax2-api +org.codehaus.woodstox + + com.fasterxml.jackson.dataformat diff --git a/streams-pojo/pom.xml b/streams-pojo/pom.xml index 0f1909c38c..ddabe697e6 100644 --- a/streams-pojo/pom.xml +++ b/streams-pojo/pom.xml @@ -85,18 +85,6 @@ - -com.fasterxml -jackson-xml-databind -0.6.2 - - -stax2-api -org.codehaus.woodstox - - - - com.fasterxml aalto-xml diff --git a/streams-schemas/streams-schema-activitystreams/pom.xml b/streams-schemas/streams-schema-activitystreams/pom.xml index 4af4574aaa..34ccf2200c 100644 --- a/streams-schemas/streams-schema-activitystreams/pom.xml +++ b/streams-schemas/streams-schema-activitystreams/pom.xml @@ -43,6 +43,25 @@ commons-io test + +com.fasterxml.jackson.core +jackson-core +${jackson.version} +test + + +com.fasterxml.jackson.core +jackson-databind +${jackson.version} +test + + +org.apache.streams +streams-testing +${project.version} +test +test-jar + org.apache.streams streams-testing This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Upgrade to latest Jackson 2.x > - > > Key: STREAMS-603 > URL: https://issues.apache.org/jira/browse/STREAMS-603 > Project: Streams > Issue Type: Improvement >Reporter: Steve Blackmon >Assignee: Steve Blackmon >Priority: Major > Fix For: 0.6.0 > > > Upgrade to latest Jackson 2.x -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-604) Update Apache parent POM to latest
[ https://issues.apache.org/jira/browse/STREAMS-604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16458214#comment-16458214 ] ASF GitHub Bot commented on STREAMS-604: jfrazee opened a new pull request #451: STREAMS-604 Update Apache parent POM version to 19 URL: https://github.com/apache/streams/pull/451 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update Apache parent POM to latest > -- > > Key: STREAMS-604 > URL: https://issues.apache.org/jira/browse/STREAMS-604 > Project: Streams > Issue Type: Improvement >Reporter: Joey Frazee >Priority: Minor > Fix For: 0.6.0 > > > The latest ApacheĀ parent POM Version is 19. Streams is still using 18. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (STREAMS-604) Update Apache parent POM to latest
[ https://issues.apache.org/jira/browse/STREAMS-604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16458215#comment-16458215 ] ASF GitHub Bot commented on STREAMS-604: jfrazee closed pull request #451: STREAMS-604 Update Apache parent POM version to 19 URL: https://github.com/apache/streams/pull/451 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork), the diff is supplied below (as it won't show otherwise due to GitHub magic): diff --git a/pom.xml b/pom.xml index 7eb511ff4d..ea34ccffbe 100644 --- a/pom.xml +++ b/pom.xml @@ -23,7 +23,7 @@ org.apache apache -18 +19 org.apache.streams This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > Update Apache parent POM to latest > -- > > Key: STREAMS-604 > URL: https://issues.apache.org/jira/browse/STREAMS-604 > Project: Streams > Issue Type: Improvement >Reporter: Joey Frazee >Priority: Minor > Fix For: 0.6.0 > > > The latest ApacheĀ parent POM Version is 19. Streams is still using 18. -- This message was sent by Atlassian JIRA (v7.6.3#76005)