<br><br><div class="gmail_quote">On Fri, Jul 29, 2011 at 5:50 PM, Stephen Woodbridge <span dir="ltr"><<a href="mailto:woodbri@swoodbridge.com">woodbri@swoodbridge.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 7/29/2011 5:49 AM, Jay Mahadeokar wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi all,<br>
<br>
I was looking at ways to optimise the tdsp implementation.<br>
<br>
Currently, if the time-interval is suppose 24, (24 hours) and the travel<br>
time expected is around 60 hours, then the tdsp-wrapper query retrieves<br>
total 60 intervals and the data is repeated for the 24 hour intervals.<br>
(I hope what i am trying to say is clear).<br>
<br>
This is unnecessary space wastage. So, I edited the weight_map class to<br>
accommodate a variables is_cyclic and the cycle_interval, which will<br>
keep track of the cyclic nature of the data. So, actually only the 24<br>
hour data will be fed to weight map and then it will be reused in cycles.<br>
<br>
Because of the good design, I guess this has scaled up really nice and<br>
there is no need to modify actual core tdsp. Just the weight_map's<br>
get_travel_time() function and the wrapper plsql function needed to be<br>
altered.<br>
</blockquote>
<br></div>
This makes perfect sense and sounds like a great optimization.<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Now, in the main query, I am thinking of adding one more boolean<br>
variable - is_cyclic which will be enabled if the data is cyclic in<br>
nature. Should we keep this? Or we should assume that the data will be<br>
always cyclic.<br>
Any other views on the same are welcome.<br>
</blockquote>
<br></div>
I guess my questions are:<br>
How does this get set?<br>
What are the advantages of setting it?<br>
Is there a way to dynamically check if it is cyclic at the start and avoid having the app builder set it up?<br>
<br></blockquote><div><br>Advantages of setting it:<br>If the data is not cyclic, then the whole data will be expected to reside in database itself. So, if the upper bound of the time is reached then by default, we can assume that the travel_time corresponding to the last existing time window should be used for such cases.<br>
<br>If data is cyclic, then we need to start all over again. These are the cases, what I could think of.<br><br>How does this get set:<br>The tdsp query must have an additional parameter - is_cyclic which should be used to set the is_cyclic flag of the weight_map.<br>
<br>Now, if data is cyclic, then the plsql function that retrieves the data should be written in such a way that if interval is suppose 0 - 24, and start time is 21, then 21->0 and 20->23. After that the cyclic flag will indicate us to loop again. The plsql function will be generally written by app developer and he should take care of this. (I have already implemented this in my code.)<br>
<br>If data is cyclic, he should pass the flag appropriately. If is_cyclic is false, we can assume that if intervals are exhausted, we will keep using time corresponding to last interval.<br><br>Is this reasonable?<br><br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I suppose it is not a big deal for the app builder to set this flag since he will know if the data loaded is cyclic in nature. I assume you are only looking for some indication if there are cyclic entries in the data and not if this specific query has a cyclic nature which would be impossible to know without analyzing the start time and max time window, in which case we should figure that out automatically because the user making the request is not likely to know anything more than start, end and start time or end time.<br>
<br>
-Steve<br></blockquote></div>-- <br>Regards,<br>-Jay Mahadeokar<br><br>