search

twitter / cassovary

Cassovary is a simple big graph processing library for the JVM
http://twitter.com/cassovary
Apache License 2.0
1.05k stars 150 forks source link

Deprecate Ostrich and use Metrics #89

Closed pankajgupta closed 9 years ago

pankajgupta commented 10 years ago

https://github.com/twitter/ostrich is currently used to maintain internal Stats in Cassovary. Ostrich does not seem to be actively maintained. Instead, should move to https://github.com/twitter/commons/tree/master/src/java/com/twitter/common/metrics as is used in Finagle-stats (http://twitter.github.io/finagle/guide/Metrics.html) as well.

AnishShah commented 9 years ago

@pankajgupta I would like to work on this! I will send a PR. If I'm having some problems, should I post here? or is there any IRC channel?

pankajgupta commented 9 years ago

Sure -- please go ahead

On Sun, Jan 11, 2015 at 5:31 AM, Anish Shah notifications@github.com wrote:

@pankajgupta https://github.com/pankajgupta I would like to work on this!

— Reply to this email directly or view it on GitHub https://github.com/twitter/cassovary/issues/89#issuecomment-69494551.

AnishShah commented 9 years ago

Hi, so I have started working on this.. We will need Twitter-Server integrated into CassoveryServer.scala instead of current Ostrict AdminHttpService.

When I added "com.twitter" % "twitter-server_2.10" % "1.9.0" to Build.scala , I'm getting the following error. I tried a lot, but I'm not able to resolve this. Can anyone help? @pankajgupta @szymonm 

[error] Modules were resolved with conflicting cross-version suffixes in {file:/home/anish/ClonedProjects/cassovary/}cassovary:
[error]    com.twitter:util-jvm _2.9.2, _2.10
[error]    com.twitter:util-core _2.9.2, _2.10
[error]    com.twitter:util-logging _2.9.2, _2.10
[error]    com.twitter:util-app _2.9.2, _2.10
[error] Modules were resolved with conflicting cross-version suffixes in {file:/home/anish/ClonedProjects/cassovary/}cassovary-core:
[error]    com.twitter:util-jvm _2.9.2, _2.10
[error]    com.twitter:util-core _2.9.2, _2.10
[error]    com.twitter:util-logging _2.9.2, _2.10
[error]    com.twitter:util-app _2.9.2, _2.10
java.lang.RuntimeException: Conflicting cross-version suffixes in: com.twitter:util-jvm, com.twitter:util-core, com.twitter:util-logging, com.twitter:util-app```
szymonm commented 9 years ago

Can you push a PR? I can lok at it W dniu niedz., 18 sty 2015 o 17:14 Anish Shah notifications@github.com napisał(a):

Hi, so I have started working on this.. We will need Twitter-Server https://twitter.github.io/twitter-server/Migration.html integrated into CassoveryServer.scala instead of current Ostrict AdminHttpService.

When I added "com.twitter" % "twitter-server_2.10" % "1.9.0" to Build.scala , I'm getting the following error. I tried a lot, but I'm not able to resolve this. Can anyone help? @pankajgupta https://github.com/pankajgupta @szymonm https://github.com/szymonm

[error] Modules were resolved with conflicting cross-version suffixes in {file:/home/anish/ClonedProjects/cassovary/}cassovary: [error] com.twitter:util-jvm _2.9.2, _2.10 [error] com.twitter:util-core _2.9.2, _2.10 [error] com.twitter:util-logging _2.9.2, _2.10 [error] com.twitter:util-app _2.9.2, _2.10 [error] Modules were resolved with conflicting cross-version suffixes in {file:/home/anish/ClonedProjects/cassovary/}cassovary-core: [error] com.twitter:util-jvm _2.9.2, _2.10 [error] com.twitter:util-core _2.9.2, _2.10 [error] com.twitter:util-logging _2.9.2, _2.10 [error] com.twitter:util-app _2.9.2, _2.10 java.lang.RuntimeException: Conflicting cross-version suffixes in: com.twitter:util-jvm, com.twitter:util-core, com.twitter:util-logging, com.twitter:util-app```

— Reply to this email directly or view it on GitHub https://github.com/twitter/cassovary/issues/89#issuecomment-70414035.

AnishShah commented 9 years ago

Should we close this now?

caniszczyk commented 9 years ago

Yes IMHO