[GitHub] nifi issue #1450: NIFI-3339b Add getDataSource() to DBCPService, second vers...

2017-03-08 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/1450
  
@ijokarumawak @mattyb149

We could add example how to use String JDBC to wiki.
No need to be part of NiFi codebase.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #1450: NIFI-3339b Add getDataSource() to DBCPService, second vers...

2017-02-04 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/1450
  
Hi @ijokarumawak
I have something like this already in use. This was meant to be temporary 
solution until “right” implementation is available.

Now I am little bit confused.
Wrapping outside of the DBCPService feels like temporary workaround? No?

And just to be sure I need to do some more testing.

Thanks
Toivo



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #1417: NIFI-3339 Add getDataSource() to DBCPService.

2017-02-03 Thread ToivoAdams
Github user ToivoAdams closed the pull request at:

https://github.com/apache/nifi/pull/1417


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #1450: NIFI-3339b Add getDataSource() to DBCPService, seco...

2017-01-29 Thread ToivoAdams
GitHub user ToivoAdams opened a pull request:

https://github.com/apache/nifi/pull/1450

NIFI-3339b Add getDataSource() to DBCPService, second version

Thank you for submitting a contribution to Apache NiFi.

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and submit an update to your PR as soon as possible.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ToivoAdams/nifi nifi-3339b

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/1450.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1450


commit 30cd21d7064af611bae6310e959b60df257249fc
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2017-01-29T11:40:37Z

NIFI-3339b Add getDataSource() to DBCPService, second version




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #984: Nifi 2624

2016-10-16 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/984
  
I created new pull request #1142
Could not reopen this PR


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #1142: NIFI-2624 JdbcCommon treats BigDecimals now as Avro...

2016-10-16 Thread ToivoAdams
GitHub user ToivoAdams opened a pull request:

https://github.com/apache/nifi/pull/1142

NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logical type usin…

Thank you for submitting a contribution to Apache NiFi.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

### For all changes:
- [ ] Is there a JIRA ticket associated with this PR? Is it referenced 
 in the commit message?

- [ ] Does your PR title start with NIFI- where  is the JIRA number 
you are trying to resolve? Pay particular attention to the hyphen "-" character.

- [ ] Has your PR been rebased against the latest commit within the target 
branch (typically master)?

- [ ] Is your initial contribution a single, squashed commit?

### For code changes:
- [ ] Have you ensured that the full suite of tests is executed via mvn 
-Pcontrib-check clean install at the root nifi folder?
- [ ] Have you written or updated unit tests to verify your changes?
- [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)? 
- [ ] If applicable, have you updated the LICENSE file, including the main 
LICENSE file under nifi-assembly?
- [ ] If applicable, have you updated the NOTICE file, including the main 
NOTICE file found under nifi-assembly?
- [ ] If adding new Properties, have you added .displayName in addition to 
.name (programmatic access) for each of the new properties?

### For documentation related changes:
- [ ] Have you ensured that format looks appropriate for the output in 
which it is rendered?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and submit an update to your PR as soon as possible.

…g bytes to hold data (not String as is was before).

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ToivoAdams/nifi nifi-2624b

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/1142.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1142


commit 4d6de9663087ebb30041341c8d55de01c4af3875
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-10-16T17:15:33Z

NIFI-2624 JdbcCommon treats BigDecimals now as Avro Logical type using 
bytes to hold data (not String as is was before).




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #984: Nifi 2624

2016-10-08 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/984
  
@trixpan 
I did rebase, resolved conflicts.
Git refused to push. So I used force push.
toivo@Optx:~/git/nifi$ git push origin -f nifi-2624
Counting objects: 2712, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (1079/1079), done.
Writing objects: 100% (2712/2712), 2.51 MiB | 1.81 MiB/s, done.
Total 2712 (delta 952), reused 2399 (delta 740)
remote: Resolving deltas: 100% (952/952), completed with 402 local objects.
To g...@github.com:ToivoAdams/nifi.git
 + 7e33a04...540ef63 nifi-2624 -> nifi-2624 (forced update)

Now PR is closed?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #984: Nifi 2624

2016-10-08 Thread ToivoAdams
Github user ToivoAdams closed the pull request at:

https://github.com/apache/nifi/pull/984


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #388: nifi-1214b Mock Framework should allow order-indepen...

2016-09-21 Thread ToivoAdams
Github user ToivoAdams closed the pull request at:

https://github.com/apache/nifi/pull/388


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #803: nifi-1214c Mock Framework should allow order-indepen...

2016-09-21 Thread ToivoAdams
Github user ToivoAdams closed the pull request at:

https://github.com/apache/nifi/pull/803


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #803: nifi-1214c Mock Framework should allow order-independent as...

2016-09-20 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/803
  
New PR #1033

I did rebase as usual.
But git refused to push and recommended pull before push.
I did pull but it seems I should force push instead of pull.

Thanks
Toivo


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #1033: nifi-1214d Mock Framework …on FlowFiles. This rep...

2016-09-20 Thread ToivoAdams
GitHub user ToivoAdams opened a pull request:

https://github.com/apache/nifi/pull/1033

nifi-1214d Mock Framework …on FlowFiles. This replaces previous nifi-1214c

New Pull Request, replaces previous #803 

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ToivoAdams/nifi nifi-1214d

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/1033.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1033


commit 639df94ed2282c0efc11e0249b94c9e739c3aef5
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-09-20T08:35:07Z

nifi-1214d Mock Framework …on FlowFiles. This replaces previous nifi-1214c




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #986: Nifi 2381

2016-09-18 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/986
  
Hi @pvillard31 

I did changes.
Thank you for reviewing.
Toivo


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #803: nifi-1214c Mock Framework should allow order-independent as...

2016-09-17 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/803
  
Git complained and I did what he recommend (git pull origin).
It seems this was mistake.
I will create another clean PR.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi pull request #984: Nifi 2624

2016-09-03 Thread ToivoAdams
GitHub user ToivoAdams opened a pull request:

https://github.com/apache/nifi/pull/984

Nifi 2624

JDBC-to-Avro processors handle BigDecimals as Strings.
JdbcCommon treats BigDecimals now as Avro Logical type using bytes to hold 
data (not String as is was before).


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ToivoAdams/nifi nifi-2624

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/nifi/pull/984.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #984


commit 96717c6f39039f1af70ab7e66510fb3dfbf5732b
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-08-28T16:56:58Z

nifi-2624 JDBC-to-Avro processors handle BigDecimals as Strings. Upgrade 
Avro 1.7.1 to 1.8.1

commit 5cf2848b92b69e3f8971883835df8e425e59698c
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-08-30T18:00:57Z

nifi-2624 JDBC-to-Avro processors handle BigDecimals as Strings. First 
version.

commit 90399c7382c93e188740433a4c653725429a2ab1
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-08-30T18:16:58Z

nifi-2624 JDBC-to-Avro processors handle BigDecimals as Strings. Fix tabs.

commit 7e33a0449244f6058726aea3f4a1591922efbe87
Author: Toivo Adams <toivo.ad...@gmail.com>
Date:   2016-09-03T08:11:30Z

nifi-2624 JDBC-to-Avro processors handle BigDecimals as Strings. Cleanup.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] nifi issue #388: nifi-1214b Mock Framework should allow order-independent as...

2016-07-23 Thread ToivoAdams
Github user ToivoAdams commented on the issue:

https://github.com/apache/nifi/pull/388
  
I hope conflicts are now resolved.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---