[ 
https://issues.apache.org/jira/browse/DRILL-8117?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17692075#comment-17692075
 ] 

ASF GitHub Bot commented on DRILL-8117:
---------------------------------------

jnturton commented on code in PR #2763:
URL: https://github.com/apache/drill/pull/2763#discussion_r1114070404


##########
docs/dev/ClusterFixture.md:
##########
@@ -125,6 +125,27 @@ In some cases, you may want to change an option in a test. 
Rather than writing o
 
 Again, you can pass a Java value which the test code will convert to a string, 
then will build the `ALTER SESSION` command.
 
+# Try-with-resource Style of Creating Single-use Client Fixtures.
+
+The benefit of Cluster Fixture framework is to define specific config for 
specific clusterFixture and clientFixture as needed flexibly.
+
+In some cases, clusterFixture has been initialized, and we need to create 
several different config clients for different test cases,

Review Comment:
   ```suggestion
   In some cases, a clusterFixture has been initialized and we need to create 
several different config clients for different test cases.
   ```





> Upgrade unit tests to the cluster fixture framework
> ---------------------------------------------------
>
>                 Key: DRILL-8117
>                 URL: https://issues.apache.org/jira/browse/DRILL-8117
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.20.1
>            Reporter: Jingchuan Hu
>            Assignee: James Turton
>            Priority: Major
>             Fix For: 1.21.0
>
>
> Upgrade various unit tests to the cluster fixture framework and replace other 
> instances of deprecated code usage.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to