Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [flux-dev] Status update

Hey Fjodor,

don’t worry, all fine… I ran your code successfully on my machine, works nicely.
And yes, the integration into the Eclipse plugin was a goal. But my assumption was that we could keep the ability to run the file watcher standalone, too… :-)

Not a requirement to successfully complete your GSoC work, though… ;-)

Cheers,
-Martin




> Am 29.08.2016 um 18:49 schrieb Vershinin Fjodor <fjodor@xxxxxxxxxxxxx>:
> 
> No, i guess not.
> Currently old version of flux file watcher already works as stand-alone application.
> So the main idea to embedded flux file watcher approach into eclipse flux plugin.
> 
> 2016-08-29 19:28 GMT+03:00 Martin Lippert <mlippert@xxxxxxxxx>:
> Hey Fjodor,
> 
> I know, but I thought the goal was to have a standalone-runnable version, too? Wasn’t it?
> So that you can run it from outside of Eclipse, too...
> 
> Cheers,
> -Martin
> 
> 
> > Am 29.08.2016 um 18:25 schrieb Vershinin Fjodor <fjodor@xxxxxxxxxxxxx>:
> >
> > Actually it is embeded into flux plugin. It starts automatically from there.
> >
> >
> > On Mon, Aug 29, 2016, 7:16 PM Martin Lippert <mlippert@xxxxxxxxx> wrote:
> > Hey Fjodor,
> >
> > thanks for this contribution, I am looking though this at the moment… :-)
> > Where can I find the Java main class to start the watcher for a file system and how do I run it (what are the parameters) from the command line?
> >
> > Cheers,
> > -Martin
> >
> >
> >
> > > Am 23.08.2016 um 11:02 schrieb Vershinin Fjodor <fjodor@xxxxxxxxxxxxx>:
> > >
> > > Hi, Martin!
> > > I had prepared gist with final evolution document, you can have a look it here: https://gist.github.com/fjodorver/b5dcfe291bebaea8f8f76fe31220177d ,
> > > or here in google docs: https://docs.google.com/document/d/1uV9LrHkCLCF10YTaznFowoGpj9z7WQQMciPVRLl3am8
> > > It would be nice if you can give some comments on this.
> > >
> > > Thanks,
> > > Fjodor
> > >
> > > 2016-08-22 10:05 GMT+03:00 Vershinin Fjodor <fjodor@xxxxxxxxxxxxx>:
> > > Hi!
> > >
> > > I hope you are doing extensive testing with all your changes…
> > >
> > > Yes, sure, I test all my changes for regressions.
> > >
> > > Btw, I have some troubles with time stamp, which I get from file system. Namely, JDK  truncates filesystem time stamp up to seconds.
> > > For example, you can have a look here: http://stackoverflow.com/questions/24804618/get-file-mtime-with-millisecond-resolution-from-java
> > > There are fix in open-jdk8, however, this patch is still not applied to Oracle JDK.
> > > So, the main problem is some changes are out of sync, because after truncation some file system changes are traveling to the past.
> > > It can happen for example when file was deleted very fast.
> > > I have an idea how to apply quick-fix, namely truncate millisecond part in System.currentTimeMillis(), like here:
> > > https://github.com/fjodorver/flux/blob/2f5165fa0eac84c32dec9ffddfaa3f64a1a0533f/org.eclipse.flux.watcher/src/main/java/org/eclipse/flux/watcher/fs/JDKProjectWatchService.java#L268
> > > What do you think?
> > >
> > > Fjodor
> > >
> > >
> > >
> > > --
> > > Thanks,
> > > Fjodor
> > > _______________________________________________
> > > flux-dev mailing list
> > > flux-dev@xxxxxxxxxxx
> > > To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> > > https://dev.eclipse.org/mailman/listinfo/flux-dev
> >
> > _______________________________________________
> > flux-dev mailing list
> > flux-dev@xxxxxxxxxxx
> > To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> > https://dev.eclipse.org/mailman/listinfo/flux-dev
> > _______________________________________________
> > flux-dev mailing list
> > flux-dev@xxxxxxxxxxx
> > To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> > https://dev.eclipse.org/mailman/listinfo/flux-dev
> 
> _______________________________________________
> flux-dev mailing list
> flux-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/flux-dev
> 
> 
> 
> -- 
> Thanks,
> Fjodor
> _______________________________________________
> flux-dev mailing list
> flux-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/flux-dev



Back to the top