Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

adds triple-to-stream in order to process viaf-dump data for entityfacts #170

Merged
merged 1 commit into from
Feb 12, 2014
Merged

adds triple-to-stream in order to process viaf-dump data for entityfacts #170

merged 1 commit into from
Feb 12, 2014

Conversation

schaeferd
Copy link
Contributor

so you are able to morph after redirect

cboehme added a commit that referenced this pull request Feb 12, 2014
@cboehme
Merge pull request #170 from schaeferd/triples-to-stream
484bb82 
adds triple-to-stream in order to process viaf-dump data for entityfacts
@cboehme cboehme merged commit 484bb82 into metafacture:master Feb 12, 2014
blackwinter added a commit that referenced this pull request Dec 13, 2024
@fsteeg @blackwinter
Integrate feedback from #170 (see #102)
1bcc5ca 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter added a commit that referenced this pull request Dec 13, 2024
@fsteeg @blackwinter
Integrate feedback from #170 (see #102)
8b27a52 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter added a commit that referenced this pull request Dec 13, 2024
@fsteeg @blackwinter
Some cleanup as suggested in #170 (see #102)
93aaf2e 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Fix test to actually use internal map (#170, #102)
6f455eb
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Remove unused MetafixToDo.Extension (#170, #102)
b376291
blackwinter added a commit that referenced this pull request Dec 13, 2024
@fsteeg @blackwinter
Handle null results directly in transform (#102, #149, #170)
238e840 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing test case for asterisk in replace_all(). (#170)
63bf98a 
Wrongly inserts subfield from second array item into first item.
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Remove unnecessary parens, add unit test (#102, #149, #170)
006e3ac
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Add test cases from review (#102, #170)
5494f42 
Co-authored-by: Jens Wille <jens.wille@hbz-nrw.de>

See ad0f3d5, 33d103f, 771205f, 49849a4, fa24c36, d5e67fc
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Add unit tests for failing integration test, mark as TODO (#170)
158c4cf
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Update passing integration tests (#102, #149, #170)
e027e1c
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Fix typo in integration test folder name (#170, #205)
96f65f4
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Only create new path if old value's path doesn't match (#102, #170)
64d8d98
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add unit test for record transformation. (#102, #170)
8cfe1a8
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Fix nested removal from array (#102, #113, #170)
e2cf74c
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add unit test for record transformation. (#102, #170)
2378455
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Preserve order in Record#transform (#102, #170)
f210dd3 
Mark tests covering faulty usage as TODOs
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Explicitly set path in record test (#102, #170)
a4d557e
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing test case for unsuccessful lookup. (f210dd3, #170)
a816a73 
Throws java.lang.IndexOutOfBoundsException.
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Delete reversed null results after ordered transform (#102, #170)
ec9f8ef
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing test case for asterisk in replace_all() with repeated f…
725097a 
…ield. (#170)
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Mark failing strictness tests as TODOs (#102, #170)
40ab054
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Update path for repeated fields to indexed array (#102, #170)
4324378
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing integration test for asterisk in replace_all() with rep…
cb4f27e 
…eated field. (#170, 725097a)
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Add unit tests for new integration test from cb4f27e (#170)
1d495ab
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Update path when adding, rename on copy/move field (#102, #170)
ca1cba1
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing test case for multiple replace_all() on same subfield. (#…
82bee32 
…170)
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Set correct path after inserting transformed value (#102, #170)
049c709
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Add failing test case for prepend() on array with wildcard. (#170)
7b74937 
String transformations should only operate on string values.
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Tweak test to produce correct order (#170)
4961b95 
Upcased value retains its position, so we add a new tmp value
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Replace matchType.ifString.orElseThrow with asString (#102, #170)
31cf8e9
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Revert "Tweak test to produce correct order (#170)"
eafca9e 
This reverts commit 4961b95.
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Update strictness tests for changed behaviour. (#170, 4961b95)
0dfcfb8
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Replace path heuristic with actual value check in lookup (#170)
2b476e2 
To reliably reject string operations on non-string fields
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Update strictness integration tests to correspond to unit tests
5e5bf5f 
See #170, 0dfcfb8
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Update TODOs on implicit append in unit & integration tests (#170)
540f43f
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Improve * wildcard behavior, additional tests (#102, #170)
6779544
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Wrap NumberFormatException into IllegalStateException (#170)
746fcf4
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Tweak shouldNotTrimIndexedArray, add shouldNotTrimHash test (#170)
9b8fa39
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Handle NumberFormatException in apply (#170)
5a3fa9b
blackwinter pushed a commit that referenced this pull request Dec 13, 2024
@fsteeg
Merge pull request #170 from metafacture/102-path
15e2237 
Consolidate path traversal
blackwinter added a commit that referenced this pull request Dec 13, 2024
@blackwinter
Only treat field name as FixPath when actually given a path. (#170, 8…
1ecbcdf 
…c42874)

IOW, don't split field names received from metadata stream (`Metafix.addValue()`).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@schaeferd @cboehme