Github user goi commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/164#issuecomment-124088782
OK That's sounds right.
I will update the code and commit again.
Thanks,
Eran
On Thu, Jul 23, 2015 at 3:50 PM DuyHai DOAN <[email protected]>
wrote:
> I guess that a rule saying: *if you mix several magics in the same output
> message, only the last one will be taken into account* can be a good
> start for this PR.
>
> This way, whatever the users do, they'll always have a *consistent* and
> *predictable* behavior.
>
> To implement this, maybe in the for loop, for each matching magic, you may
> save:
>
> 1. the magic word
> 2. the index of this magic word
>
> And after the for loop, just take the magic with highest index and strip
> out all the text before.
>
> How does it sound ?
>
> â
> Reply to this email directly or view it on GitHub
>
<https://github.com/apache/incubator-zeppelin/pull/164#issuecomment-124084610>
> .
>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---