Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Lucene: Java-Dev

remove FieldsEnum?

 

 

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded


rcmuir at gmail

Aug 11, 2012, 6:27 AM

Post #1 of 7 (143 views)
Permalink
remove FieldsEnum?

FieldsEnum seems kinda awkward, can we just have something like
Iterable<String> on Fields?

so the consuming code i think would be easier.

for (String field : fields) {
Terms terms = fields.terms(field);
...
}

I don't like that there are two ways to get the terms for a field
today, so there is duplicate logic or crazy
stuff to handle this Fields.terms(String) versus FieldsEnum.terms()

--
lucidimagination.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


lucene at mikemccandless

Aug 11, 2012, 6:29 AM

Post #2 of 7 (142 views)
Permalink
Re: remove FieldsEnum? [In reply to]

+1

Mike McCandless

http://blog.mikemccandless.com


On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail> wrote:
> FieldsEnum seems kinda awkward, can we just have something like
> Iterable<String> on Fields?
>
> so the consuming code i think would be easier.
>
> for (String field : fields) {
> Terms terms = fields.terms(field);
> ...
> }
>
> I don't like that there are two ways to get the terms for a field
> today, so there is duplicate logic or crazy
> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
>
> --
> lucidimagination.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> For additional commands, e-mail: dev-help [at] lucene
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


rcmuir at gmail

Aug 11, 2012, 6:53 PM

Post #3 of 7 (135 views)
Permalink
Re: remove FieldsEnum? [In reply to]

Here's a patch: http://pastebin.com/PQGSgBtT

On Sat, Aug 11, 2012 at 9:29 AM, Michael McCandless
<lucene [at] mikemccandless> wrote:
> +1
>
> Mike McCandless
>
> http://blog.mikemccandless.com
>
>
> On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail> wrote:
>> FieldsEnum seems kinda awkward, can we just have something like
>> Iterable<String> on Fields?
>>
>> so the consuming code i think would be easier.
>>
>> for (String field : fields) {
>> Terms terms = fields.terms(field);
>> ...
>> }
>>
>> I don't like that there are two ways to get the terms for a field
>> today, so there is duplicate logic or crazy
>> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
>>
>> --
>> lucidimagination.com
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>> For additional commands, e-mail: dev-help [at] lucene
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> For additional commands, e-mail: dev-help [at] lucene
>



--
lucidworks.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


serera at gmail

Aug 11, 2012, 10:13 PM

Post #4 of 7 (132 views)
Permalink
Re: remove FieldsEnum? [In reply to]

Is there a JIRA issue for it? The patch is large and the issue seems
important enough to be handled through JIRA?

Shai

On Sun, Aug 12, 2012 at 4:53 AM, Robert Muir <rcmuir [at] gmail> wrote:

> Here's a patch: http://pastebin.com/PQGSgBtT
>
> On Sat, Aug 11, 2012 at 9:29 AM, Michael McCandless
> <lucene [at] mikemccandless> wrote:
> > +1
> >
> > Mike McCandless
> >
> > http://blog.mikemccandless.com
> >
> >
> > On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail> wrote:
> >> FieldsEnum seems kinda awkward, can we just have something like
> >> Iterable<String> on Fields?
> >>
> >> so the consuming code i think would be easier.
> >>
> >> for (String field : fields) {
> >> Terms terms = fields.terms(field);
> >> ...
> >> }
> >>
> >> I don't like that there are two ways to get the terms for a field
> >> today, so there is duplicate logic or crazy
> >> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
> >>
> >> --
> >> lucidimagination.com
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> >> For additional commands, e-mail: dev-help [at] lucene
> >>
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> > For additional commands, e-mail: dev-help [at] lucene
> >
>
>
>
> --
> lucidworks.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> For additional commands, e-mail: dev-help [at] lucene
>
>


rcmuir at gmail

Aug 11, 2012, 10:32 PM

Post #5 of 7 (132 views)
Permalink
Re: remove FieldsEnum? [In reply to]

JIRA has been down for a while now. We can't just stop development
because of this: personally I will just continue working and sending
patches to the list for review.

If someone wants, they can create JIRAs for these things after the
fact: but I'm not letting it get in my way.

On Sun, Aug 12, 2012 at 1:13 AM, Shai Erera <serera [at] gmail> wrote:
> Is there a JIRA issue for it? The patch is large and the issue seems
> important enough to be handled through JIRA?
>
> Shai
>
>
> On Sun, Aug 12, 2012 at 4:53 AM, Robert Muir <rcmuir [at] gmail> wrote:
>>
>> Here's a patch: http://pastebin.com/PQGSgBtT
>>
>> On Sat, Aug 11, 2012 at 9:29 AM, Michael McCandless
>> <lucene [at] mikemccandless> wrote:
>> > +1
>> >
>> > Mike McCandless
>> >
>> > http://blog.mikemccandless.com
>> >
>> >
>> > On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail> wrote:
>> >> FieldsEnum seems kinda awkward, can we just have something like
>> >> Iterable<String> on Fields?
>> >>
>> >> so the consuming code i think would be easier.
>> >>
>> >> for (String field : fields) {
>> >> Terms terms = fields.terms(field);
>> >> ...
>> >> }
>> >>
>> >> I don't like that there are two ways to get the terms for a field
>> >> today, so there is duplicate logic or crazy
>> >> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
>> >>
>> >> --
>> >> lucidimagination.com
>> >>
>> >> ---------------------------------------------------------------------
>> >> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>> >> For additional commands, e-mail: dev-help [at] lucene
>> >>
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>> > For additional commands, e-mail: dev-help [at] lucene
>> >
>>
>>
>>
>> --
>> lucidworks.com
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>> For additional commands, e-mail: dev-help [at] lucene
>>
>



--
lucidimagination.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene


serera at gmail

Aug 11, 2012, 10:37 PM

Post #6 of 7 (133 views)
Permalink
Re: remove FieldsEnum? [In reply to]

Ah I see.

Shai

On Sun, Aug 12, 2012 at 8:32 AM, Robert Muir <rcmuir [at] gmail> wrote:

> JIRA has been down for a while now. We can't just stop development
> because of this: personally I will just continue working and sending
> patches to the list for review.
>
> If someone wants, they can create JIRAs for these things after the
> fact: but I'm not letting it get in my way.
>
> On Sun, Aug 12, 2012 at 1:13 AM, Shai Erera <serera [at] gmail> wrote:
> > Is there a JIRA issue for it? The patch is large and the issue seems
> > important enough to be handled through JIRA?
> >
> > Shai
> >
> >
> > On Sun, Aug 12, 2012 at 4:53 AM, Robert Muir <rcmuir [at] gmail> wrote:
> >>
> >> Here's a patch: http://pastebin.com/PQGSgBtT
> >>
> >> On Sat, Aug 11, 2012 at 9:29 AM, Michael McCandless
> >> <lucene [at] mikemccandless> wrote:
> >> > +1
> >> >
> >> > Mike McCandless
> >> >
> >> > http://blog.mikemccandless.com
> >> >
> >> >
> >> > On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail>
> wrote:
> >> >> FieldsEnum seems kinda awkward, can we just have something like
> >> >> Iterable<String> on Fields?
> >> >>
> >> >> so the consuming code i think would be easier.
> >> >>
> >> >> for (String field : fields) {
> >> >> Terms terms = fields.terms(field);
> >> >> ...
> >> >> }
> >> >>
> >> >> I don't like that there are two ways to get the terms for a field
> >> >> today, so there is duplicate logic or crazy
> >> >> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
> >> >>
> >> >> --
> >> >> lucidimagination.com
> >> >>
> >> >> ---------------------------------------------------------------------
> >> >> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> >> >> For additional commands, e-mail: dev-help [at] lucene
> >> >>
> >> >
> >> > ---------------------------------------------------------------------
> >> > To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> >> > For additional commands, e-mail: dev-help [at] lucene
> >> >
> >>
> >>
> >>
> >> --
> >> lucidworks.com
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> >> For additional commands, e-mail: dev-help [at] lucene
> >>
> >
>
>
>
> --
> lucidimagination.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> For additional commands, e-mail: dev-help [at] lucene
>
>


lucene at mikemccandless

Aug 12, 2012, 11:29 AM

Post #7 of 7 (135 views)
Permalink
Re: remove FieldsEnum? [In reply to]

+1, another great cleanup!

Mike McCandless

http://blog.mikemccandless.com

On Sat, Aug 11, 2012 at 9:53 PM, Robert Muir <rcmuir [at] gmail> wrote:
> Here's a patch: http://pastebin.com/PQGSgBtT
>
> On Sat, Aug 11, 2012 at 9:29 AM, Michael McCandless
> <lucene [at] mikemccandless> wrote:
>> +1
>>
>> Mike McCandless
>>
>> http://blog.mikemccandless.com
>>
>>
>> On Sat, Aug 11, 2012 at 9:27 AM, Robert Muir <rcmuir [at] gmail> wrote:
>>> FieldsEnum seems kinda awkward, can we just have something like
>>> Iterable<String> on Fields?
>>>
>>> so the consuming code i think would be easier.
>>>
>>> for (String field : fields) {
>>> Terms terms = fields.terms(field);
>>> ...
>>> }
>>>
>>> I don't like that there are two ways to get the terms for a field
>>> today, so there is duplicate logic or crazy
>>> stuff to handle this Fields.terms(String) versus FieldsEnum.terms()
>>>
>>> --
>>> lucidimagination.com
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>>> For additional commands, e-mail: dev-help [at] lucene
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
>> For additional commands, e-mail: dev-help [at] lucene
>>
>
>
>
> --
> lucidworks.com
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe [at] lucene
> For additional commands, e-mail: dev-help [at] lucene
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe [at] lucene
For additional commands, e-mail: dev-help [at] lucene

Lucene java-dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.