Skip to content

feature(metrics): Add InfluxDB for metrics#3

Open
yk-sgr wants to merge 1 commit intomasterfrom
feature/add-metrics
Open

feature(metrics): Add InfluxDB for metrics#3
yk-sgr wants to merge 1 commit intomasterfrom
feature/add-metrics

Conversation

@yk-sgr
Copy link
Copy Markdown
Member

@yk-sgr yk-sgr commented Mar 29, 2020

No description provided.

@yk-sgr yk-sgr requested a review from DRSchlaubi March 29, 2020 12:30
@yk-sgr yk-sgr self-assigned this Mar 29, 2020
@yk-sgr yk-sgr linked an issue Mar 29, 2020 that may be closed by this pull request
scheduler.cancel()
context.close()
}
} No newline at end of file
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.

Add a new line here because they're important


private val context = Executors.newSingleThreadExecutor().asCoroutineDispatcher()

@OptIn(ObsoleteCoroutinesApi::class)
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.

I think you need to add Xopt-in=kotlin.RequiresOptIn as a compiler argument to the build.gradle.kts

*/
const val NAME = "service_name"
}
} No newline at end of file
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.

add a new line here as well

## What to change
- Package name: `app.ryss.service`
- Main Service class: `app.ryss.service.core.Service`
- Service Name: `app.ryss.service.core.Service#NAME`
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.

Afaik you don't actually use the # to reference properties in Kotlin you just use a normal .

@DRSchlaubi DRSchlaubi linked an issue Mar 29, 2020 that may be closed by this pull request
@DRSchlaubi DRSchlaubi removed a link to an issue Mar 29, 2020
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.

Add InfluxDB for metrics

2 participants