-
Notifications
You must be signed in to change notification settings - Fork 76
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
Rya 135 collection name #305
base: master
Are you sure you want to change the base?
Conversation
3593c53
to
f9691c7
Compare
Refer to this link for build results (access rights to CI server needed): |
There is no need for the triples collection to be configurable, as only the mongo database name will change the rya instance.
f9691c7
to
c7406d7
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): Build result: FAILURE[...truncated 92.71 MB...][INFO] ------------------------------------------------------------------------[INFO] Total time: 01:06 h[INFO] Finished at: 2018-10-01T17:56:20+00:00[INFO] Final Memory: 418M/2808M[INFO] ------------------------------------------------------------------------Waiting for Jenkins to finish collecting data[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.2:compile (default-compile) on project rya.kafka.connect.mongo: Compilation failure[ERROR] /home/jenkins/jenkins-slave/workspace/incubator-rya-master-with-optionals-pull-requests/extras/kafka.connect/mongo/src/main/java/org/apache/rya/kafka/connect/mongo/MongoRyaSinkTask.java:[106,18] cannot find symbol[ERROR] symbol: method setMongoDBName(java.lang.String)[ERROR] location: variable ryaConfig of type org.apache.rya.mongodb.MongoDBRdfConfiguration[ERROR] -> [Help 1][ERROR] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.[ERROR] Re-run Maven using the -X switch to enable full debug logging.[ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles:[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException[ERROR] [ERROR] After correcting the problems, you can resume the build with the command[ERROR] mvn -rf :rya.kafka.connect.mongochannel stoppedSetting status of c7406d7 to FAILURE with url https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/781/ and message: 'FAILURE 'Using context: Jenkins: clean package -Pgeoindexing |
Refer to this link for build results (access rights to CI server needed): Failed Tests: 3incubator-rya-master-with-optionals-pull-requests/org.apache.rya:rya.prospector: 3 |
|
||
// MongoDB Server connection values. | ||
public static final String MONGO_HOSTNAME = "mongo.db.instance"; | ||
public static final String MONGO_PORT = "mongo.db.port"; | ||
|
||
// MongoDB Database values. | ||
public static final String MONGO_DB_NAME = "mongo.db.name"; | ||
public static final String RYA_INSTANCE_NAME = "mongo.db.name"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this field should be renamed. It's still the Mongo DB Name. Elsewhere in the code we should decide that the rya instance name also happens to be the db name. If this is in fact just the instance name, and the DB name isn't allowed to be configured anymore, then the field's value shouldn't be "mongo.db.name" anymore.
Description
Merging existing content into master. Already reviewed and merged into 3.x