It's certainly not bad very much at all, but there are a couple of things I'd like to mention.
From 01:41 to 02:03, you have an interesting chord progression that uses what's called a borrowed chord. This song is in A minor, but the tonic chord (the "root" or "home" chord) is "borrowed" from A major.
That means that the melody is gonna have to do some funky business with what scale it's in at different points in time, depending on what chord it's over. When the melody is being played over that A major chord, you have to make the melody in A major, which you did. But, over any other chord that's not borrowed, you have to act like you're in A minor again, which you didn't.
No worries, it's pretty technical, but it's just something to be wary of.
(I don't want to sound condescending at all, it's just that I don't know if you know music theory... xD)
Secondly, and more importantly, a song should never, ever, EVER, E V E R take a year to finish!
1) You run a very large risk of becoming overly attached to that song, and you'll find it hard to move on from it.
2) You won't want to make another song because it feels like it might take another year.
3) Making songs faster makes you improve faster. The amount of improvement is directly related to how many songs you make. Makes sense, right? So, if you make songs faster, you improve faster.
Just... don't catch perfection-itis. If you do get it, my best advice is to stop caring. Just put random crap on NG or something. Don't worry if it sounds good. Make it just to make it for a while. If you want to be more active in eradicating that disease of perfectionism, then you might want to consider adopting the "20 hour guillotine," where after working on your song for 20 hours total, you immediately export your song and call it completely finished. (There are VSTs that track how much time you've worked on a song.)
Still though, this is pretty nice. I like the style of it. You may want to smooth out your transitions a bit, but otherwise, I like it.