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.

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

Support JSON-like resource structures #460

Closed
wants to merge 9 commits into from
Closed

Support JSON-like resource structures #460

wants to merge 9 commits into from

Conversation

bdenton
Copy link
Collaborator

@bdenton bdenton commented Oct 4, 2016

(This is resubmission after refactoring to eliminate impact on existing interfaces)

This enhancement addresses the need to be able to serialize FHIR resources to and from streams other that Json-P or Gson. Here are two examples:

  1. When building an equivalent of the jpaserver implemented in native MongoDB (no JPA or JDO), it was necessary to be able to serialize/deserialize FHIR resources via BSONReader and BSONWriter. This is possible (with some pain) with Json-P but is not at all possible with Gson.

  2. We have built a general purpose data mapping engine that is able to map from a set of customer/application-specific sets of data into FHIR resources. The output of this mapping engine is a JSON-like data structure implemented in Java Maps, Lists, and scalars.

This pull request extends the IParser interface and updates the JSONParser to implement the sub-interface so that any sort of Json-like data structure or framework can be integrated in the future.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.5%) to 87.335% when pulling dd50a0a on bdenton:support-json-like-resources into 61d6f1b on jamesagnew:master.

@jamesagnew
Copy link
Collaborator

Hi Bill,

I'm trying to give this a review, but I'm having a lot of trouble reading it. GitHub shows that the patch touches 1761 files so it won't even put it all on the diff screen.. I'm assuming for example that you didn't actually modify android-sources.xml for example, so I think something might be wrong with the PR?

Any ideas? I'd like to get this merged but it seems like in its current form it modifies 3/4 of the repo for some reason..

@bdenton
Copy link
Collaborator Author

bdenton commented Oct 11, 2016

James..

I am working on redoing the pull request..

What happened is that I originally submitted the PR from a branch on my fork.. that went along having build errors not related to my changes..

So, then , there ended up being a conflict in my branch so, I did an upstream merge into my local branch, fixed the conflict (not related to my change).. and re-committed and re-pushed my changed branch

This was obviously a mistake… I did not intentionally mean to change 1700+ files.. I actually only changed one file in the HAPI code stream. That one file is JsonParser. I added the package “ca.uhn.fhir.parser.json” and added the IJsonLikeParser overloaded interface… the rest is my GIT bumbling

I am kind of new to GIT and its collaboration features so any guidance on how to make changes and correctly submit pull requests would be appreciated since I obviously don’t have the correct procedure going for me.

Cheers, Bill

@bdenton
Copy link
Collaborator Author

bdenton commented Oct 14, 2016

I am going to close this pull request yet again and try it again.. hopefully GIT will cooperate this time.

@bdenton bdenton closed this Oct 14, 2016
@bdenton bdenton deleted the support-json-like-resources branch October 14, 2016 23:47
@bdenton bdenton restored the support-json-like-resources branch October 14, 2016 23:47
@bdenton bdenton deleted the support-json-like-resources branch October 14, 2016 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants