[GitHub] sruehl closed pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-06-13 Thread GitBox
sruehl closed pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8 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

[GitHub] dlaboss commented on issue #1: Warning cleanup

2018-01-09 Thread GitBox
dlaboss commented on issue #1: Warning cleanup URL: https://github.com/apache/incubator-plc4x/pull/1#issuecomment-356292913 Hi Justin, I was expecting to merge it myself, but thanks :-) Isn't is just better to consistently work in the github fork/feature-branch/PR flow so as to

[GitHub] chrisdutz commented on issue #1: Warning cleanup

2018-01-09 Thread GitBox
chrisdutz commented on issue #1: Warning cleanup URL: https://github.com/apache/incubator-plc4x/pull/1#issuecomment-356294407 Well we should get the Travis support working first or you will be working without CI ... I've already setup the travis file thing, but still have to have Infra

[GitHub] justinmclean closed pull request #1: Warning cleanup

2018-01-08 Thread GitBox
justinmclean closed pull request #1: Warning cleanup URL: https://github.com/apache/incubator-plc4x/pull/1 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

[GitHub] justinmclean commented on issue #1: Warning cleanup

2018-01-08 Thread GitBox
justinmclean commented on issue #1: Warning cleanup URL: https://github.com/apache/incubator-plc4x/pull/1#issuecomment-356121460 IMO no need for a PR for still like this feel free to commit yourself. This is an automated

[GitHub] dlaboss opened a new pull request #1: Warning cleanup

2018-01-08 Thread GitBox
dlaboss opened a new pull request #1: Warning cleanup URL: https://github.com/apache/incubator-plc4x/pull/1 Cleanup a number of warnings reported by Eclipse. There are still 50+ warnings related to raw types, casts, type safety as well as unused field/local-variables. Please

[GitHub] asfgit closed pull request #2: [WIP]Test travis integration

2018-01-26 Thread GitBox
asfgit closed pull request #2: [WIP]Test travis integration URL: https://github.com/apache/incubator-plc4x/pull/2 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] crazzle closed pull request #3: Remove response reference in request item

2018-01-13 Thread GitBox
crazzle closed pull request #3: Remove response reference in request item URL: https://github.com/apache/incubator-plc4x/pull/3 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] niclash opened a new pull request #4: Change from AsserttJ to Hamcrest

2018-02-14 Thread GitBox
niclash opened a new pull request #4: Change from AsserttJ to Hamcrest URL: https://github.com/apache/incubator-plc4x/pull/4 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] dlaboss opened a new pull request #6: cleanup sonar edgent complaints

2018-02-26 Thread GitBox
dlaboss opened a new pull request #6: cleanup sonar edgent complaints URL: https://github.com/apache/incubator-plc4x/pull/6 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] asfgit closed pull request #6: cleanup sonar edgent complaints

2018-02-26 Thread GitBox
asfgit closed pull request #6: cleanup sonar edgent complaints URL: https://github.com/apache/incubator-plc4x/pull/6 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

[GitHub] asfgit closed pull request #7: cleanup some warnings (in Eclipse)

2018-02-26 Thread GitBox
asfgit closed pull request #7: cleanup some warnings (in Eclipse) URL: https://github.com/apache/incubator-plc4x/pull/7 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

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833373 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833246 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcReadRequest.java

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170832548 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833305 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcWriteRequest.java

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833183 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170831995 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833285 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcReadResponse.java

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833353 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833513 ## File path: plc4j/core/src/test/java/org/apache/plc4x/java/PlcDriverManagerTest.java ##

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170832428 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833118 ## File path:

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170833397 ## File path: plc4j/core/src/test/java/org/apache/plc4x/java/PlcDriverManagerTest.java ##

[GitHub] sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-26 Thread GitBox
sruehl commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170832166 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcWriteResponse.java

[GitHub] sruehl commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-409959876 one note at this point. As far as I understand netty throws this exception to indicate that it dislikes when the thread gets blocked. By using

[GitHub] JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-409958048 @chrisdutz your fixes in the master made it work for me also. As I found a solution to avoid the blocking exception in the meanwhile I

[GitHub] JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-409963436 @sruehl my understanding is that netty avoids a scenario where one uses the event loop executors threads for blocking and thus, risks a

[GitHub] chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-409995467 Well I Ave started working on an option that works exactly the same way we are doing the connection, by creating a CompletableFuture and

[GitHub] chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-03 Thread GitBox
chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-410171496 And I don't agree with Sebastians assumption. Even if it might be the same thread when running in unit-tests (Here we have to write the

[GitHub] chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-03 Thread GitBox
chrisdutz commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-410170620 Ok ... so I just checked in the version I was talking about. It is basically structurally equal to the way we are doing things in the

[GitHub] JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close.

2018-08-03 Thread GitBox
JulianFeinauer commented on issue #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#issuecomment-410172284 @chrisdutz I guess due to my partial scala background I totally ignore Javas Completable futures as superior and thus didnt think about

[GitHub] JulianFeinauer opened a new pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
JulianFeinauer opened a new pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10 Fix for the NPE in S7PlcConnection's close method. After fixing the NPE there was a netty blockingException, thus I had to refactor the program logic

[GitHub] chrisdutz commented on a change in pull request #9: Added some javadoc to S7 communication path and several todos that ma…

2018-08-02 Thread GitBox
chrisdutz commented on a change in pull request #9: Added some javadoc to S7 communication path and several todos that ma… URL: https://github.com/apache/incubator-plc4x/pull/9#discussion_r207142023 ## File path:

[GitHub] JulianFeinauer commented on a change in pull request #9: Added some javadoc to S7 communication path and several todos that ma…

2018-08-02 Thread GitBox
JulianFeinauer commented on a change in pull request #9: Added some javadoc to S7 communication path and several todos that ma… URL: https://github.com/apache/incubator-plc4x/pull/9#discussion_r207149237 ## File path:

[GitHub] asfgit closed pull request #9: Added some javadoc to S7 communication path and several todos that ma…

2018-08-02 Thread GitBox
asfgit closed pull request #9: Added some javadoc to S7 communication path and several todos that ma… URL: https://github.com/apache/incubator-plc4x/pull/9 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

[GitHub] sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207145613 ## File path: plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java ##

[GitHub] sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207149145 ## File path: plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java ##

[GitHub] sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207148665 ## File path: plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java ##

[GitHub] JulianFeinauer commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
JulianFeinauer commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207178132 ## File path: plc4j/protocols/s7/src/main/java/org/apache/plc4x/java/s7/connection/S7PlcConnection.java

[GitHub] chrisdutz commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
chrisdutz commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207224977 ## File path: plc4j/protocols/s7/src/test/java/org/apache/plc4x/java/s7/connection/S7PlcConnectionIT.java

[GitHub] JulianFeinauer opened a new pull request #9: Added some javadoc to S7 communication path and several todos that ma…

2018-08-01 Thread GitBox
JulianFeinauer opened a new pull request #9: Added some javadoc to S7 communication path and several todos that ma… URL: https://github.com/apache/incubator-plc4x/pull/9 Hey all, i went through the S7 Communication path to understand it for myself and by doing that I tried adding

[GitHub] sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207198528 ## File path: plc4j/protocols/s7/src/test/java/org/apache/plc4x/java/s7/connection/S7PlcConnectionIT.java ##

[GitHub] JulianFeinauer commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
JulianFeinauer commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207200954 ## File path: plc4j/protocols/s7/src/test/java/org/apache/plc4x/java/s7/connection/S7PlcConnectionIT.java

[GitHub] sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close.

2018-08-02 Thread GitBox
sruehl commented on a change in pull request #10: Fixed NPE in S7PlcConnection#close. URL: https://github.com/apache/incubator-plc4x/pull/10#discussion_r207197726 ## File path: plc4j/protocols/s7/src/test/java/org/apache/plc4x/java/s7/connection/S7PlcConnectionIT.java ##

[GitHub] asfgit closed pull request #11: Implement basic example of connecting an S7 device to Azure IoT Hub

2018-08-13 Thread GitBox
asfgit closed pull request #11: Implement basic example of connecting an S7 device to Azure IoT Hub URL: https://github.com/apache/incubator-plc4x/pull/11 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

[GitHub] chrisdutz commented on issue #12: fix a few typos

2018-08-13 Thread GitBox
chrisdutz commented on issue #12: fix a few typos URL: https://github.com/apache/incubator-plc4x/pull/12#issuecomment-412528417 Well I guess I can simply merge this from the GitHub ui ... thanks for this :-) This is an

[GitHub] chrisdutz commented on issue #12: fix a few typos

2018-08-13 Thread GitBox
chrisdutz commented on issue #12: fix a few typos URL: https://github.com/apache/incubator-plc4x/pull/12#issuecomment-412530954 And it does seem as if this revealed syncing problems from GitHub back to GitBox ... will investigate

[GitHub] bodewig commented on issue #12: fix a few typos

2018-08-13 Thread GitBox
bodewig commented on issue #12: fix a few typos URL: https://github.com/apache/incubator-plc4x/pull/12#issuecomment-412537899 As somebody toying with the idea of using gitbox for some other projects I'd be interested in hearing what went wrong

[GitHub] chrisdutz closed pull request #12: fix a few typos

2018-08-13 Thread GitBox
chrisdutz closed pull request #12: fix a few typos URL: https://github.com/apache/incubator-plc4x/pull/12 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

[GitHub] DerDackel commented on issue #11: Implement basic example of connecting an S7 device to Azure IoT Hub

2018-08-10 Thread GitBox
DerDackel commented on issue #11: Implement basic example of connecting an S7 device to Azure IoT Hub URL: https://github.com/apache/incubator-plc4x/pull/11#issuecomment-412079165 Whatever do you mean? ;) Realized my mistake immediately after submitting this PR and re-pushed the

[GitHub] pisquaredover6 opened a new pull request #13: Basic example to connect S7 device to Google Cloud IoT Core

2018-08-28 Thread GitBox
pisquaredover6 opened a new pull request #13: Basic example to connect S7 device to Google Cloud IoT Core URL: https://github.com/apache/incubator-plc4x/pull/13 Implement basic example of connecting a S7 device to Google Cloud IoT Core

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-09-05 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r214037249 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcWriteRequest.java ## @@ -24,6 +24,8 @@ Licensed

[GitHub] chrisdutz commented on a change in pull request #14: WIP: Api Refactoring

2018-09-05 Thread GitBox
chrisdutz commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r215170934 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcFieldRequest.java ## @@ -20,11 +20,13 @@

[GitHub] asfgit closed pull request #13: Basic example to connect S7 device to Google Cloud IoT Core

2018-09-06 Thread GitBox
asfgit closed pull request #13: Basic example to connect S7 device to Google Cloud IoT Core URL: https://github.com/apache/incubator-plc4x/pull/13 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:

[GitHub] skorikov opened a new pull request #16: Implemented throttling in Kafka Source Connector

2018-09-10 Thread GitBox
skorikov opened a new pull request #16: Implemented throttling in Kafka Source Connector URL: https://github.com/apache/incubator-plc4x/pull/16 A new configuration option `rate` is implemented, which allows to limit the polling rate to a given value (for example every 1000 milliseconds).

[GitHub] chrisdutz closed pull request #16: Implemented throttling in Kafka Source Connector

2018-09-10 Thread GitBox
chrisdutz closed pull request #16: Implemented throttling in Kafka Source Connector URL: https://github.com/apache/incubator-plc4x/pull/16 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

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213965185 ## File path: integrations/apache-camel/src/main/java/org/apache/plc4x/camel/Plc4XConsumer.java ## @@ -84,22 +79,29 @@

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213966073 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/connection/PlcProprietarySender.java ## @@ -1,41 +0,0 @@ -/*

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213967016 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcFieldResponse.java ## @@ -0,0 +1,42 @@ +/*

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213965327 ## File path: integrations/apache-camel/src/main/java/org/apache/plc4x/camel/Plc4XConsumer.java ## @@ -112,25 +114,25 @@

[GitHub] chrisdutz commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
chrisdutz commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213980628 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/connection/PlcProprietarySender.java ## @@ -1,41 +0,0 @@

[GitHub] chrisdutz commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
chrisdutz commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213981646 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcFieldRequest.java ## @@ -0,0 +1,31 @@ +/*

[GitHub] chrisdutz commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
chrisdutz commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213981663 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcFieldResponse.java ## @@ -0,0 +1,42 @@ +/*

[GitHub] chrisdutz commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
chrisdutz commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213981675 ## File path: plc4j/api/src/test/java/org/apache/plc4x/java/api/connection/PlcReaderTest.java ## @@ -38,24 +36,24 @@

[GitHub] sruehl commented on a change in pull request #14: WIP: Api Refactoring

2018-08-30 Thread GitBox
sruehl commented on a change in pull request #14: WIP: Api Refactoring URL: https://github.com/apache/incubator-plc4x/pull/14#discussion_r213981954 ## File path: plc4j/api/src/test/java/org/apache/plc4x/java/api/connection/PlcReaderTest.java ## @@ -38,24 +36,24 @@

[GitHub] skorikov opened a new pull request #18: Add support for multiple tasks in kafka sink connector

2018-09-11 Thread GitBox
skorikov opened a new pull request #18: Add support for multiple tasks in kafka sink connector URL: https://github.com/apache/incubator-plc4x/pull/18 This is an automated message from the Apache Git Service. To respond to

[GitHub] asfgit closed pull request #17: Add support for multiple queries in kafka source connector

2018-09-11 Thread GitBox
asfgit closed pull request #17: Add support for multiple queries in kafka source connector URL: https://github.com/apache/incubator-plc4x/pull/17 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:

[GitHub] asfgit closed pull request #18: Add support for multiple tasks in kafka sink connector

2018-09-11 Thread GitBox
asfgit closed pull request #18: Add support for multiple tasks in kafka sink connector URL: https://github.com/apache/incubator-plc4x/pull/18 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

[GitHub] asfgit closed pull request #19: Add URL Field to Key Schema in Kafka Source Connector

2018-09-11 Thread GitBox
asfgit closed pull request #19: Add URL Field to Key Schema in Kafka Source Connector URL: https://github.com/apache/incubator-plc4x/pull/19 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

[GitHub] dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-27 Thread GitBox
dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170929853 ## File path:

[GitHub] dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-27 Thread GitBox
dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r170931306 ## File path: plc4j/core/src/test/java/org/apache/plc4x/java/PlcDriverManagerTest.java ##

[GitHub] dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver

2018-02-27 Thread GitBox
dlaboss commented on a change in pull request #8: cleanup warnings in plc4j-api, plc4j-core, dummy-driver URL: https://github.com/apache/incubator-plc4x/pull/8#discussion_r171087658 ## File path: plc4j/api/src/main/java/org/apache/plc4x/java/api/messages/PlcWriteResponse.java

[GitHub] JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version

2018-09-30 Thread GitBox
JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/26#discussion_r221467015 ## File path:

[GitHub] JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version

2018-09-30 Thread GitBox
JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/26#discussion_r221466988 ## File path:

[GitHub] JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version

2018-09-30 Thread GitBox
JulianFeinauer commented on a change in pull request #26: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/26#discussion_r221466969 ## File path:

[GitHub] JulianFeinauer commented on issue #21: Made PlcInvalidFieldException.java checked.

2018-09-30 Thread GitBox
JulianFeinauer commented on issue #21: Made PlcInvalidFieldException.java checked. URL: https://github.com/apache/incubator-plc4x/pull/21#issuecomment-425737626 Hey @chrisdutz and @sruehl just wanted to bump that up. I think we should decide here on something and either merge or decline

[GitHub] JulianFeinauer opened a new pull request #29: Opm plcentitymanager

2018-10-22 Thread GitBox
JulianFeinauer opened a new pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29 I just finished the first (working) implementation of the Object-Plc-Mapping (OPM). This is a JPA Like functionality on top of plc4x which allows to issue requests

[GitHub] JulianFeinauer commented on issue #26: connection manager for connection pool initial version

2018-10-22 Thread GitBox
JulianFeinauer commented on issue #26: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/26#issuecomment-431838474 This PR *should* not be merged but all necessary changes should be done in #30.

[GitHub] JulianFeinauer opened a new pull request #30: connection manager for connection pool initial version

2018-10-22 Thread GitBox
JulianFeinauer opened a new pull request #30: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/30 This is a duplicate of #26 to allow @sruehl or @JulianFeinauer to do changes on the branch as Marcel is currenlty not available.

[GitHub] skorikov opened a new pull request #32: Provide connection metadata

2018-10-26 Thread GitBox
skorikov opened a new pull request #32: Provide connection metadata URL: https://github.com/apache/incubator-plc4x/pull/32 As discussed in the mailing list, connection capabilities should be obtained via metadata and remove `Optional`s from the `PlcConnection`.

[GitHub] sruehl closed pull request #30: connection manager for connection pool initial version

2018-10-26 Thread GitBox
sruehl closed pull request #30: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/30 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit closed pull request #32: Provide connection metadata

2018-10-26 Thread GitBox
asfgit closed pull request #32: Provide connection metadata URL: https://github.com/apache/incubator-plc4x/pull/32 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[GitHub] chrisdutz commented on issue #36: Add simple mock driver

2018-11-01 Thread GitBox
chrisdutz commented on issue #36: Add simple mock driver URL: https://github.com/apache/incubator-plc4x/pull/36#issuecomment-435013325 I thought it would make more sense to start removing all of these mock drivers and instead extend the "Test-Driver" as this should provide everything

[GitHub] JulianFeinauer commented on issue #36: Add simple mock driver

2018-11-01 Thread GitBox
JulianFeinauer commented on issue #36: Add simple mock driver URL: https://github.com/apache/incubator-plc4x/pull/36#issuecomment-435101505 @chrisdutz I'll check if I can add this Mock functionality in there. As I recall the test driver is currently working against internal values or

[GitHub] JulianFeinauer closed pull request #36: Add simple mock driver

2018-11-01 Thread GitBox
JulianFeinauer closed pull request #36: Add simple mock driver URL: https://github.com/apache/incubator-plc4x/pull/36 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] JulianFeinauer opened a new pull request #37: Extend test driver with mock

2018-11-01 Thread GitBox
JulianFeinauer opened a new pull request #37: Extend test driver with mock URL: https://github.com/apache/incubator-plc4x/pull/37 @chrisdutz here is my approach as extension to the Test Driver. @sruehl It is still not in the Test-utils package but I'm unsure if it should as I'm unsure

[GitHub] sruehl commented on issue #30: connection manager for connection pool initial version

2018-10-26 Thread GitBox
sruehl commented on issue #30: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/30#issuecomment-433380970 I haven't closed this... What the...??? This is an automated

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228232133 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228230429 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228231695 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228230640 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on issue #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on issue #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#issuecomment-433106003 looks good to me :) This is an automated message from the Apache Git Service. To respond to the

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228231802 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228230727 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228230145 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntity.java ## @@ -0,0 +1,34 @@ +/* + Licensed to

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228230930 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] sruehl commented on a change in pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
sruehl commented on a change in pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29#discussion_r228231315 ## File path: plc4j/utils/opm/src/main/java/org/apache/plc4x/java/opm/PlcEntityManager.java ## @@ -0,0 +1,380 @@ +/* +

[GitHub] asfgit closed pull request #29: Opm plcentitymanager

2018-10-25 Thread GitBox
asfgit closed pull request #29: Opm plcentitymanager URL: https://github.com/apache/incubator-plc4x/pull/29 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use

[GitHub] sruehl closed pull request #26: connection manager for connection pool initial version

2018-10-25 Thread GitBox
sruehl closed pull request #26: connection manager for connection pool initial version URL: https://github.com/apache/incubator-plc4x/pull/26 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

  1   2   3   4   5   6   7   8   9   10   >