<div dir="ltr">Hi, Alexey<div><br></div><div>I plan to split the implementation to several steps, because it will take weeks to complete. I'm afraid it would be too big a patch to review if I submit all changes in one patch set.</div><div><br></div><div>Instead I want to get comments as earlier as possible. Each submit will be covered by additional unit test and keep backward compatibility.</div><div><br></div><div>Detail replies inline.</div><div><br></div><div><div class="gmail_quote"><div dir="ltr">On Tue, Nov 15, 2016 at 5:51 PM Weyl, Alexey (Nokia - IL) <<a href="mailto:alexey.weyl@nokia.com">alexey.weyl@nokia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Yujun,<br class="gmail_msg">
<br class="gmail_msg">
Good job! This is a very important change for Vitrage.<br class="gmail_msg">
<br class="gmail_msg">
I have a couple of questions please:<br class="gmail_msg">
1. Why do we want to create a new datasource ‘static’ and not rename the current ‘static_physical’ datasource and change it to work with the new format?<br class="gmail_msg"></blockquote><div>I don't want to break it during the evolution.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2. How are you planning to use the old 'static_physical' datasource  as a proxy if you said that you want to remove it?<br class="gmail_msg"></blockquote><div>Good point. Any suggestion on how we hide the deprecated modules? Move it as a submodule in new module.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
3. What kind of merge is needed in the evaluator?<br class="gmail_msg"></blockquote><div>Parsing of `definition` section would be a common module for both evaluator and static data source </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
4. I saw the implementation of the driver.py in static, and it doesn't do anything at the moment? (if you are still working on one of the patches then please market it as -1 in the code-review that we will know that you are still working on it.<br class="gmail_msg"></blockquote><div>Yes, skeleton is skeleton. Since I'm working remotely with vitrage team. I want to keep you updated on the progress. Still, it is complete with unit test and backward compatibility and will reduce further review work.</div></div></div></div>