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

Dataset contains several resources of the same name. #223

Open
eea71 opened this issue Oct 2, 2015 · 4 comments
Open

Dataset contains several resources of the same name. #223

eea71 opened this issue Oct 2, 2015 · 4 comments

Comments

@eea71
Copy link

eea71 commented Oct 2, 2015

image
Some hints. DPU - uv-l-relationalToCkan - contains check box which is not checked by default. The question is what is wished functionality - Do we want the same resource twice and more? The next result is, that resource at the Public catalogue is not consequently updated. Because of the same name!

@skrchnavy
Copy link
Contributor

This is feature of CKAN, resource is identified by it's ID not name.
Is this against some requirement?

@eea71
Copy link
Author

eea71 commented Oct 2, 2015

If there is more than one resource with the same name. Then it raised problems with updating in PC, because here it is according the name.

@hanecak
Copy link
Contributor

hanecak commented Dec 3, 2015

Desired behaviour:

  1. to have "Overwrite existing resources" checked by default
  2. to fail to create (new, duplicate) resource(s) if "Overwrite existing resources" is not checked and resource with same name already exists

That would be a consistent behaviour and in line with how uv-l-filesToCkan is already implementing it:
screenshot from 2015-12-03 15 14 02

Execution failed because: Resource data-catalog-stats-current.csv already exists at destinationeu.unifiedviews.dpu.DPUException: Resource data-catalog-stats-current.csv already exists at destination at eu.unifiedviews.helpers.dpu.context.ContextUtils.dpuException(ContextUtils.java:226)

@eea03
Copy link

eea03 commented Dec 3, 2015

Which version of DPU was used?
I cannot reproduce it. From what I can say, it behaves as expected.
When overwrite not checked and resource already exists, it raises the warning (probably should be error):

Resource was skipped because it already exists in the catalog and overwrite mode is not enabled

I tested it with various resource names including "INE"
I cannot understand why this would not work, it uses the same mechanism as FilesToCkan.

I was testing with DPU v 1.1.6 but there weren't any changes that could influence this since v1.1.4.

Execution debug logs would help a lot, or even better access to the test server.

I tested it on eDemo DEV server and could not reproduce it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants