E.J. BLOG

Archive for the ‘WTH’ Category

[WTH] critical flaw in our review process

leave a comment

Image Credit

code_review

Let me tell you a story

Recently, I have been assigned as a reviewer for a patch provided by our partner and this is okay to me because as a peer of Settings app, it’s part of my duty to review these patches. But … sadly, there are so many problems in that patch and I thought “Ok, it’s fine, because we all would make mistakes at first especially if you are not familiar with the part of codes”, so I tried my best to comment down all problematic codes with references (like to MDN or some related codes) to make sure they  can get familiar with them more quickly and easily.

After several rounds of this process, we finally made it and I thought this patch was good enough to be landed, so I gave them r+ on this patch.

If the story ended here, then I don’t have to write any bits of words like this article. As you may know, this is not the end.

What happened next

After signing-off this patch, there are still some other reviewing processes undergoing for different part of codes. For me, I have done my works, so I just left them alone and kept focusing on new features. Someday in the morning, I randomly checked the messages on Github and noticed there was one comment about the r+ patch, just out of curiosity, I decided to click it and checked it out.

That was a comment about missing change on the entry point and this definitely broke Settings app (You can’t even do anything). But … I was sure that this did work when I gave r+ because I did try the entry point to jump to that specific app when reviewing. So what’s going on right now !?

After a while, I finally realized that they just changed the code without any further notification and because the code change is kinda huge, no one would realize this problem when reviewing patch. So what does this mean ? If you really want to set someone up in this review process, you just have to get his/her r+ on your patch and use some magical git commands to break it and NO ONE WOULD NOTICE THIS AND WOULD BLAME ON THE REVIEWER !! (I wasn’t blamed yet, but if this patch got merged and I didn’t notice this, I would be the one)

Final

We all know this is the critical flaw in our review process, but in order to trust anyone, we haven’t forced people to tell us if the patch got changed after r+. By doing so, this would make the review process more easy and we don’t have to get stuck in some kinds of principles. Sadly, this happened to me few days ago.

Not sure what to fix here because if you are familiar with git, you can do whatever you want on commits. It is a double-edged sword which provides you so many advantages and also some shortages. I just hope this was not made on purpose …

SUCK.

Written by EragonJ

October 27th, 2014 at 6:05 pm

Posted in WTH

Tagged with , ,

[WTH] I hate Influenza

2 comments

Image Credit

influenza

上上個星期五的中午,和同事吃完午餐回到公司後,就開始瘋狂覺得莫名的冷,就算連切換到 Mozilla Space 還是覺得很冷,我原本一直以為是空調的問題,但是直到我一直發抖到有點無法控制之後才覺得有點不太對,只能立馬請假去看醫生,而在離開前同事很強的不知道去哪邊弄到一台體溫槍,不量還好,一量就 38.6 度 … 只能趕快去找計程車回家了。

一開始還很慘的跑到一間已經關門的診所,當時真的不知道我是如何在大太陽下走到遠端的另一間診所,然後還等了快 20 分鐘看醫生,明明就沒什麼人而且看起來也沒什麼問題也是在這邊排隊,我當時整個就是靠在牆上呈現一個快死亡的狀態。

好不容易輪到我了,快速給醫生診斷之後就量了一下體溫,此時已經來到 40 度了,當時心想人生的尾端大概也不過如此吧。在拿了醫生的爛藥之後(後面有去給別的醫生看,只是退燒藥而已),趕快衝回家昏迷,就這樣整整睡了一晚,還好室友回來還有順路帶個水及簡單的食物,雖然沒什麼食慾但至少還有一種活著的感覺。

隔天,全身莫名的痠痛(流感典型症狀)然後都是汗,覺得這下去不行就跑去沖澡,但是洗到一半整個人就快昏倒,只能用最快的速度衝了出來(還來不及穿東西),就在這個當下我短暫的失去了幾秒的記憶,等到我有意識的時候是倒在地板上了,我連我怎麼跌倒都不知道,這真的非常嚴重,只能趕快爬到房間打電話給隔壁的室友來救命。我還記得當時的心跳真的超快,有種會不會就這樣和這個世界告別的感覺。

人生第一次的短暫無意識就這樣送給了我第一次得的流感。

之後就是一連串的睡覺還有休養,整個週末都在床上渡過,只能感謝家人還有室友、好友才能好好的活著,雖然大家都一直叫我要小心、注意,但是我到現在還是對於為什麼我吃完午餐就得到流感這件事情沒有頭緒,也許很多時候事情都沒有什麼頭緒可言吧。

請大家也多注意身體,夏天好像反而是流感的高峰期,別想要體會短暫無意識的感覺,因為,

真的,很糟。

Written by EragonJ

July 14th, 2014 at 11:31 pm

Posted in WTH

Tagged with , , ,

[WTH] The blog is back

leave a comment

Image Credit

 

最近因為 Server 有點問題所以造成 Blog 的 Routing 出了點問題,也是因為這樣才會使得 feedburner 抓不到資料。

儘管如此,最終還是修好了。

 

而我,也有幸在台灣某大資安 T 公司工作,為這個世界付出一些貢獻。

加油 😛

Written by EragonJ

October 1st, 2012 at 9:26 pm

Posted in WTH

Tagged with ,