cgivre commented on PR #2705: URL: https://github.com/apache/drill/pull/2705#issuecomment-1313908517
@jnturton The PR looks good. I had a few questions: 1. Could you please document what types of input would be expected in these fields and what they do? I'm not really clear myself what the `owner` field actually does. If you're not sure, then let's leave it, but if you had some additional info, that would be helpful. 2. I had question about the `token` and `cookie`? Are these fields something that we'd expect to be set in a config, or are they something that should be somehow populated dynamically? This seemed strange to me. -- 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
