search

JanusGraph / janusgraph-foundationdb

FoundationDB storage adapter for JanusGraph
Other
53 stars 18 forks source link

Remove trailing commas from yaml config #58

Open rpuch opened 3 years ago

rpuch commented 3 years ago

Actually, I'm not 100% sure that this is the correct fix to do as, according to git annotate, these lines are 3 years old, but with JanusGraph-0.5.2 I get the following error when starting the server:

74   [main] ERROR org.apache.tinkerpop.gremlin.server.GremlinServer  - Configuration file at conf/gremlin-server/gremlin-server.yaml could not be found or parsed properly. [while parsing a block mapping; expected <block end>, but found FlowEntry;  in 'reader', line 44, column 54:
     ... nabled: false, interval: 180000},                                         ^]

Removal of the trailing commas resolved the issue for me.

linux-foundation-easycla[bot] commented 3 years ago

CLA Signed

The committers are authorized under a signed CLA.

rngcntr commented 3 years ago

Hi @rpuch, thanks for your contribution! Could you please redo your commit with git commit --amend --no-edit --signoff so that the DCO check passes?

rpuch commented 3 years ago

Hi @rpuch, thanks for your contribution! Could you please redo your commit with git commit --amend --no-edit --signoff so that the DCO check passes?

Done!