-
Notifications
You must be signed in to change notification settings - Fork 9
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
Section 6.6.1 Automatically deriving datatypes is underspecified #91
Comments
Strong preference for proposal 2. |
Then that means that some test cases need to be adapted (e.g., some JSON values have integer values that should be transformed as such). And somebody creating those notes, of course. |
+1 |
+1 for proposal 2 |
+1 for proposal 2, but in line with the respective specs, JSON has following primitive types:
for XML: why not take over the datatype as specified in XML (sure it'll be XSD in most cases, but all other cases should also be covered no?) |
The datatype for XML without a schema would be a string and one of the schemas if it can be looked up. XML data types only "exist" in the schema. XML DTDs, another XML schema language, only has character data which are strings. The problem, however, is that you then have two cases:
I believe core should support the basic case, and IO could handle both |
I feel this issue is already being addressed in the rml-io-registry repository, right? Or do we want a default/basic behaviour in the rml-core? @pmaria |
Section "6.6.1 Automatically deriving datatypes" is underspecified. The test cases assume that all values are string literals. The spec does mention automatically deriving datatypes for SQL with no "conversion tables" as specified in R2RML.
Proposal 1:
Proposal 2:
Proposal 2 would then allow 6.6.1 be rewritten as
The text was updated successfully, but these errors were encountered: