Return-Path: Delivered-To: apmail-lucene-java-dev-archive@www.apache.org Received: (qmail 84573 invoked from network); 22 Feb 2007 22:10:12 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 22 Feb 2007 22:10:12 -0000 Received: (qmail 14286 invoked by uid 500); 22 Feb 2007 22:10:15 -0000 Delivered-To: apmail-lucene-java-dev-archive@lucene.apache.org Received: (qmail 14247 invoked by uid 500); 22 Feb 2007 22:10:15 -0000 Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: java-dev@lucene.apache.org Delivered-To: mailing list java-dev@lucene.apache.org Received: (qmail 14236 invoked by uid 99); 22 Feb 2007 22:10:15 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Feb 2007 14:10:15 -0800 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (herse.apache.org: local policy) Received: from [209.86.89.66] (HELO elasmtp-spurfowl.atl.sa.earthlink.net) (209.86.89.66) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 22 Feb 2007 14:10:02 -0800 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=dk20050327; d=ix.netcom.com; b=rd+aTs4apzdDccjaLCYNZiuiRVK8RJiTM+cp1Co3/nw1YZX9qnMbZAdo3xX+Ew6Y; h=Received:Mime-Version:In-Reply-To:References:Content-Type:Message-Id:Content-Transfer-Encoding:From:Subject:Date:To:X-Mailer:X-ELNK-Trace:X-Originating-IP; Received: from [68.255.105.15] (helo=[192.168.1.64]) by elasmtp-spurfowl.atl.sa.earthlink.net with asmtp (Exim 4.34) id 1HKM8D-0002Zf-04 for java-dev@lucene.apache.org; Thu, 22 Feb 2007 17:09:41 -0500 Mime-Version: 1.0 (Apple Message framework v752.3) In-Reply-To: <0D75F53E-D6E4-4842-AC45-C6CFE843D01E@ix.netcom.com> References: <18044975.1172181365878.JavaMail.jira@brutus> <0D75F53E-D6E4-4842-AC45-C6CFE843D01E@ix.netcom.com> Content-Type: text/plain; charset=US-ASCII; delsp=yes; format=flowed Message-Id: <3EEFB0EC-284C-414A-8759-1738924627E2@ix.netcom.com> Content-Transfer-Encoding: 7bit From: robert engels Subject: Re: [jira] Updated: (LUCENE-811) Public API inconsistency Date: Thu, 22 Feb 2007 16:09:40 -0600 To: java-dev@lucene.apache.org X-Mailer: Apple Mail (2.752.3) X-ELNK-Trace: 33cbdd8ed9881ca8776432462e451d7bd15d05d9470ff710500183ea7c8e1c9b20b25a61f29f3478350badd9bab72f9c350badd9bab72f9c350badd9bab72f9c X-Originating-IP: 68.255.105.15 X-Virus-Checked: Checked by ClamAV on apache.org Actually, scratch that request. As long as they are package level, I can put the class in the same package. Problem was that in the past some important states fields have been private. On Feb 22, 2007, at 4:04 PM, robert engels wrote: > I think the getters need to be public, as it makes many of the > 'cache by segment' code much cleaner. Currently I cannot subclass > and must replace the class in the jar because I need access to much > o this information. > > Is there some reason not to make the getters be public, (or copy on > read public)? > > On Feb 22, 2007, at 3:56 PM, Michael McCandless (JIRA) wrote: > >> >> [ https://issues.apache.org/jira/browse/LUCENE-811? >> page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] >> >> Michael McCandless updated LUCENE-811: >> -------------------------------------- >> >> Attachment: LUCENE-811.patch >> >> Attaching proposed patch to change SegmentInfos (and all of its >> methods/fields) back to package level protection. >> >> I also found a few other cases (IndexFileNames, >> IndexFileNamesFilter, SegmentInfo) that I had added public methods >> which should be kept at package protection. >> >>> Public API inconsistency >>> ------------------------ >>> >>> Key: LUCENE-811 >>> URL: https://issues.apache.org/jira/browse/ >>> LUCENE-811 >>> Project: Lucene - Java >>> Issue Type: Bug >>> Components: Index >>> Affects Versions: 2.1 >>> Environment: 2.1.0 release >>> Reporter: Andrzej Bialecki >>> Assigned To: Michael McCandless >>> Priority: Minor >>> Attachments: LUCENE-811.patch >>> >>> >>> org.apache.lucene.index.SegmentInfos is public, and contains >>> public methods (which is good for expert-level index manipulation >>> tools such as Luke). However, SegmentInfo class has package >>> visibility. This leads to a strange result that it's possible to >>> read SegmentInfos, but it's not possible to access its details >>> (SegmentInfos.info(int)) from a user application. >>> The solution is to make SegmentInfo class public. >> >> -- >> This message is automatically generated by JIRA. >> - >> You can reply to this email to add a comment to the issue online. >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org >> For additional commands, e-mail: java-dev-help@lucene.apache.org >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org > For additional commands, e-mail: java-dev-help@lucene.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org For additional commands, e-mail: java-dev-help@lucene.apache.org