dump #50

Merged
claudio.atzori merged 98 commits from miriam.baglioni/dnet-hadoop:dump into master 4 years ago
Collaborator

changes in this PR:

  • Class to create an archive moved to common module. It is now possible to specify the maz size in GB of each archive to be created
  • changed the parameter for the upload in Zenodo. Now we have three possible values for the uload:
    • new for a brand new deposition
    • version for a new version of an old deposition (the concept record id is needed in this case)
    • upload to continue uploading a not yet published deposition (deposition id needed in this case)
changes in this PR: - Class to create an archive moved to common module. It is now possible to specify the maz size in GB of each archive to be created - changed the parameter for the upload in Zenodo. Now we have three possible values for the uload: - new for a brand new deposition - version for a new version of an old deposition (the concept record id is needed in this case) - upload to continue uploading a not yet published deposition (deposition id needed in this case)
miriam.baglioni added the
enhancement
label 4 years ago
sandro.labruzzo was assigned by miriam.baglioni 4 years ago
claudio.atzori was assigned by miriam.baglioni 4 years ago
claudio.atzori reviewed 4 years ago
@ -110,3 +73,1 @@
ar.close();
}
// private static TarArchiveOutputStream getTar(FileSystem fileSystem, String outputPath) throws IOException {
Owner

Please delete commented code. It was factored out in the MakeTar utility class, right?

Please delete commented code. It was factored out in the MakeTar utility class, right?
claudio.atzori reviewed 4 years ago
@ -381,0 +398,4 @@
}
private static <I extends Instance> void setCommonValue(eu.dnetlib.dhp.schema.oaf.Instance i, I instance) {// <I
// extends
Owner

further comments to drop

further comments to drop
claudio.atzori requested changes 4 years ago
claudio.atzori left a comment
Owner

Please move all the PidGraph stuff in a different branch.

Please move all the PidGraph stuff in a different branch.
Poster
Collaborator

Check it now. I have removed the pid graph dump stuff and also the commented lines

Check it now. I have removed the pid graph dump stuff and also the commented lines
claudio.atzori closed this pull request 4 years ago
The pull request has been merged as f5f346dd2b.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b miriam.baglioni-dump master
git pull dump

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff miriam.baglioni-dump
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No project
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: D-Net/dnet-hadoop#50
Loading…
There is no content yet.