kingswanwho commented on PR #2756: URL: https://github.com/apache/drill/pull/2756#issuecomment-1430637603
> > > One other question. Should we document this in the developer documentation? > > > > > > I think we do have developer documentation describing cluster fixture tests, or do you mean something else? > > I was referring to the markdown files in the `/docs` folder. With this PR do those need to be updated? (It doesn't have to be a part of this PR.) Hi Charles, I have checked /docs developer information, this PR transfer test framework from BaseTestQuery to ClusterTest, and doesn't change the test logic of ClusterTest. James helps to find a clean way to create new ClientFixture, and handle UserException. I can help to update those information in /docs in a new PR. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
