> On May 15, 2015, 3:18 a.m., Amareshwari Sriramadasu wrote: > > lens-server/src/main/java/org/apache/lens/server/InitialRequestFilter.java, > > line 52 > > <https://reviews.apache.org/r/34210/diff/2/?file=959328#file959328line52> > > > > Will this be available for all threads, not just the thread which > > served the request? I think its not. > > > > If not, estimate api involves different threads from thread pool. > > Himanshu Gahlaut wrote: > It will be available for all http server threads and child threads of > http server threads. Will add query handle for estimate threads. > > Amareshwari Sriramadasu wrote: > Estimate and explain may not have query handle. Only if query is accepted > for execution, query handle is generated. We might have to go with requestid > itself for simple estimate/explain api. > > Himanshu Gahlaut wrote: > QuerySubmitter also makes use of background threads of estimate pool to > do estimate operations and it doesn't have hold of request id. Will it be > appropriate to use query handle for estimates triggered by QuerySubmitter and > request id for estimates triggered by QueryServiceResource ?
>> Will it be appropriate to use query handle for estimates triggered by >> QuerySubmitter and request id for estimates triggered by >> QueryServiceResource ? Sounds OK to me. Could not think of anyother option. - Amareshwari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34210/#review83890 ----------------------------------------------------------- On May 14, 2015, 2:06 p.m., Himanshu Gahlaut wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34210/ > ----------------------------------------------------------- > > (Updated May 14, 2015, 2:06 p.m.) > > > Review request for lens. > > > Bugs: LENS-514 > https://issues.apache.org/jira/browse/LENS-514 > > > Repository: lens > > > Description > ------- > > LENS-514: Adding unique identifier to log lines for log segregation > > > Diffs > ----- > > lens-api/src/main/java/org/apache/lens/api/query/QueryHandle.java > 7b615e6216d7a31a743416dd839a63517be88e1f > lens-api/src/main/java/org/apache/lens/api/query/QueryPrepareHandle.java > efa04f1c7e7fddbd9176922425c66057f4be3089 > lens-api/src/main/java/org/apache/lens/api/response/LensResponse.java > f6c3593a131ab6779186be014d79b9f4c4cb9534 > > lens-server-api/src/main/java/org/apache/lens/server/api/common/Constant.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/PreparedQueryContext.java > 1ce89ac66fa817b3e47482a81b040259d2194477 > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java > 169ac8dec4b98a0cab1f652fe85afe9737f02e39 > lens-server/src/main/java/org/apache/lens/server/AuthenticationFilter.java > b64d8223727d755aad95dee288f9561b767a5448 > lens-server/src/main/java/org/apache/lens/server/InitialRequestFilter.java > PRE-CREATION > lens-server/src/main/java/org/apache/lens/server/LensApplication.java > cb452e83e6edbe4f13a3f29b4ce0cbe45effcde3 > lens-server/src/main/java/org/apache/lens/server/LensServer.java > c6d7ea154f4c3f78eaa0675d99364a311351bf4c > lens-server/src/main/java/org/apache/lens/server/query/QueryApp.java > 0d23726a895a11208249b7ce458cb34e2265feeb > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > 3bf180c9ab6f2a9fec6259848ebf65ecd147cfcf > > lens-server/src/main/java/org/apache/lens/server/query/QueryServiceResource.java > 9b6d6bc223069b28367d789db847cbf05ed386c7 > > lens-server/src/test/java/org/apache/lens/server/query/QueryAPIErrorResponseTest.java > 5e135be0348068125b69016274e00b01140c4f21 > lens-server/src/test/resources/log4j.properties > 65d065fa699f9797dcb13580359c4d41cfc52e9d > tools/conf-pseudo-distr/server/log4j.properties > afadc2f2856e7723e23265553440984bf2859869 > tools/conf/server/log4j.properties afadc2f2856e7723e23265553440984bf2859869 > > Diff: https://reviews.apache.org/r/34210/diff/ > > > Testing > ------- > > Testing done by executing query from lens-cli and verifying lens-server logs. > > Running test suite. Will update test results. > > > Thanks, > > Himanshu Gahlaut > >
