Perl 6 - the future is here, just unevenly distributed

IRC log for #gluster-dev, 2015-12-14

| Channels | #gluster-dev index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:55 EinstCrazy joined #gluster-dev
00:58 zhangjn joined #gluster-dev
02:13 zhangjn joined #gluster-dev
02:35 kshlm joined #gluster-dev
03:26 bmikhael joined #gluster-dev
03:35 atinm joined #gluster-dev
03:47 vmallika joined #gluster-dev
03:48 nbalacha joined #gluster-dev
03:49 atinm joined #gluster-dev
03:52 aspandey joined #gluster-dev
03:59 itisravi joined #gluster-dev
04:03 jiffin joined #gluster-dev
04:09 kanagaraj joined #gluster-dev
04:26 shubhendu joined #gluster-dev
04:27 ppai joined #gluster-dev
04:35 zhangjn joined #gluster-dev
04:38 hagarth joined #gluster-dev
04:43 ndarshan joined #gluster-dev
04:55 Manikandan joined #gluster-dev
04:57 Manikandan joined #gluster-dev
04:58 hgowtham joined #gluster-dev
05:01 poornimag joined #gluster-dev
05:06 Humble joined #gluster-dev
05:14 ashiq joined #gluster-dev
05:14 ashiq_ joined #gluster-dev
05:16 overclk joined #gluster-dev
05:16 ashiq joined #gluster-dev
05:21 zhangjn joined #gluster-dev
05:23 mchangir joined #gluster-dev
05:24 Apeksha joined #gluster-dev
05:28 ppai joined #gluster-dev
05:28 pranithk joined #gluster-dev
05:43 nishanth joined #gluster-dev
05:44 ashiq joined #gluster-dev
05:44 ashiq_ joined #gluster-dev
05:44 ashiq joined #gluster-dev
05:48 pppp joined #gluster-dev
05:54 aspandey joined #gluster-dev
05:55 kdhananjay joined #gluster-dev
06:00 itisravi joined #gluster-dev
06:00 skoduri joined #gluster-dev
06:01 zhangjn joined #gluster-dev
06:41 zhangjn joined #gluster-dev
06:44 pranithk1 joined #gluster-dev
06:57 zhangjn joined #gluster-dev
06:58 rjoseph|afk joined #gluster-dev
06:58 zhangjn joined #gluster-dev
07:02 Saravana_ joined #gluster-dev
07:04 zhangjn joined #gluster-dev
07:28 rafi joined #gluster-dev
07:29 ppai joined #gluster-dev
07:43 msvbhat joined #gluster-dev
08:02 pranithk joined #gluster-dev
08:36 nbalacha joined #gluster-dev
08:41 spalai joined #gluster-dev
08:42 purpleidea joined #gluster-dev
08:42 purpleidea joined #gluster-dev
08:42 ppai joined #gluster-dev
09:00 ggarg joined #gluster-dev
09:03 zhangjn joined #gluster-dev
09:31 spalai1 joined #gluster-dev
10:01 zhangjn joined #gluster-dev
10:03 zhangjn joined #gluster-dev
10:06 pranithk joined #gluster-dev
10:08 josferna joined #gluster-dev
10:18 Ameet joined #gluster-dev
10:37 spalai joined #gluster-dev
10:46 spalai1 joined #gluster-dev
10:56 firemanxbr joined #gluster-dev
11:06 pppp joined #gluster-dev
11:10 badone joined #gluster-dev
11:10 spalai joined #gluster-dev
11:11 spalai joined #gluster-dev
11:13 zhangjn joined #gluster-dev
11:27 pppp joined #gluster-dev
11:44 EinstCrazy joined #gluster-dev
11:52 ppai joined #gluster-dev
12:06 hchiramm joined #gluster-dev
12:10 overclk joined #gluster-dev
12:18 ppai joined #gluster-dev
12:25 Saravana_ joined #gluster-dev
12:32 lpabon joined #gluster-dev
12:33 kanagaraj joined #gluster-dev
12:43 kdhananjay xavih++
12:43 glusterbot kdhananjay: xavih's karma is now 20
12:45 zhangjn joined #gluster-dev
13:09 rjoseph|afk joined #gluster-dev
13:11 spalai joined #gluster-dev
13:18 poornimag joined #gluster-dev
13:38 mchangir joined #gluster-dev
13:41 zhangjn joined #gluster-dev
13:45 zhangjn joined #gluster-dev
14:03 shaunm joined #gluster-dev
14:05 zhangjn joined #gluster-dev
14:13 kdhananjay joined #gluster-dev
14:13 jiffin joined #gluster-dev
14:16 ira joined #gluster-dev
14:22 ndevos dlambrig: a comment from a tiering expert on http://review.gluster.org/12960 would be appreciated (additional lookups for T-files after READDIRPLUS)
14:23 ndevos jiffin: sorry, but I really think that ^ is wrong :)
14:24 shyam joined #gluster-dev
14:28 Manikandan joined #gluster-dev
14:29 rjoseph joined #gluster-dev
14:33 dlambrig niels: 12960 is an important patch for us. It fixes ls on tiered volumes for NFS. Yes, will look.
14:34 ndevos dlambrig: yeah, I understand the issue it addresses, but I do not think this is the correct solution
14:49 dlambrig ndevos: what is the correct solution
14:50 kshlm joined #gluster-dev
14:50 ndevos dlambrig: imho, readdirp() should always return correctly filled directory entries, never T-files
14:51 ndevos dlambrig: there was a reason readdirp() was added, if the attributes of the entries are not needed, a plain readdir() should be used
14:52 zhangjn joined #gluster-dev
14:53 dlambrig ndevos: there is a long story behind this.. suggest to add your comments to the bug.
14:54 ndevos dlambrig: hmm, I added my comments to the review, because it is a NFS change, is that not sufficient?
14:54 dlambrig ndevos: yes, to the review.
14:54 dlambrig ndevos: thanks
14:55 ndevos dlambrig: sure, np
14:55 dlambrig left #gluster-dev
14:55 ndevos shyam: do you have an opinion about readdirp() returning T-files instead of the correct attributes?
14:55 shyam yes
14:55 shyam I do
14:55 shyam :)
14:56 ndevos maybe drop them in http://review.gluster.org/12960 ?
14:56 shyam ndevos: brb, in a call
14:56 ndevos shyam: sure!
14:56 shyam Yup already have that opened.... seeing this conversation.
14:56 ndevos ah, cool, thanks
14:56 jiffin ndevos: did dlambrig answer ur question?
14:56 rafi joined #gluster-dev
14:57 ndevos jiffin: no, he said "long story" but left after that :)
14:57 jiffin ndevos: I heard that long story before
14:58 jiffin :)
14:58 ndevos jiffin: thats nice, but it should be in the bug, and in the commit message :)
14:58 ndevos or a pointer to an email on gluster-devel, that would be fine too
14:58 jiffin it is very long story
14:59 ndevos in that case it is even more important to document it clearly
15:00 jiffin ndevos: http://review.gluster.org/#/c/12722/
15:00 jiffin this commit message may help u bit
15:02 ndevos jiffin: hmm, maybe, but I still think it is the responsibility of the xlator that uses T-files to not return them in a readdirp reply :)
15:03 ndevos and I do not really care how that is solved, but I do not like it when xlators break readdirp functionality
15:03 jiffin ndevos: me too, but under current circumstance, that change will
15:04 jiffin cost a lot of testing efforts again
15:04 ndevos jiffin: I do not see an issue with that, we should fix it the right way, not with hacks
15:05 jiffin ndevos: And I am not sure whether tier can identity readdir and readdirp seperately
15:06 ndevos jiffin: I think it should be able to, readdir() and readdirp() are different FOPs
15:07 zhangjn joined #gluster-dev
15:08 jiffin ndevos: In gluster-nfs, both readdir and readdirp uses nfs_readdirp()
15:11 ndevos jiffin: we could split that if it makes sense, but maybe we need it for some attributes that a NFS-readdir expects
15:11 jiffin ndevos: hmm
15:18 hagarth joined #gluster-dev
15:21 jiffin joined #gluster-dev
15:22 rafi joined #gluster-dev
15:31 rafi joined #gluster-dev
15:31 jiffin1 joined #gluster-dev
15:32 dlambrig1 joined #gluster-dev
15:40 spalai joined #gluster-dev
15:43 wushudoin joined #gluster-dev
15:46 spalai joined #gluster-dev
15:48 skoduri joined #gluster-dev
16:06 shaunm joined #gluster-dev
16:25 EinstCrazy joined #gluster-dev
16:27 shyam ndevos: Posted comments on the review discussed earlier
16:29 shyam ndevos: If you get a minute, check the same and let me know it it answers your questions (for the record I think the fix is right, but the decision making should change based on entry->inode and also by the sub-function)
16:37 rafi joined #gluster-dev
16:48 ggarg joined #gluster-dev
16:55 kanagaraj joined #gluster-dev
16:56 hgowtham joined #gluster-dev
16:59 shubhendu joined #gluster-dev
17:33 jiffin1 joined #gluster-dev
18:06 ira_ joined #gluster-dev
18:28 lpabon joined #gluster-dev
18:53 bmikhael joined #gluster-dev
18:53 bmikhael_ joined #gluster-dev
19:12 spalai joined #gluster-dev
19:15 spalai left #gluster-dev
19:18 EinstCrazy joined #gluster-dev
20:09 dlambrig1 left #gluster-dev
20:12 ira joined #gluster-dev
22:36 EinstCrazy joined #gluster-dev
23:56 ira joined #gluster-dev

| Channels | #gluster-dev index | Today | | Search | Google Search | Plain-Text | summary