Drop unnecessary method

integration/e2e/compression_test.go

@@ -46,9 +46,9 @@ func TestCompression(t *testing.T) {
 func queryAndAssertTraceCompression(t *testing.T, client *tempoUtil.Client, info *tempoUtil.TraceInfo) {
 
 	// The received client will strip the header before we have a chance to inspect it, so just validate that the compressed client works as expected.
-	result, resp, err := client.QueryTraceWithResponse(info.HexID())
+	result, err := client.QueryTrace(info.HexID())
 	require.NoError(t, err)
-	require.NotNil(t, resp)
+	require.NotNil(t, result)
 
 	expected, err := info.ConstructTraceFromEpoch()
 	require.NoError(t, err)

pkg/util/client.go

@@ -109,19 +109,14 @@ func (c *Client) Search(tags string) (*tempopb.SearchResponse, error) {
 }
 
 func (c *Client) QueryTrace(id string) (*tempopb.Trace, error) {
-	m, _, err := c.QueryTraceWithResponse(id)
-	return m, err
-}
-
-func (c *Client) QueryTraceWithResponse(id string) (*tempopb.Trace, *http.Response, error) {
 	m := &tempopb.Trace{}
 	resp, err := c.getFor(c.BaseURL+QueryTraceEndpoint+"/"+id, m)
 	if err != nil {
 		if resp.StatusCode == http.StatusNotFound {
-			return nil, resp, ErrTraceNotFound
+			return nil, ErrTraceNotFound
 		}
-		return nil, resp, err
+		return nil, err
 	}
 
-	return m, resp, nil
+	return m, nil
 }