iamsanjay commented on code in PR #2764:
URL: https://github.com/apache/solr/pull/2764#discussion_r1867901890


##########
solr/modules/sql/src/test/org/apache/solr/handler/sql/TestSQLHandler.java:
##########


Review Comment:
   Yeah this one I commented because somewhere in the code they 
`InputStreamResponseParser` is being used and but didn't closed properly upon 
exception. I believe it was to create JsonTupleStream, need to check again. So 
to avoid the ObjectTracker error I commented it then. In the new commit, I 
uncomment it and pushed the changes. Now the `TestSQLHandler` should fail!



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to