-
Notifications
You must be signed in to change notification settings - Fork 211
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
Align naming convention for db.operation.batch.size
and messaging.batch.message_count
?
#1984
Comments
Hi @trask , I would like to contribute to the issue. This is my first time contributing to Otel so what would be a good starting point to approach this issue? |
Alternatives:
Also align with
try (Statement stmt = con.createStatement()) {
ResultSet rs = stmt.executeQuery(query);
while (rs.next()) {
String coffeeName = rs.getString("COF_NAME");
int supplierID = rs.getInt("SUP_ID");
float price = rs.getFloat("PRICE");
int sales = rs.getInt("SALES");
int total = rs.getInt("TOTAL");
System.out.println(coffeeName + ", " + supplierID + ", " + price +
", " + sales + ", " + total);
}
} need to document what's the boundary of sql span and whether it includes Might be a problem with |
One thing to consider: there is not an easy way to determine if a query returns multiple result sets. Common databases can return multiple results from a single stored procedure. May need to clarify whether the batch size attribute is for returned data, or if some batch detection is needed.
Aside: span per result set is an interesting thought exercise. Is there high level guidance for where to draw boundaries? |
Not (necessarily) suggesting we need a top-level
batch.*
namespace but wondering if it makes sense to align the naming pattern at least.Maybe the most straightforward solution would be to rename
messaging.batch.message_count
tomessaging.operation.batch.size
.The text was updated successfully, but these errors were encountered: