Skip to content
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

Evaluator Server Update #62

Merged
merged 6 commits into from
Sep 19, 2019
Merged

Conversation

kiroco12
Copy link
Contributor

Resolved compilation problems caused by the dependencies update. This PR does not solve compilation problems in the test module, this will be added in a future PR.

This evaluator resolves dependencies, compiles and executes code using a REST API. This request code has been used to test the evaluator using postman:

{ "code": "{import cats.effect._\nimport cats.syntax.all._\nimport fetch._\nimport cats.data.NonEmptyList\n\ntrait DataSource[F[_], Identity, Result]{\n def data: Data[Identity, Result]\n\n def CF: Concurrent[F]\n\n def fetch(id: Identity): F[Option[Result]]\n\n def batch(ids: NonEmptyList[Identity]): F[Map[Identity, Result]]\n}\ntype UserId = Int\ncase class User(id: UserId, username: String)\n\ndef latency[F[_] : Concurrent](msg: String): F[Unit] = for {\n _ <- Sync[F].delay(println(s\"--> [${Thread.currentThread.getId}] $msg\"))\n_ <- Sync[F].delay(Thread.sleep(100))\n _ <- Sync[F].delay(println(s\"<-- [${Thread.currentThread.getId}] $msg\"))\n} yield ()}", "resolvers": [ "https://oss.sonatype.org/content/repositories/snapshots", "https://oss.sonatype.org/content/repositories/public" ], "dependencies": [ { "groupId": "com.47deg", "version": "1.2.0", "artifactId": "fetch_2.11" } ] }

Remaining parts to finish the evaluator update are the solution of compilation problems in the server/test module and the creation of a new http client for the client module.

project/ProjectPlugin.scala Outdated Show resolved Hide resolved
Copy link
Contributor

@juanpedromoreno juanpedromoreno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@kiroco12 kiroco12 merged commit a777e75 into enrique-2-12-10-update Sep 19, 2019
@kiroco12 kiroco12 deleted the enrique-update-server branch September 19, 2019 09:49
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.

4 participants