Skip to content

Commit 629a722

Browse files
committed
Remove Serializable interface from OntologyResource
It makes no sense to have OntologyResource serializable if the actual implementation relies on non-serializable states supplied by Jena. Remove unnecessary transient on Jena fields. Fix a few fields that should be final.
1 parent d0ec535 commit 629a722

14 files changed

Lines changed: 9 additions & 48 deletions

src/ubic/basecode/ontology/jena/AbstractOntologyResource.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,11 @@ abstract class AbstractOntologyResource implements OntologyResource {
3737

3838
protected static Logger log = LoggerFactory.getLogger( AbstractOntologyResource.class );
3939

40-
private static final long serialVersionUID = 1L;
41-
4240
private static final Comparator<OntologyResource> comparator = Comparator
4341
.comparing( OntologyResource::getScore, nullsLast( reverseOrder() ) )
4442
.thenComparing( OntologyResource::getUri, nullsLast( naturalOrder() ) );
4543

46-
private transient final OntResource res;
44+
private final OntResource res;
4745
@Nullable
4846
private final Double score;
4947

src/ubic/basecode/ontology/jena/AbstractOntologyService.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,6 @@
3030
import org.apache.commons.lang3.time.StopWatch;
3131
import org.slf4j.Logger;
3232
import org.slf4j.LoggerFactory;
33-
import ubic.basecode.ontology.jena.vocabulary.BFO;
34-
import ubic.basecode.ontology.jena.vocabulary.RO;
3533
import ubic.basecode.ontology.model.OntologyIndividual;
3634
import ubic.basecode.ontology.model.OntologyModel;
3735
import ubic.basecode.ontology.model.OntologyResource;

src/ubic/basecode/ontology/jena/AnnotationPropertyImpl.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import com.hp.hpl.jena.rdf.model.Resource;
2323
import com.hp.hpl.jena.rdf.model.Statement;
2424
import com.hp.hpl.jena.vocabulary.RDFS;
25-
import ubic.basecode.ontology.jena.vocabulary.OBO;
2625
import ubic.basecode.ontology.model.AnnotationProperty;
2726

2827
import javax.annotation.Nullable;

src/ubic/basecode/ontology/jena/DatatypePropertyImpl.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@
2525
*/
2626
class DatatypePropertyImpl extends OntologyPropertyImpl implements DatatypeProperty {
2727

28-
/**
29-
*
30-
*/
31-
private static final long serialVersionUID = 1L;
32-
3328
private final Class<?> type;
3429

3530
public DatatypePropertyImpl( com.hp.hpl.jena.ontology.DatatypeProperty resource ) {

src/ubic/basecode/ontology/jena/IndexerSelector.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
import com.hp.hpl.jena.rdf.model.*;
2222
import com.hp.hpl.jena.vocabulary.OWL2;
2323
import com.hp.hpl.jena.vocabulary.RDFS;
24-
import ubic.basecode.ontology.jena.vocabulary.OBO;
2524

2625
import java.util.Collection;
2726
import java.util.HashSet;

src/ubic/basecode/ontology/jena/ObjectPropertyImpl.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,11 @@
3434
*/
3535
class ObjectPropertyImpl extends OntologyPropertyImpl implements ubic.basecode.ontology.model.ObjectProperty {
3636

37-
/**
38-
*
39-
*/
40-
private static final long serialVersionUID = 1L;
4137
private final com.hp.hpl.jena.ontology.ObjectProperty resource;
4238
private final Set<Restriction> additionalRestrictions;
4339

4440
public ObjectPropertyImpl( ObjectProperty resource, Set<Restriction> additionalRestrictions ) {
4541
super( resource );
46-
this.isFunctional = resource.isFunctionalProperty();
4742
this.resource = resource;
4843
this.additionalRestrictions = additionalRestrictions;
4944
}

src/ubic/basecode/ontology/jena/OntologyCardinalityRestrictionImpl.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,8 @@
2929
class OntologyCardinalityRestrictionImpl extends OntologyRestrictionImpl implements
3030
OntologyCardinalityRestriction {
3131

32-
/**
33-
*
34-
*/
35-
private static final long serialVersionUID = 1L;
36-
int cardinality = 0;
37-
CardinalityType cardType;
32+
private final int cardinality;
33+
private final CardinalityType cardType;
3834

3935
public OntologyCardinalityRestrictionImpl( Restriction resource, Set<Restriction> additionalRestrictions ) {
4036
super( resource, additionalRestrictions );

src/ubic/basecode/ontology/jena/OntologyClassRestrictionImpl.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,6 @@
3434
* @author pavlidis
3535
*/
3636
class OntologyClassRestrictionImpl extends OntologyRestrictionImpl implements OntologyClassRestriction {
37-
/**
38-
*
39-
*/
40-
private static final long serialVersionUID = 1L;
4137
private final RDFNode value;
4238
private final OntologyTerm restrictedTo;
4339
private final Set<Restriction> additionalRestrictions;

src/ubic/basecode/ontology/jena/OntologyDatatypeRestrictionImpl.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@
2929
*/
3030
class OntologyDatatypeRestrictionImpl extends OntologyRestrictionImpl implements OntologyDatatypeRestriction {
3131

32-
/**
33-
*
34-
*/
35-
private static final long serialVersionUID = 1L;
3632
private final Class<?> type;
3733

3834
public OntologyDatatypeRestrictionImpl( Restriction resource, Set<Restriction> additionalRestrcitions ) {

src/ubic/basecode/ontology/jena/OntologyIndividualImpl.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@
3535
*/
3636
class OntologyIndividualImpl extends AbstractOntologyResource implements OntologyIndividual {
3737

38-
private static final long serialVersionUID = -6164561945940667693L;
39-
4038
private final Individual ind;
4139
private final Set<Restriction> additionalRestrictions;
4240

0 commit comments

Comments
 (0)