Skip to content

Write the javadoc of the object layer and the R-tree search operators as javadoc reads it - #118

Open
estebanzimanyi wants to merge 1 commit into
MobilityDB:mainfrom
estebanzimanyi:build/javadoc-without-warnings
Open

estebanzimanyi wants to merge 1 commit into
MobilityDB:mainfrom
estebanzimanyi:build/javadoc-without-warnings

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

ObjectLayerGenerator names the object-model class a generated interface serves as code, not
as a link: the class is a MEOS object-model key, and TBigint, TNumber, TInt and TFloat are not
types the generated package imports. RTreeSearchOp documents each operator in a javadoc
comment before its constant.

Witness. The build reported 33 javadoc warnings: "reference not found" for TBigint, TFloat,
TInt and TNumber, six times each, and "invalid input: '<'" for the three trailing /**<
comments of RTreeSearchOp, three times each.

Measured. Against the catalog and libmeos of MobilityDB 8a23781e4c with MEOS-API 8dddf59,
the build reports no javadoc warning, and the suites run 106 and 1,810 tests, all passing.

… as javadoc reads it

ObjectLayerGenerator names the object-model class a generated interface serves as code, not
as a link: the class is a MEOS object-model key, and TBigint, TNumber, TInt and TFloat are not
types the generated package imports. RTreeSearchOp documents each operator in a javadoc
comment before its constant.

Witness. The build reported 33 javadoc warnings: "reference not found" for TBigint, TFloat,
TInt and TNumber, six times each, and "invalid input: '<'" for the three trailing /**<
comments of RTreeSearchOp, three times each.

Measured. Against the catalog and libmeos of MobilityDB 8a23781e4c with MEOS-API 8dddf59,
the build reports no javadoc warning, and the suites run 106 and 1,810 tests, all passing.

This branch has not been deployed

No deployments
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.

1 participant