Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for cassandra #789

Closed
dawidmalina opened this issue Jul 30, 2015 · 8 comments
Closed

Add support for cassandra #789

dawidmalina opened this issue Jul 30, 2015 · 8 comments
Labels

Comments

@dawidmalina
Copy link
Contributor

It would be.nice to have dedicated profiler for cassandra same as we have for redis.

@youtome
Copy link
Contributor

youtome commented Jul 30, 2015

+1

@emeroad
Copy link
Member

emeroad commented Jul 30, 2015

We have a plan to support hbase and cassandra.

@dawidmalina
Copy link
Contributor Author

👍

@dawidmalina
Copy link
Contributor Author

Is there any updates for this?

@dawidmalina
Copy link
Contributor Author

I found this: https://github.com/dynaTrace/Dynatrace-Apache-Cassandra-Fastpack

Based on this I've created a list with classes and methods that (probably) should be traced. I hope it will help.

Class Name Method Name
org.apache.cassandra.service.AbstractWriteResponseHandler get
org.apache.cassandra.thrift.CassandraServer getSlice, state
org.apache.cassandra.db.compaction.CompactionManager doCompaction
org.apache.cassandra.thrift.Cassandra$Processor$ProcessFunction process
org.apache.cassandra.thrift.Cassandra$Processor process
org.apache.cassandra.service.StorageProxy mutate, performWrite, insertLocal, mutateCounter, applyCounterMutationOnLeader, applyCounterMutationOnCoordinator, read, scan
org.apache.cassandra.db.compaction.CompactionTask execute
org.apache.cassandra.db.compaction.CompactionManager$CompactionExecutorStatsCollector beginCompaction
me.prettyprint.cassandra.model.ExecutingKeyspace doExecute
org.apache.cassandra.thrift.Cassandra$Iface get, get_slice, get_count, multiget_slice, multiget_count, get_range_slices, get_indexed_slices, insert, remove, remove_counter, batch_mutate
com.datastax.tutorial.InsertRowsForColumnFamilies$ExecTask run

@Xylus
Copy link
Contributor

Xylus commented Dec 3, 2015

For what it's worth, the Thrift generated classes (at least the client side) should already be handled by the Thrift plugin

org.apache.cassandra.thrift.Cassandra$Iface (client)
org.apache.cassandra.thrift.Cassandra$Processor$ProcessFunction (server)
org.apache.cassandra.thrift.Cassandra$Processor (server)

A proper Cassandra support would be the way to go eventually, and this should be in the works (not 100% sure on this)

@dawidmalina
Copy link
Contributor Author

dawidmalina added a commit to dawidmalina/pinpoint that referenced this issue Jan 7, 2016
dawidmalina added a commit to dawidmalina/pinpoint that referenced this issue Jan 12, 2016
dawidmalina added a commit to dawidmalina/pinpoint that referenced this issue Feb 16, 2016
Xylus pushed a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
Xylus added a commit to Xylus/pinpoint that referenced this issue Mar 8, 2016
dawidmalina added a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
dawidmalina added a commit to dawidmalina/pinpoint that referenced this issue Mar 20, 2016
Xylus pushed a commit to Xylus/pinpoint that referenced this issue Mar 22, 2016
Xylus added a commit that referenced this issue Mar 23, 2016
Cassandra Datastax plugin records host address instead of host name.
Integration tests should check against the host address as well.
@Xylus
Copy link
Contributor

Xylus commented Mar 23, 2016

Merged via #1643
Possible future additions - bind variable tracing.

@Xylus Xylus closed this as completed Mar 23, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants