Skip to content

JENA-1835: Capture and expose lexicalForm and dataType in DatatypeFormatException#686

Merged
afs merged 2 commits intoapache:masterfrom
superfell:datatype_format_exception
Feb 5, 2020
Merged

JENA-1835: Capture and expose lexicalForm and dataType in DatatypeFormatException#686
afs merged 2 commits intoapache:masterfrom
superfell:datatype_format_exception

Conversation

@superfell
Copy link
Copy Markdown
Contributor

@superfell superfell commented Feb 3, 2020

This implements the TODO in this class.

Copy link
Copy Markdown
Member

@rvesse rvesse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one minor Javadoc typo to address

/**
* The datatype that has an invalid lexical form.
*
* @return the datatype that this excpetion is related to. Maybe null depending on how the exception was constructed.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo, excpetion -> exception

@afs
Copy link
Copy Markdown
Member

afs commented Feb 4, 2020

Recorded as https://issues.apache.org/jira/browse/JENA-1835, changed the PR title.

@superfell - Is your JIRA id the same as GH? I see 4 JIRA accounts with the same name.

@afs afs changed the title Capture and expose lexicalForm and dataType in DatatypeFormatException JENA-1835: Capture and expose lexicalForm and dataType in DatatypeFormatException Feb 5, 2020
@superfell
Copy link
Copy Markdown
Contributor Author

@afs My Jira username is sfell

@afs afs merged commit 6641533 into apache:master Feb 5, 2020
@superfell superfell deleted the datatype_format_exception branch February 5, 2020 17:57
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.

3 participants