[jira] [Commented] (CODEC-134) Base32 would decode some invalid Base32 encoded string into arbitrary value

2019-06-16 Thread Pavan Raju (JIRA)


[ 
https://issues.apache.org/jira/browse/CODEC-134?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865221#comment-16865221
 ] 

Pavan Raju commented on CODEC-134:
--

Hey folks,

Thanks for the fix above! Much appreciated.

Is there a plan for a release coming up soon? We're also looking at using the 
1.13 release.

Thanks,

Pavan

> Base32 would decode some invalid Base32 encoded string into arbitrary value
> ---
>
> Key: CODEC-134
> URL: https://issues.apache.org/jira/browse/CODEC-134
> Project: Commons Codec
>  Issue Type: Bug
>Affects Versions: 1.6
> Environment: All
>Reporter: Hanson Char
>Assignee: Gary Gregory
>Priority: Major
>  Labels: security
> Fix For: 1.13
>
> Attachments: diff-120305-20.txt
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Example, there is no byte array value that can be encoded into the string 
> "C5CYMIHWQUUZMKUGZHGEOSJSQDE4L===", but the existing Base32 implementation 
> would not reject it but decode it into an arbitrary value which if re-encoded 
> again using the same implementation would result in the string 
> "C5CYMIHWQUUZMKUGZHGEOSJSQDE4K===".
> Instead of blindly decoding the invalid string, the Base32 codec should 
> reject it (eg by throwing IlleglArgumentException) to avoid security 
> exploitation (such as tunneling additional information via seemingly valid 
> base 32 strings).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-vfs] garydgregory commented on issue #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
garydgregory commented on issue #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65#issuecomment-502495691
 
 
   Fixed in `changes.xml`, thanks @DavidSeptimus.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Closed] (VFS-698) SFTP file attributes are fetched multiple times leading to very slow directory listing

2019-06-16 Thread Gary Gregory (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-698?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary Gregory closed VFS-698.

   Resolution: Fixed
Fix Version/s: 2.4

> SFTP file attributes are fetched multiple times leading to very slow 
> directory listing
> --
>
> Key: VFS-698
> URL: https://issues.apache.org/jira/browse/VFS-698
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: krishnan
>Priority: Major
> Fix For: 2.4
>
>
> getChildren() applied on SftpFileObject is very slow compared to JSCH 
> implementation. This is because, the SftpATTRS which is readily available for 
> the children after an "ls" call is again fetched for each child file since 
> they are independently resolved. So if a directory contains 10 files, it 
> results in 1 (ls) + 10 (stat) calls to server.
> For a folder with 100 files (AWS), it took about 35 secs instead of 1.5 secs 
> to  getChildren().
>  
> *doListChildrenResolved:*
> {{final FileObject fo = 
> getFileSystem().resolveFile(getFileSystem().getFileSystemManager()}}
>  \{{ .resolveName(getName(), UriParser.encode(name), NameScope.CHILD));}}
> {{{color:#ff}((SftpFileObject) 
> FileObjectUtils.getAbstractFileObject(fo)).setStat(stat.getAttrs());{color}}}
>  
> The resolveFile call, creates a SftpFileObject and calls its resolve method, 
> which results in getting the (stats) SftpATTRS for each child file. This stat 
> is already available as part of the 'ls' call we made. The setStat call above 
> (highlighted is red) is redundant, since stat for each child file is already 
> fetched one at a time.
> The solution would be to avoid getting the stat for each child file after an 
> 'ls' call. May be, the framework makes it difficult to do this easily.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-vfs] DavidSeptimus commented on issue #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
DavidSeptimus commented on issue #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65#issuecomment-502493920
 
 
   Ah, I just realized that I referenced the wrong Jira story. It should have 
been VFS-698.
   
   https://issues.apache.org/jira/browse/VFS-698


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Resolved] (NET-670) Apache Commons Net ftpClient.java changeWorkingDirectory() Function CRLF Injection Remote Command Execution

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/NET-670?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved NET-670.
--
Resolution: Not A Bug

> Apache Commons Net ftpClient.java changeWorkingDirectory() Function CRLF 
> Injection Remote Command Execution
> ---
>
> Key: NET-670
> URL: https://issues.apache.org/jira/browse/NET-670
> Project: Commons Net
>  Issue Type: Bug
>  Components: FTP
>Affects Versions: 3.6
>Reporter: David Hurst
>Priority: Critical
>  Labels: patch
> Fix For: 3.7
>
>
> Per BlackDuck Hub VulnDB 171100
> Apache Commons Net contains a flaw in the changeWorkingDirectory() function 
> in ftpClient.java that is triggered as user-supplied input is not properly 
> sanitized. This may allow a remote attacker to use a newline character in a 
> specially crafted string to execute arbitrary commands.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (NET-670) Apache Commons Net ftpClient.java changeWorkingDirectory() Function CRLF Injection Remote Command Execution

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/NET-670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865195#comment-16865195
 ] 

Sebb commented on NET-670:
--

NET is intended as a low-level interface to various internet protocols.

The caller application must validate user input before calling NET methods.

> Apache Commons Net ftpClient.java changeWorkingDirectory() Function CRLF 
> Injection Remote Command Execution
> ---
>
> Key: NET-670
> URL: https://issues.apache.org/jira/browse/NET-670
> Project: Commons Net
>  Issue Type: Bug
>  Components: FTP
>Affects Versions: 3.6
>Reporter: David Hurst
>Priority: Critical
>  Labels: patch
> Fix For: 3.7
>
>
> Per BlackDuck Hub VulnDB 171100
> Apache Commons Net contains a flaw in the changeWorkingDirectory() function 
> in ftpClient.java that is triggered as user-supplied input is not properly 
> sanitized. This may allow a remote attacker to use a newline character in a 
> specially crafted string to execute arbitrary commands.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CODEC-256) ColognePhonetic.encode() calculates wrong code

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CODEC-256?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865193#comment-16865193
 ] 

Sebb commented on CODEC-256:


Are you sure that code 0 is completely ignored?

According to Wikipedia, code 0 is dropped AFTER duplicates have been 
eliminated, not before.
That makes more sense, as we then get:

Roman => 70606 => 766
Rom => 706 =>  76

rather than both having the value 76.

Unless there is independent evidence that 0 is eliminated before 
de-duplication, I think this issue is invalid


> ColognePhonetic.encode() calculates wrong code
> --
>
> Key: CODEC-256
> URL: https://issues.apache.org/jira/browse/CODEC-256
> Project: Commons Codec
>  Issue Type: Bug
>Affects Versions: 1.12
> Environment: Java 8u181 64, Windows 7 Professional 64
>Reporter: Andreas Schnur
>Priority: Major
> Attachments: ColognePhonetic.java
>
>
> Example String: "ARTMANN". Code of M is 6, Code of A is Zero (and therefore 
> is left out), Code of N is 6 and should be left out because 6 is previous, 
> but Code assumes wrongly 0 as previous (which was left out). So the result is 
> 07266, but has to be 0726 (see Description of algorithm in class).
> Solution: In method colognePhonetic() move the last two LOC into if-block 
> above, see attached version of class.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (VFS-717) Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.

2019-06-16 Thread Gary Gregory (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-717?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary Gregory updated VFS-717:
-
Summary: Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.  
(was: Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.)

> Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.
> 
>
> Key: VFS-717
> URL: https://issues.apache.org/jira/browse/VFS-717
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: Gary Gregory
>Priority: Major
>
> Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CODEC-253) Upgrade to Java 8

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CODEC-253?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865189#comment-16865189
 ] 

Sebb commented on CODEC-253:


Agree with Michael and Emmanuel.
Code should remain on Java 7 until such time as there is a genuine need to 
upgrade to the next version of Java.

> Upgrade to Java 8
> -
>
> Key: CODEC-253
> URL: https://issues.apache.org/jira/browse/CODEC-253
> Project: Commons Codec
>  Issue Type: Improvement
>Reporter: Rob Tompkins
>Assignee: Rob Tompkins
>Priority: Major
> Fix For: 1.13
>
>
> Upgrade to java 8 as the target of of the build.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (VFS-717) Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.

2019-06-16 Thread Gary Gregory (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-717?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary Gregory updated VFS-717:
-
Description: Update org.apache.httpcomponents:httpclient from 4.5.8 to 
4.5.9.  (was: Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.)

> Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.
> 
>
> Key: VFS-717
> URL: https://issues.apache.org/jira/browse/VFS-717
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: Gary Gregory
>Priority: Major
>
> Update org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Reopened] (CODEC-253) Upgrade to Java 8

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CODEC-253?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb reopened CODEC-253:


This issue has not been properly resolved

> Upgrade to Java 8
> -
>
> Key: CODEC-253
> URL: https://issues.apache.org/jira/browse/CODEC-253
> Project: Commons Codec
>  Issue Type: Improvement
>Reporter: Rob Tompkins
>Assignee: Rob Tompkins
>Priority: Major
> Fix For: 1.13
>
>
> Upgrade to java 8 as the target of of the build.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (VFS-717) Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.

2019-06-16 Thread Gary Gregory (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-717?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gary Gregory updated VFS-717:
-
Summary: Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.  
(was: Udpate org.apache.httpcomponents:httpclient fro)

> Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.
> 
>
> Key: VFS-717
> URL: https://issues.apache.org/jira/browse/VFS-717
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: Gary Gregory
>Priority: Major
>
> Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (VFS-717) Udpate org.apache.httpcomponents:httpclient fro

2019-06-16 Thread Gary Gregory (JIRA)
Gary Gregory created VFS-717:


 Summary: Udpate org.apache.httpcomponents:httpclient fro
 Key: VFS-717
 URL: https://issues.apache.org/jira/browse/VFS-717
 Project: Commons VFS
  Issue Type: Bug
Affects Versions: 2.3
Reporter: Gary Gregory


Udpate org.apache.httpcomponents:httpclient from 4.5.8 to 4.5.9.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (EMAIL-188) Issues with Apache commons email library

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/EMAIL-188?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved EMAIL-188.

Resolution: Not A Bug

> Issues with Apache commons email library
> 
>
> Key: EMAIL-188
> URL: https://issues.apache.org/jira/browse/EMAIL-188
> Project: Commons Email
>  Issue Type: Bug
>Reporter: Anurag Garg
>Priority: Major
>
> Hi team,
> We are facing below issues with apache commons email library
> 1) When a user is sending email having inline image in body along with image 
> as an attachment in that case when we are calling getAttachments() method of 
> MimeMessageparser it is giving both the images because of that we are not 
> able to distinguish between the actual image as attachment vs inline image.
> 2) When a user is sending email with .msg file as attachment in that case it 
> is not even coming as attachment when we are calling getAttachments() method 
> of MimeMessageparser. 
> Not sure if the above behavior is expected or not? Can you please look into 
> this.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Work logged] (VFS-619) Issue creating HttpClient timeout

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-619?focusedWorklogId=261071=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261071
 ]

ASF GitHub Bot logged work on VFS-619:
--

Author: ASF GitHub Bot
Created on: 16/Jun/19 22:23
Start Date: 16/Jun/19 22:23
Worklog Time Spent: 10m 
  Work Description: garydgregory commented on pull request #65: VFS-619 
Fixes performance issue with SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261071)
Time Spent: 20m  (was: 10m)

> Issue creating HttpClient timeout
> -
>
> Key: VFS-619
> URL: https://issues.apache.org/jira/browse/VFS-619
> Project: Commons VFS
>  Issue Type: Bug
>Reporter: Ben Xiang
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> Hi,
> Recently we met an issue that the thread got stuck during the http connection 
> using HttpClient. And It showed that the connection timeout was set to 0, 
> which was not what we set. 
> let me briefly show here how we are using VFS2:
> Create a FileSystemOptions with timeout set:
> {code}
> final FileSystemOptions fso = new FileSystemOptions();
> final HttpFileSystemConfigBuilder httpBuilder = 
> HttpFileSystemConfigBuilder.getInstance();
> httpBuilder.setConnectionTimeout(fso, connectionTimeout);
> httpBuilder.setSoTimeout(fso, soTimeout);
> {code}
> Create a FileObject using FileSystemManager:
> {code}
> vfs2FileManager.resolveFile(fileLocation, fso);
> {code}
> Finally get the input stream using DefaultFileContent:
> {code}
> final InputStream instr = fileObject.getInputStream();
> {code}
> We noticed that in executeMethod() of HttpMethodDirector, line 153, it is 
> setting the connection timeout using client's params. However,
> in HttpClientFactory, it's setting fileSystemOptions in connectionMgrParams, 
> and the HttpClient's params would not have this value.
> So we added following 2 lines to make it work:
> In package org.apache.commons.vfs2.provider.http, HttpClientFactory, added 
> line 148 and 149:
> {code}
> client.getParams().setConnectionManagerTimeout(connectionMgrParams.getConnectionTimeout());
> client.getParams().setSoTimeout(connectionMgrParams.getSoTimeout());
> {code}
> Please let me know your thought about this and if there is a patch needed, or 
> is there something we are missing or doing wrong ?
> Thanks!



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-vfs] garydgregory merged pull request #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
garydgregory merged pull request #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-vfs] garydgregory commented on issue #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
garydgregory commented on issue #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65#issuecomment-502490732
 
 
   Squashing and merged. @DavidSeptimus & @ecki Thank you.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Created] (IO-609) FileUtils.copyToFile backward incompatibility bug

2019-06-16 Thread Lingchao Chen (JIRA)
Lingchao Chen created IO-609:


 Summary: FileUtils.copyToFile backward incompatibility bug
 Key: IO-609
 URL: https://issues.apache.org/jira/browse/IO-609
 Project: Commons IO
  Issue Type: Bug
Affects Versions: 2.6
Reporter: Lingchao Chen


Hi,

The following code snippets throw an IOException: Stream closed. It works well 
before commons-io 2.6. When I update commons-io to 2.6, it failed. There is a 
backward incompatibility bug behind it.

The function code:

{code:java}
public class Demo {
  private void backupFile(String srcPath, String entryPath, 
ZipOutputStream stream) throws IOException {
ZipEntry zipEntry = new ZipEntry(entryPath);
stream.putNextEntry(zipEntry);
Files.copy(Paths.get(srcPath), stream);
  }
  private void backupDir(String srcDir, String dstDir, ZipOutputStream 
stream) throws IOException {
File dir = new File(srcDir);
for (String path : dir.list()) {
  System.out.println(path);
  backupFile(dir.getAbsolutePath() + File.separator + path, 
dstDir + File.separator + path, stream);
}
  }
  public void backup(String name) throws IOException {
  
Files.createDirectories(Paths.get("/Users/chenlingchao/eclipse_projects/workspace/BBI.BugDetection"));
  ZipOutputStream stream = new ZipOutputStream(
  
Files.newOutputStream(Paths.get("/Users/chenlingchao/eclipse_projects/workspace/BBI.BugDetection/tmp"
 + File.separator + name)));
  try {

backupDir("/Users/chenlingchao/eclipse_projects/workspace/BBI.BugDetection/tmp",
 "meta" + File.separator + "tables", stream);
stream.closeEntry();
  } finally {
stream.close();
  }
}
  
  public void restore(String name) throws IOException {
  ZipInputStream stream = new ZipInputStream(
  
Files.newInputStream(Paths.get("/Users/chenlingchao/eclipse_projects/workspace/BBI.BugDetection/tmp"
 + File.separator + name)));
  try {
ZipEntry entry;
while ((entry = stream.getNextEntry()) != null) {
  FileUtils.copyToFile(stream, new 
File("/Users/chenlingchao/eclipse_projects/workspace/BBI.BugDetection/tmp" + 
File.separator + entry.getName()));
}
  } finally {
stream.close();
  }
  }
}
{code}

The test code:

{code:java}
@Test
public void TestDemo() throws IOException{
Demo test = new Demo();
test.backup("test.zip");
test.restore("test.zip");
}

{code}





--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Comment Edited] (COMMONSSITE-100) "mvn site" fails with commons-parent 43

2019-06-16 Thread Gilles (JIRA)


[ 
https://issues.apache.org/jira/browse/COMMONSSITE-100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865184#comment-16865184
 ] 

Gilles edited comment on COMMONSSITE-100 at 6/16/19 10:03 PM:
--

Issue still exists when running
{noformat}
$ mvn clean site
{noformat}
for "Commons Math" after upgrading to CP 48 (cf. commit 
418f2eb7f469c80b0e125377b1669f9be068f6ff on "master").


was (Author: erans):
Issue still exists on my machine with CP 48 (as of commit 
418f2eb7f469c80b0e125377b1669f9be068f6ff on "master").

> "mvn site" fails with commons-parent 43
> ---
>
> Key: COMMONSSITE-100
> URL: https://issues.apache.org/jira/browse/COMMONSSITE-100
> Project: Commons All
>  Issue Type: Bug
>  Components: Commons Parent Pom
>Reporter: Gilles
>Priority: Major
>
> See [this post|http://markmail.org/message/zgfb5h774aigatx2].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (COMMONSSITE-100) "mvn site" fails with commons-parent 43

2019-06-16 Thread Gilles (JIRA)


[ 
https://issues.apache.org/jira/browse/COMMONSSITE-100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865184#comment-16865184
 ] 

Gilles commented on COMMONSSITE-100:


Issue still exists on my machine with CP 48 (as of commit 
418f2eb7f469c80b0e125377b1669f9be068f6ff on "master").

> "mvn site" fails with commons-parent 43
> ---
>
> Key: COMMONSSITE-100
> URL: https://issues.apache.org/jira/browse/COMMONSSITE-100
> Project: Commons All
>  Issue Type: Bug
>  Components: Commons Parent Pom
>Reporter: Gilles
>Priority: Major
>
> See [this post|http://markmail.org/message/zgfb5h774aigatx2].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (COLLECTIONS-721) functors.InvokerTransformer backward incompatibility bug

2019-06-16 Thread Lingchao Chen (JIRA)
Lingchao Chen created COLLECTIONS-721:
-

 Summary: functors.InvokerTransformer backward incompatibility bug
 Key: COLLECTIONS-721
 URL: https://issues.apache.org/jira/browse/COLLECTIONS-721
 Project: Commons Collections
  Issue Type: Bug
Affects Versions: 4.3, 4.2, 4.1
Reporter: Lingchao Chen


The following code snippets throw a NotSerializableException. It works well 
before commons-collections4 4.0 and commons-collection3. When I update it to 
4.1 or newer versions, it failed.


{code:java}
@Test
public void Demo() throws IOException{
InvokerTransformer transformer = new 
InvokerTransformer("toString", new Class[]{}, new Object[]{});
Queue priorityQueue = new PriorityQueue(2, new 
TransformingComparator(transformer));
priorityQueue.add(1);

ByteArrayOutputStream bout = new ByteArrayOutputStream();
ObjectOutputStream out = new ObjectOutputStream(bout);
out.writeObject(priorityQueue);

}
{code}




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (MATH-1489) Test failure because of incompatible version specification of JaCoCo in pom.xml

2019-06-16 Thread Gilles (JIRA)


[ 
https://issues.apache.org/jira/browse/MATH-1489?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865178#comment-16865178
 ] 

Gilles commented on MATH-1489:
--

Thanks for the report.
I've attempted some cleanup and
{noformat}
$ mvn clean package
{noformat}
should work (if we exclude the {{FastMathTest}} failure) as of commit 
418f2eb7f469c80b0e125377b1669f9be068f6ff ("master").

However
{noformat}
$ mvn clean site
{noformat}
fails while building the Javadoc.

> Test failure because of incompatible version specification of JaCoCo in 
> pom.xml
> ---
>
> Key: MATH-1489
> URL: https://issues.apache.org/jira/browse/MATH-1489
> Project: Commons Math
>  Issue Type: Bug
> Environment: OpenJDK 11.0.3
> Debian GNU/Linux 9.9 stretch
>  
>Reporter: M. Z.
>Priority: Critical
>  Labels: build, easyfix, newbie
> Attachments: commons-math-buildlog-2019-06-15.log
>
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> I've experienced constant build failures on the attempt to "mvn clean package"
> on a working copy of [http://gitbox.apache.org/repos/asf/commons-math.git] ( 
> commit 4b0f52c0dd9df564d58140e1d717a2da218f0139 )
> It has to do with the specified version of jacoco in pom.xml. See attached 
> buildlog for concrete error.
> My solution:
> $ git diff
> --- a/pom.xml
> +++ b/pom.xml
> @@ -493,7 +493,7 @@
>  2.7.2
>  
>  
> -    0.8.0
> +    0.8.4 
>  0.96
>  0.8
>  0.8
>  
> See 
> [https://stackoverflow.com/questions/55272870/surefire-maven-plugin-corrupted-stdout-by-directly-writing-to-native-stream-in]
>  for a previously reported issue with the use jacoco and OpenJDK 11.
> After the proposed change, I've got
> $ mvn clean package
> 
> [INFO] Results:
> [INFO] 
> [ERROR] Failures: 
> [ERROR]   FastMathTest.checkMissingFastMathClasses:1275 FastMath should 
> implement all StrictMath methods
> [INFO] 
> [ERROR] Tests run: 4864, Failures: 1, Errors: 0, Skipped: 31
> [INFO] 
> [INFO] 
> 
> [INFO] BUILD FAILURE
> [INFO] 
> 
> ...
> So tests should have been executed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (LANG-1465) DateUtils.parseDate backward Incompatibility bug

2019-06-16 Thread Lingchao Chen (JIRA)
Lingchao Chen created LANG-1465:
---

 Summary: DateUtils.parseDate backward Incompatibility bug
 Key: LANG-1465
 URL: https://issues.apache.org/jira/browse/LANG-1465
 Project: Commons Lang
  Issue Type: Bug
  Components: lang.time.*
Affects Versions: 3.8.1, 3.9, 3.8, 3.7, 3.6, 3.5
 Environment: System: Linux testinglab 4.4.0-131-generic 
#157~14.04.1-Ubuntu


Java version "1.8.0_191"
Java(TM) SE Runtime Environment (build 1.8.0_191-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)

 

 
Reporter: Lingchao Chen


Hi,

The following code snippets throw a ParseException when trying to parse the 
time. It works well before commons-lang 3.4. When I update commons-lang to 3.5 
or newer versions, it failed.


{code:java}
@Test
public void Demo() throws ParseException {
assertEquals("Tue Jun 11 15:06:11 CDT 2019", 
DateUtils.parseDate("2019-06-11T15:06:11.716-0500", 
"-MM-dd'T'HH:mm:ss.SSSZZ"));

}
{code}




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-135) Char escape doesn't work properly with quoting

2019-06-16 Thread Opher Shachar (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865157#comment-16865157
 ] 

Opher Shachar commented on CSV-135:
---

Sorry, I probably did not make my point clear before.
 If the field value is:
 * "D:\Property Documents\ID 1.docx"

the `\` is obviously a literal not an escape. While for 
 * "3rd Lincoln boulevard\, Miami\, FL"

the `\` may or may-not be an escape.

The solution could be by adding a new flag (or a new {{enum}}) that modifies 
{{QuoteMode}}'s handling by making it explicit whether `\` are literals, or an 
escape only if followed by specific chars (i.e. `,`,`\`) or always an escape.

> Char escape doesn't work properly with quoting
> --
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-vfs] DavidSeptimus commented on issue #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
DavidSeptimus commented on issue #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65#issuecomment-502484151
 
 
   That's a fair point.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-vfs] ecki commented on issue #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
ecki commented on issue #65: VFS-619 Fixes performance issue with 
SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65#issuecomment-502483677
 
 
   It is probably better to remove this refresh implementation completely, the 
super implementation will call detach/doDetach which already sets `attrs = 
null`  


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Created] (MATH-1491) Percentile computational accuracy issue

2019-06-16 Thread Lingchao Chen (JIRA)
Lingchao Chen created MATH-1491:
---

 Summary: Percentile computational accuracy issue
 Key: MATH-1491
 URL: https://issues.apache.org/jira/browse/MATH-1491
 Project: Commons Math
  Issue Type: Bug
Affects Versions: 3.6.1, 3.6, 3.5, 3.4.1, 3.4
 Environment: System: Linux testinglab 4.4.0-131-generic 
#157~14.04.1-Ubuntu


Java version "1.8.0_191"
Java(TM) SE Runtime Environment (build 1.8.0_191-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)

 

 
Reporter: Lingchao Chen
 Attachments: BugDemo.java

Hi, 

The percentile method works well on the older versions, e.g., the version 
before 3.4. However, when I update commons-math to the newer version, there 
produces a computational accuracy issue. There is a backward compatibility bug 
behind it.




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (CSV-135) Char escape doesn't work properly with quoting

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved CSV-135.
--
Resolution: Fixed

7754cd4  CSV-135 - Char escape doesn't work properly with quoting

> Char escape doesn't work properly with quoting
> --
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (VFS-698) SFTP file attributes are fetched multiple times leading to very slow directory listing

2019-06-16 Thread David Septimus (JIRA)


[ 
https://issues.apache.org/jira/browse/VFS-698?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865152#comment-16865152
 ] 

David Septimus commented on VFS-698:


PR raised: [https://github.com/apache/commons-vfs/pull/65]

> SFTP file attributes are fetched multiple times leading to very slow 
> directory listing
> --
>
> Key: VFS-698
> URL: https://issues.apache.org/jira/browse/VFS-698
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: krishnan
>Priority: Major
>
> getChildren() applied on SftpFileObject is very slow compared to JSCH 
> implementation. This is because, the SftpATTRS which is readily available for 
> the children after an "ls" call is again fetched for each child file since 
> they are independently resolved. So if a directory contains 10 files, it 
> results in 1 (ls) + 10 (stat) calls to server.
> For a folder with 100 files (AWS), it took about 35 secs instead of 1.5 secs 
> to  getChildren().
>  
> *doListChildrenResolved:*
> {{final FileObject fo = 
> getFileSystem().resolveFile(getFileSystem().getFileSystemManager()}}
>  \{{ .resolveName(getName(), UriParser.encode(name), NameScope.CHILD));}}
> {{{color:#ff}((SftpFileObject) 
> FileObjectUtils.getAbstractFileObject(fo)).setStat(stat.getAttrs());{color}}}
>  
> The resolveFile call, creates a SftpFileObject and calls its resolve method, 
> which results in getting the (stats) SftpATTRS for each child file. This stat 
> is already available as part of the 'ls' call we made. The setStat call above 
> (highlighted is red) is redundant, since stat for each child file is already 
> fetched one at a time.
> The solution would be to avoid getting the stat for each child file after an 
> 'ls' call. May be, the framework makes it difficult to do this easily.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Work logged] (VFS-619) Issue creating HttpClient timeout

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/VFS-619?focusedWorklogId=261060=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261060
 ]

ASF GitHub Bot logged work on VFS-619:
--

Author: ASF GitHub Bot
Created on: 16/Jun/19 20:23
Start Date: 16/Jun/19 20:23
Worklog Time Spent: 10m 
  Work Description: DavidSeptimus commented on pull request #65: VFS-619 
Fixes performance issue with SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65
 
 
   This PR removes an unnecessary invocation of `SftpObject.getType()` by 
`refresh()`. 
   
   This invocation causes performance degradation of 
`SftpFileObject.getChildren()` when `CacheStrategy.ON_RESOLVE` is in use as 
this forces a call to `refresh()`, which in turn calls `getType()`, making an 
additional `stat()` call to the SFTP server for each child.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261060)
Time Spent: 10m
Remaining Estimate: 0h

> Issue creating HttpClient timeout
> -
>
> Key: VFS-619
> URL: https://issues.apache.org/jira/browse/VFS-619
> Project: Commons VFS
>  Issue Type: Bug
>Reporter: Ben Xiang
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Hi,
> Recently we met an issue that the thread got stuck during the http connection 
> using HttpClient. And It showed that the connection timeout was set to 0, 
> which was not what we set. 
> let me briefly show here how we are using VFS2:
> Create a FileSystemOptions with timeout set:
> {code}
> final FileSystemOptions fso = new FileSystemOptions();
> final HttpFileSystemConfigBuilder httpBuilder = 
> HttpFileSystemConfigBuilder.getInstance();
> httpBuilder.setConnectionTimeout(fso, connectionTimeout);
> httpBuilder.setSoTimeout(fso, soTimeout);
> {code}
> Create a FileObject using FileSystemManager:
> {code}
> vfs2FileManager.resolveFile(fileLocation, fso);
> {code}
> Finally get the input stream using DefaultFileContent:
> {code}
> final InputStream instr = fileObject.getInputStream();
> {code}
> We noticed that in executeMethod() of HttpMethodDirector, line 153, it is 
> setting the connection timeout using client's params. However,
> in HttpClientFactory, it's setting fileSystemOptions in connectionMgrParams, 
> and the HttpClient's params would not have this value.
> So we added following 2 lines to make it work:
> In package org.apache.commons.vfs2.provider.http, HttpClientFactory, added 
> line 148 and 149:
> {code}
> client.getParams().setConnectionManagerTimeout(connectionMgrParams.getConnectionTimeout());
> client.getParams().setSoTimeout(connectionMgrParams.getSoTimeout());
> {code}
> Please let me know your thought about this and if there is a patch needed, or 
> is there something we are missing or doing wrong ?
> Thanks!



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (CSV-135) Char escape doesn't work properly with quoting

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb updated CSV-135:
-
Summary: Char escape doesn't work properly with quoting  (was: Char escape 
doesn't work)

> Char escape doesn't work properly with quoting
> --
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-vfs] DavidSeptimus opened a new pull request #65: VFS-619 Fixes performance issue with SftpFileObject.getChildren()

2019-06-16 Thread GitBox
DavidSeptimus opened a new pull request #65: VFS-619 Fixes performance issue 
with SftpFileObject.getChildren()
URL: https://github.com/apache/commons-vfs/pull/65
 
 
   This PR removes an unnecessary invocation of `SftpObject.getType()` by 
`refresh()`. 
   
   This invocation causes performance degradation of 
`SftpFileObject.getChildren()` when `CacheStrategy.ON_RESOLVE` is in use as 
this forces a call to `refresh()`, which in turn calls `getType()`, making an 
additional `stat()` call to the SFTP server for each child.


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (CSV-135) Char escape doesn't work

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865150#comment-16865150
 ] 

Sebb commented on CSV-135:
--

bq. This is exactly where QuoteMode comes into play: If it is NONE then it is 
strictly print-with-escapes, otherwise it's some flavor of Quoting.

Yes, but the other QuoteMode settings failed to take account of embedded 
escapes.

> Char escape doesn't work
> 
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-135) Char escape doesn't work

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865149#comment-16865149
 ] 

Sebb commented on CSV-135:
--

This fixes the problem as originally reported.

f7c2ca2  CSV-135 - Char escape doesn't work

If there are related problems, please raise a separate issue.

> Char escape doesn't work
> 
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (MATH-1490) Percentile computational accuracy issue

2019-06-16 Thread Lingchao Chen (JIRA)
Lingchao Chen created MATH-1490:
---

 Summary: Percentile computational accuracy issue
 Key: MATH-1490
 URL: https://issues.apache.org/jira/browse/MATH-1490
 Project: Commons Math
  Issue Type: Bug
Affects Versions: 3.6.1, 3.6, 3.5, 3.4.1, 3.4
 Environment: System: Linux testinglab 4.4.0-131-generic 
#157~14.04.1-Ubuntu


Java version "1.8.0_191"
Java(TM) SE Runtime Environment (build 1.8.0_191-b12)
Java HotSpot(TM) 64-Bit Server VM (build 25.191-b12, mixed mode)

 

 
Reporter: Lingchao Chen
 Attachments: BugDemo.java

The percentile method works well on the older versions, e.g., the version 
before 3.4. However, when I update commons-math to the newer version, there 
produces a computational accuracy issue. There is a backward compatibility bug 
behind it.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-135) Char escape doesn't work

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865147#comment-16865147
 ] 

Sebb commented on CSV-135:
--

The problem is two fold:
1) {{printWithQuotes}} does not check for embedded escapes when deciding 
whether to encapsulate the value
2) it does not escape embedded escape chars in the value


> Char escape doesn't work
> 
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (VFS-698) SFTP file attributes are fetched multiple times leading to very slow directory listing

2019-06-16 Thread David Septimus (JIRA)


[ 
https://issues.apache.org/jira/browse/VFS-698?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865112#comment-16865112
 ] 

David Septimus commented on VFS-698:


This issue is cause by a call to {{getType()}} inside from {{SftpFileObject}}'s 
implementation of {{FileObject.refresh()}}. This forces a {{stat}} call to the 
SFTP server to refresh the instance's {{FileType}}.

Based on my understanding of the refresh process, the primary purpose of 
{{refresh()}} is to clear the internal state of the {{FileObject}} instance, so 
the next operation called on the instance is forced to refresh its state from 
the underlying file sytstem. {{SftpFileObject.refresh()}} is the only 
implementation that  actually makes a file system call.

> SFTP file attributes are fetched multiple times leading to very slow 
> directory listing
> --
>
> Key: VFS-698
> URL: https://issues.apache.org/jira/browse/VFS-698
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: krishnan
>Priority: Major
>
> getChildren() applied on SftpFileObject is very slow compared to JSCH 
> implementation. This is because, the SftpATTRS which is readily available for 
> the children after an "ls" call is again fetched for each child file since 
> they are independently resolved. So if a directory contains 10 files, it 
> results in 1 (ls) + 10 (stat) calls to server.
> For a folder with 100 files (AWS), it took about 35 secs instead of 1.5 secs 
> to  getChildren().
>  
> *doListChildrenResolved:*
> {{final FileObject fo = 
> getFileSystem().resolveFile(getFileSystem().getFileSystemManager()}}
>  \{{ .resolveName(getName(), UriParser.encode(name), NameScope.CHILD));}}
> {{{color:#ff}((SftpFileObject) 
> FileObjectUtils.getAbstractFileObject(fo)).setStat(stat.getAttrs());{color}}}
>  
> The resolveFile call, creates a SftpFileObject and calls its resolve method, 
> which results in getting the (stats) SftpATTRS for each child file. This stat 
> is already available as part of the 'ls' call we made. The setStat call above 
> (highlighted is red) is redundant, since stat for each child file is already 
> fetched one at a time.
> The solution would be to avoid getting the stat for each child file after an 
> 'ls' call. May be, the framework makes it difficult to do this easily.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (VFS-698) SFTP file attributes are fetched multiple times leading to very slow directory listing

2019-06-16 Thread David Septimus (JIRA)


[ 
https://issues.apache.org/jira/browse/VFS-698?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865089#comment-16865089
 ] 

David Septimus commented on VFS-698:


As far as I can tell, this is only an issue when your {{FileSystem}} is using 
{{CacheStrategy.ON_RESOLVE}}. This forces the {{FileSystem}} to call 
{{refresh()}} on the newly created {{SftpFileObject}} (see: 
{{AbstractFileSystem.resolveFile(...)}})  and because {{attrs}} is null at the 
the time of resolution, a call is made to the server in order to determine the 
file's {{FileType}} (see: {{SftpFileObject.refresh()}} and 
{{SftpFileObject.doGetType()}}).

> SFTP file attributes are fetched multiple times leading to very slow 
> directory listing
> --
>
> Key: VFS-698
> URL: https://issues.apache.org/jira/browse/VFS-698
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: krishnan
>Priority: Major
>
> getChildren() applied on SftpFileObject is very slow compared to JSCH 
> implementation. This is because, the SftpATTRS which is readily available for 
> the children after an "ls" call is again fetched for each child file since 
> they are independently resolved. So if a directory contains 10 files, it 
> results in 1 (ls) + 10 (stat) calls to server.
> For a folder with 100 files (AWS), it took about 35 secs instead of 1.5 secs 
> to  getChildren().
>  
> *doListChildrenResolved:*
> {{final FileObject fo = 
> getFileSystem().resolveFile(getFileSystem().getFileSystemManager()}}
>  \{{ .resolveName(getName(), UriParser.encode(name), NameScope.CHILD));}}
> {{{color:#ff}((SftpFileObject) 
> FileObjectUtils.getAbstractFileObject(fo)).setStat(stat.getAttrs());{color}}}
>  
> The resolveFile call, creates a SftpFileObject and calls its resolve method, 
> which results in getting the (stats) SftpATTRS for each child file. This stat 
> is already available as part of the 'ls' call we made. The setStat call above 
> (highlighted is red) is redundant, since stat for each child file is already 
> fetched one at a time.
> The solution would be to avoid getting the stat for each child file after an 
> 'ls' call. May be, the framework makes it difficult to do this easily.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-135) Char escape doesn't work

2019-06-16 Thread Opher Shachar (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865078#comment-16865078
 ] 

Opher Shachar commented on CSV-135:
---

On the printing front
{quote}{color:#33}However, for printing, if both are specified and the 
chars are different, which should take precedence?{color}
{color:#33} (Be strict in what you generate){color}
{quote}
This is *_exactly_* where {color:#14892c}*QuoteMode*{color} comes into play: If 
it is *NONE* then it is strictly +_print-with-escapes_+, otherwise it's some 
flavor of Quoting.

For parsing how would you parse a record such as?

 
|ID,Address,Image Path,Document Path
1,"3rd Lincoln boulevard\, Miami\, FL",D:\Images\Houses\ID_1.png,"D:\Property 
Documents\ID 1.docx"|

 

> Char escape doesn't work
> 
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-177) Support trimming leading and trailing blanks

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-177?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865067#comment-16865067
 ] 

Sebb commented on CSV-177:
--

Since found out that withTrim() affects the storage of values in 
CSVParser#addRecordValue

> Support trimming leading and trailing blanks
> 
>
> Key: CSV-177
> URL: https://issues.apache.org/jira/browse/CSV-177
> Project: Commons CSV
>  Issue Type: New Feature
>  Components: Parser, Printer
>Affects Versions: 1.2
>Reporter: Gary Gregory
>Assignee: Gary Gregory
>Priority: Major
> Fix For: 1.3
>
>
> Support trimming leading and trailing blanks in the printer and parser.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (STATISTICS-17) RELEVANT INFORMATION LIST

2019-06-16 Thread Ben Nguyen (JIRA)


 [ 
https://issues.apache.org/jira/browse/STATISTICS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ben Nguyen updated STATISTICS-17:
-
Summary: RELEVANT INFORMATION LIST  (was: CS Regression: RELEVANT 
INFORMATION LIST)

> RELEVANT INFORMATION LIST
> -
>
> Key: STATISTICS-17
> URL: https://issues.apache.org/jira/browse/STATISTICS-17
> Project: Apache Commons Statistics
>  Issue Type: Sub-task
>Reporter: Ben Nguyen
>Priority: Major
>
> *+Relevant JIRA Issues+*
>  * OLSMultipleLinearRegression estimates different residuals with different 
> order of input
>  ** https://issues.apache.org/jira/browse/MATH-1428
>  * SimpleRegression#getSlopeConfidenceInterval recalculates t distribution on 
> every call
>  ** https://issues.apache.org/jira/browse/MATH-1441
> +*Relevant Legacy Pull Requests*+
>  * [https://github.com/apache/commons-math/pull/106]
>  ** Related JIRA:
>  *** Pull request for GLSMultipleLinearRegression
>  *** https://issues.apache.org/jira/browse/MATH-1482



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Updated] (STATISTICS-17) RELEVANT INFORMATION LIST

2019-06-16 Thread Ben Nguyen (JIRA)


 [ 
https://issues.apache.org/jira/browse/STATISTICS-17?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ben Nguyen updated STATISTICS-17:
-
Priority: Blocker  (was: Major)

> RELEVANT INFORMATION LIST
> -
>
> Key: STATISTICS-17
> URL: https://issues.apache.org/jira/browse/STATISTICS-17
> Project: Apache Commons Statistics
>  Issue Type: Sub-task
>Reporter: Ben Nguyen
>Priority: Blocker
>
> *+Relevant JIRA Issues+*
>  * OLSMultipleLinearRegression estimates different residuals with different 
> order of input
>  ** https://issues.apache.org/jira/browse/MATH-1428
>  * SimpleRegression#getSlopeConfidenceInterval recalculates t distribution on 
> every call
>  ** https://issues.apache.org/jira/browse/MATH-1441
> +*Relevant Legacy Pull Requests*+
>  * [https://github.com/apache/commons-math/pull/106]
>  ** Related JIRA:
>  *** Pull request for GLSMultipleLinearRegression
>  *** https://issues.apache.org/jira/browse/MATH-1482



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Created] (STATISTICS-17) CS Regression: RELEVANT INFORMATION LIST

2019-06-16 Thread Ben Nguyen (JIRA)
Ben Nguyen created STATISTICS-17:


 Summary: CS Regression: RELEVANT INFORMATION LIST
 Key: STATISTICS-17
 URL: https://issues.apache.org/jira/browse/STATISTICS-17
 Project: Apache Commons Statistics
  Issue Type: Sub-task
Reporter: Ben Nguyen


*+Relevant JIRA Issues+*
 * OLSMultipleLinearRegression estimates different residuals with different 
order of input
 ** https://issues.apache.org/jira/browse/MATH-1428
 * SimpleRegression#getSlopeConfidenceInterval recalculates t distribution on 
every call
 ** https://issues.apache.org/jira/browse/MATH-1441

+*Relevant Legacy Pull Requests*+
 * [https://github.com/apache/commons-math/pull/106]
 ** Related JIRA:
 *** Pull request for GLSMultipleLinearRegression
 *** https://issues.apache.org/jira/browse/MATH-1482



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-numbers] coveralls edited a comment on issue #48: [WIP] perform the actual migration to JUnit 5

2019-06-16 Thread GitBox
coveralls edited a comment on issue #48: [WIP] perform the actual migration to 
JUnit 5
URL: https://github.com/apache/commons-numbers/pull/48#issuecomment-500176439
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/24023995/badge)](https://coveralls.io/builds/24023995)
   
   Coverage remained the same at 94.381% when pulling 
**b1a114f406e90b2432b6e925cfcc2497477ce339 on 
grimreaper:eax/simple-migrate-map** into 
**934be57412ff6ab9fd67f0eeeddba49895afff06 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Commented] (VFS-698) SFTP file attributes are fetched multiple times leading to very slow directory listing

2019-06-16 Thread David Septimus (JIRA)


[ 
https://issues.apache.org/jira/browse/VFS-698?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865060#comment-16865060
 ] 

David Septimus commented on VFS-698:


I just ran into this issue myself trying to list a directory of ~200,000 files. 
The call to getChildren() ran for over an hour before I killed it whereas the 
underlying call to Channel.ls(...) ran for ~30 seconds for the same directory.

Due to time constraints, I went with a workaround – extending the FileObject 
interface with a streamChildren() method that returns a Stream and  
an SftpFileObject implementation of that method to lazily convert LsEntry 
instances to SftpFileObject as needed.

> SFTP file attributes are fetched multiple times leading to very slow 
> directory listing
> --
>
> Key: VFS-698
> URL: https://issues.apache.org/jira/browse/VFS-698
> Project: Commons VFS
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: krishnan
>Priority: Major
>
> getChildren() applied on SftpFileObject is very slow compared to JSCH 
> implementation. This is because, the SftpATTRS which is readily available for 
> the children after an "ls" call is again fetched for each child file since 
> they are independently resolved. So if a directory contains 10 files, it 
> results in 1 (ls) + 10 (stat) calls to server.
> For a folder with 100 files (AWS), it took about 35 secs instead of 1.5 secs 
> to  getChildren().
>  
> *doListChildrenResolved:*
> {{final FileObject fo = 
> getFileSystem().resolveFile(getFileSystem().getFileSystemManager()}}
>  \{{ .resolveName(getName(), UriParser.encode(name), NameScope.CHILD));}}
> {{{color:#ff}((SftpFileObject) 
> FileObjectUtils.getAbstractFileObject(fo)).setStat(stat.getAttrs());{color}}}
>  
> The resolveFile call, creates a SftpFileObject and calls its resolve method, 
> which results in getting the (stats) SftpATTRS for each child file. This stat 
> is already available as part of the 'ls' call we made. The setStat call above 
> (highlighted is red) is redundant, since stat for each child file is already 
> fetched one at a time.
> The solution would be to avoid getting the stat for each child file after an 
> 'ls' call. May be, the framework makes it difficult to do this easily.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (CSV-135) Char escape doesn't work

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16865056#comment-16865056
 ] 

Sebb commented on CSV-135:
--

On the contrary, quote and escape chars can both be specified; indeed many 
formats do this.
For example ORACLE and INFORMIX_UNLOAD.

For parsing, this should not be a problem, as both  and 
 should generate .
(Be liberal in what you accept)

However, for printing, if both are specified and the chars are different, which 
should take precedence?
(Be strict in what you generate)
This currently applies to the following formats:
INFORMIX_UNLOAD, ORACLE, POSTGRESQL_TEXT

> Char escape doesn't work
> 
>
> Key: CSV-135
> URL: https://issues.apache.org/jira/browse/CSV-135
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Printer
>Affects Versions: 1.0
>Reporter: Mateusz Zakarczemny
>Priority: Major
> Fix For: Patch Needed
>
>
> Following code:
> {code}
> CSVFormat format = CSVFormat.DEFAULT
>   .withRecordSeparator('\n')
>   .withQuote('"')
>   .withEscape('\\');
> CSVPrinter printer = new CSVPrinter(System.out, format);
> List l = new LinkedList();
> l.add("\"");
> l.add("\n");
> l.add("\\");
> printer.printRecord(l);
> {code}
> produces
> {code}
> ,"
> ","\"
> {code}
> instead of
> {code}
> "\"","\n","\\"
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-numbers] coveralls edited a comment on issue #48: [WIP] perform the actual migration to JUnit 5

2019-06-16 Thread GitBox
coveralls edited a comment on issue #48: [WIP] perform the actual migration to 
JUnit 5
URL: https://github.com/apache/commons-numbers/pull/48#issuecomment-500176439
 
 
   
   [![Coverage 
Status](https://coveralls.io/builds/24023244/badge)](https://coveralls.io/builds/24023244)
   
   Coverage remained the same at 94.381% when pulling 
**fe63782a789e31f2ffb5f93c5696a009f138a85a on 
grimreaper:eax/simple-migrate-map** into 
**934be57412ff6ab9fd67f0eeeddba49895afff06 on apache:master**.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Resolved] (CSV-235) WRONG Implementation for RFC4180

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CSV-235?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved CSV-235.
--
Resolution: Fixed

AFAICT the implementation is correct; see:

https://github.com/apache/commons-csv/commit/c025d73d31ca9c9c467f3bad142ca62d7ebee76b

If that is not your experience, please re-open with at least a test case that 
shows the issue you are having.

> WRONG Implementation for RFC4180  
> --
>
> Key: CSV-235
> URL: https://issues.apache.org/jira/browse/CSV-235
> Project: Commons CSV
>  Issue Type: Bug
>  Components: Parser
>Affects Versions: 1.6
>Reporter: Edward Xia
>Priority: Major
>
> [https://tools.ietf.org/html/rfc4180#section-2]
>7.  If double-quotes are used to enclose fields, then a double-quote
>appearing inside a field must be escaped by preceding it with
>another double quote.  For example:
>"aaa","b""bb","ccc"
> Apparently, base on a previous issue: 
> https://issues.apache.org/jira/browse/CSV-208, it turns out common-csv *does 
> not even support quote and escape to be the same character*.
> RFC 4180 defines that quote and escape are both DQUOTE, however in common-csv 
> implementation, the default escape character is not DQUOTE, and it does not 
> work if changed to DQUOTE.
> This means common csv is not rfc4180 compliant.
> Also, I'm puzzled by the fact that someone marked CSV-208 as fixed when 
> nothing is fixed. Instead, it changed the behavior without documenting that 
> the POSTGRESQL_CSV format does not even work out of the box with the default 
> csv format that postgresql produces.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Resolved] (CSV-243) CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CSV-243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb resolved CSV-243.
--
Resolution: Fixed

bc64fb5  CSV-243 CSVFormat withTrim() and withIgnoreSurroundingSpaces()

> CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs
> ---
>
> Key: CSV-243
> URL: https://issues.apache.org/jira/browse/CSV-243
> Project: Commons CSV
>  Issue Type: Bug
>Reporter: Sebb
>Priority: Major
>
> It looks like withIgnoreSurroundingSpaces() is a parser option and withTrim() 
> is a printer option.
> These need better Javadoc.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Comment Edited] (CSV-243) CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs

2019-06-16 Thread Sebb (JIRA)


[ 
https://issues.apache.org/jira/browse/CSV-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16864992#comment-16864992
 ] 

Sebb edited comment on CSV-243 at 6/16/19 11:22 AM:


Turns out that withTrim() is used by CSVParser as well as CSVFormat#print.

CSVParser#addRecord uses it to determine whether to trim the field value before 
storing in the recordList.


was (Author: s...@apache.org):
Turns out that withTrim() is used by CSVParser as well as CSVFormat#print

> CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs
> ---
>
> Key: CSV-243
> URL: https://issues.apache.org/jira/browse/CSV-243
> Project: Commons CSV
>  Issue Type: Bug
>Reporter: Sebb
>Priority: Major
>
> It looks like withIgnoreSurroundingSpaces() is a parser option and withTrim() 
> is a printer option.
> These need better Javadoc.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Reopened] (CSV-243) CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs

2019-06-16 Thread Sebb (JIRA)


 [ 
https://issues.apache.org/jira/browse/CSV-243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sebb reopened CSV-243:
--

Turns out that withTrim() is used by CSVParser as well as CSVFormat#print

> CSVFormat withTrim() and withIgnoreSurroundingSpaces() need better docs
> ---
>
> Key: CSV-243
> URL: https://issues.apache.org/jira/browse/CSV-243
> Project: Commons CSV
>  Issue Type: Bug
>Reporter: Sebb
>Priority: Major
>
> It looks like withIgnoreSurroundingSpaces() is a parser option and withTrim() 
> is a printer option.
> These need better Javadoc.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-configuration] sebbASF commented on issue #16: [CONFIGURATION-731] Allow user to specify the comments & separator ch…

2019-06-16 Thread GitBox
sebbASF commented on issue #16: [CONFIGURATION-731] Allow user to specify the 
comments & separator ch…
URL: 
https://github.com/apache/commons-configuration/pull/16#issuecomment-502441549
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF commented on issue #15: CONFIGURATION-729: fixes inconsistency in example

2019-06-16 Thread GitBox
sebbASF commented on issue #15: CONFIGURATION-729: fixes inconsistency in 
example
URL: 
https://github.com/apache/commons-configuration/pull/15#issuecomment-502441532
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF commented on issue #14: [CONFIGURATION-715/716]: Updated the User's guide to include JupIOFac…

2019-06-16 Thread GitBox
sebbASF commented on issue #14: [CONFIGURATION-715/716]: Updated the User's 
guide to include JupIOFac…
URL: 
https://github.com/apache/commons-configuration/pull/14#issuecomment-502441518
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF commented on issue #11: Configuration 703

2019-06-16 Thread GitBox
sebbASF commented on issue #11: Configuration 703
URL: 
https://github.com/apache/commons-configuration/pull/11#issuecomment-502441492
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-logging] sebbASF commented on issue #1: default methods to Log interface to prevent unnecessary payload creation

2019-06-16 Thread GitBox
sebbASF commented on issue #1: default methods to Log interface to prevent 
unnecessary payload creation
URL: https://github.com/apache/commons-logging/pull/1#issuecomment-502441458
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF commented on issue #4: JCS-195 Update element attributes size on serialization

2019-06-16 Thread GitBox
sebbASF commented on issue #4: JCS-195 Update element attributes size on 
serialization
URL: https://github.com/apache/commons-jcs/pull/4#issuecomment-502441296
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF commented on issue #3: JCS-193: [CDI] [JCache annotations] Set configuration more friendly.

2019-06-16 Thread GitBox
sebbASF commented on issue #3: JCS-193: [CDI] [JCache annotations] Set 
configuration more friendly.
URL: https://github.com/apache/commons-jcs/pull/3#issuecomment-502441252
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF commented on issue #2: Javadoc Fixes

2019-06-16 Thread GitBox
sebbASF commented on issue #2: Javadoc Fixes
URL: https://github.com/apache/commons-jcs/pull/2#issuecomment-502441095
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-proxy] sebbASF commented on issue #1: PROXY-25 Adds various utility methods dealing with Spring proxies

2019-06-16 Thread GitBox
sebbASF commented on issue #1: PROXY-25 Adds various utility methods dealing 
with Spring proxies
URL: https://github.com/apache/commons-proxy/pull/1#issuecomment-502440976
 
 
   The PR was auto-closed because trunk was deleted; please re-open against 
master if it still applies


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Work logged] (CONFIGURATION-729) inconsistency in "Quick start guide"

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/CONFIGURATION-729?focusedWorklogId=261008=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261008
 ]

ASF GitHub Bot logged work on CONFIGURATION-729:


Author: ASF GitHub Bot
Created on: 16/Jun/19 10:12
Start Date: 16/Jun/19 10:12
Worklog Time Spent: 10m 
  Work Description: sebbASF commented on pull request #15: 
CONFIGURATION-729: fixes inconsistency in example
URL: https://github.com/apache/commons-configuration/pull/15
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261008)
Time Spent: 10m
Remaining Estimate: 0h

> inconsistency in "Quick start guide"
> 
>
> Key: CONFIGURATION-729
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-729
> Project: Commons Configuration
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 2.3
>Reporter: Moritz Maisel
>Priority: Trivial
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> The guide states 
> {quote}We assume that it is stored as {{database.properties}} in the local 
> file system:
> {quote}
> but in the code example below the name _config.properties_ is used:
> {quote}Configuration config = configs.properties(new 
> File("config.properties"));
> {quote}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Work logged] (JCS-195) Update element attributes size on serialization

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/JCS-195?focusedWorklogId=261010=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261010
 ]

ASF GitHub Bot logged work on JCS-195:
--

Author: ASF GitHub Bot
Created on: 16/Jun/19 10:12
Start Date: 16/Jun/19 10:12
Worklog Time Spent: 10m 
  Work Description: sebbASF commented on pull request #4: JCS-195 Update 
element attributes size on serialization
URL: https://github.com/apache/commons-jcs/pull/4
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261010)
Time Spent: 10m
Remaining Estimate: 0h

> Update element attributes size on serialization
> ---
>
> Key: JCS-195
> URL: https://issues.apache.org/jira/browse/JCS-195
> Project: Commons JCS
>  Issue Type: Improvement
>  Components: Composite Cache
>Affects Versions: jcs-2.2
>Reporter: quietmint
>Assignee: Thomas Vandahl
>Priority: Major
>  Labels: admin-interface, beginner, documentation, easyfix, 
> pull-request-available
> Fix For: jcs-3.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> {{ElementAttributes}} class has a field to store the size in bytes (e.g., for 
> use in an admin-style page). Calculating the size is an expensive operation, 
> so it makes sense to save the size whenever we serialize the element. Admin 
> pages that want this data could then get a cheap approximation from the last 
> time the element was serialized.
> Additionally, the javadocs say the size field is populated as -1 by default, 
> but it's really 0 by default.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Work logged] (JCS-193) [JCACHE] How to configure caches used by interceptors (i.e. CacheResultInterceptor) related to JCache annotations

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/JCS-193?focusedWorklogId=261011=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261011
 ]

ASF GitHub Bot logged work on JCS-193:
--

Author: ASF GitHub Bot
Created on: 16/Jun/19 10:12
Start Date: 16/Jun/19 10:12
Worklog Time Spent: 10m 
  Work Description: sebbASF commented on pull request #3: JCS-193: [CDI] 
[JCache annotations] Set configuration more friendly.
URL: https://github.com/apache/commons-jcs/pull/3
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261011)
Time Spent: 10m
Remaining Estimate: 0h

> [JCACHE] How to configure caches used by interceptors (i.e. 
> CacheResultInterceptor) related to JCache annotations
> -
>
> Key: JCS-193
> URL: https://issues.apache.org/jira/browse/JCS-193
> Project: Commons JCS
>  Issue Type: Improvement
>  Components: Composite Cache
>Affects Versions: jcs-2.2
>Reporter: Ricard Nàcher Roig
>Priority: Major
>  Labels: pull-request-available
> Fix For: jcs-3.0
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> I started using JCS cache as a JCache implementation provider a few days ago 
> and everything was fine until I needed to set up expiration for cached data 
> or even the number of elements stored in the cache. I realized that caches 
> were created with EternalPolicy so I started to figure out how to set up both 
> implementation specific properties and Standard JCache ones. After a couple 
> of days, I don't have a pretty idea about how to configure the caches related 
> to methods annotated with JCache annotations like CacheResult, CacheRemove 
> and so on.
> I mean that I already know that I can use the CacheDefaults annotation, both 
> at class and method level, in order to set the name of the cache to be used. 
> but another thing is to also set the CacheResolverFactory. Should I always 
> set the CacheResolverFactory in the annotation in order to be able to create 
> the cache using a specific set of properties? Or maybe the idea is to create 
> the cache programmatically before is is needed? Anyway, It seems too much 
> boilerplate to me. Maybe I am wrong but I think that we are not using the CDI 
> power at all.
> please, let me know if there is better/simpler approach.
> Another thing that I cannot understand is the readConfig method of the 
> JCSCachingManager class. It is loading the folder structure because it is 
> loading the resources from the default uri of the provider! 
> where uri is "jcs://jcache.ccf", so the path is "/". I think that this URI 
> should not be used for this, maybe the uri should be something different (URI 
> of the default configuration?) or just use another implementation specific 
> property.  Notice that adding an extra slash will look for /jcache.ccf file! 
> maybe it is just a bug and there is a slash missing : "jcs:///jcache.ccf"
> {code:java}
>  final Enumeration resources = loader.getResources(uri.getPath());
> {code}
>  
> What about this?
> Add CachingProvider and CacheManager as beans if they are not already beans. 
> Notice that this is what jcs-jcache-extras module already does! 
> So as a developer I can configure my CachingProvider and even the cache 
> manager already configured as beans.  They will be injected where needed 
> instead of being created without configuration.
> no configuration in CacheResolverFactory 
> {code:java}
> public CacheResolverFactoryImpl()
> {
>   provider = Caching.getCachingProvider();
>   cacheManager = provider.getCacheManager(provider.getDefaultURI(), 
> provider.getDefaultClassLoader());
> }
> {code}
> but in JCacheFilter class some properties are set
>  
> {code:java}
> manager = provider.getCacheManager(URI.create(uri), classLoader, properties);
> {code}
> One step further into CDI should be to also provide the CacheResolverFactory 
> as a bean and also the 
> CacheResolverFactoryImpl could be also injected with the cacheProvider and 
> CacheManager and some CompleteConfigurationResolver of something like that in 
> order to avoid creating caches with a hard-coded MutableConfiguration. 
> {code:java}
> public class CacheResolverFactoryImpl implements CacheResolverFactory
> {
> ...
> private Cache createCache(final String exceptionCacheName)
> {
>   cacheManager.createCache(exceptionCacheName, new   
> 

[jira] [Work logged] (CONFIGURATION-715) PropertiesConfiguration should not trim whitespace from the property value

2019-06-16 Thread ASF GitHub Bot (JIRA)


 [ 
https://issues.apache.org/jira/browse/CONFIGURATION-715?focusedWorklogId=261009=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-261009
 ]

ASF GitHub Bot logged work on CONFIGURATION-715:


Author: ASF GitHub Bot
Created on: 16/Jun/19 10:12
Start Date: 16/Jun/19 10:12
Worklog Time Spent: 10m 
  Work Description: sebbASF commented on pull request #14: 
[CONFIGURATION-715/716]: Updated the User's guide to include JupIOFac…
URL: https://github.com/apache/commons-configuration/pull/14
 
 
   
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 261009)
Time Spent: 10m
Remaining Estimate: 0h

> PropertiesConfiguration should not trim whitespace from the property value
> --
>
> Key: CONFIGURATION-715
> URL: https://issues.apache.org/jira/browse/CONFIGURATION-715
> Project: Commons Configuration
>  Issue Type: Bug
>Affects Versions: 2.3
>Reporter: Patrick Schmidt
>Priority: Major
> Fix For: 2.4
>
>  Time Spent: 10m
>  Remaining Estimate: 0h
>
> Currently, PropertiesConfiguration will trim all trailing whitespace from 
> property values. This is different from how java.util.Properties behaves. The 
> Javadoc explicitly states
> {code:java}
> All remaining characters on the line become part of the associated element 
> string{code}
> The responsible code is in the method _readProperty()_ of _PropertiesReader_.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] [commons-proxy] sebbASF closed pull request #1: PROXY-25 Adds various utility methods dealing with Spring proxies

2019-06-16 Thread GitBox
sebbASF closed pull request #1: PROXY-25 Adds various utility methods dealing 
with Spring proxies
URL: https://github.com/apache/commons-proxy/pull/1
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF closed pull request #2: Javadoc Fixes

2019-06-16 Thread GitBox
sebbASF closed pull request #2: Javadoc Fixes
URL: https://github.com/apache/commons-jcs/pull/2
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF closed pull request #11: Configuration 703

2019-06-16 Thread GitBox
sebbASF closed pull request #11: Configuration 703
URL: https://github.com/apache/commons-configuration/pull/11
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF closed pull request #14: [CONFIGURATION-715/716]: Updated the User's guide to include JupIOFac…

2019-06-16 Thread GitBox
sebbASF closed pull request #14: [CONFIGURATION-715/716]: Updated the User's 
guide to include JupIOFac…
URL: https://github.com/apache/commons-configuration/pull/14
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF closed pull request #4: JCS-195 Update element attributes size on serialization

2019-06-16 Thread GitBox
sebbASF closed pull request #4: JCS-195 Update element attributes size on 
serialization
URL: https://github.com/apache/commons-jcs/pull/4
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF closed pull request #15: CONFIGURATION-729: fixes inconsistency in example

2019-06-16 Thread GitBox
sebbASF closed pull request #15: CONFIGURATION-729: fixes inconsistency in 
example
URL: https://github.com/apache/commons-configuration/pull/15
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-jcs] sebbASF closed pull request #3: JCS-193: [CDI] [JCache annotations] Set configuration more friendly.

2019-06-16 Thread GitBox
sebbASF closed pull request #3: JCS-193: [CDI] [JCache annotations] Set 
configuration more friendly.
URL: https://github.com/apache/commons-jcs/pull/3
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-configuration] sebbASF closed pull request #16: [CONFIGURATION-731] Allow user to specify the comments & separator ch…

2019-06-16 Thread GitBox
sebbASF closed pull request #16: [CONFIGURATION-731] Allow user to specify the 
comments & separator ch…
URL: https://github.com/apache/commons-configuration/pull/16
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [commons-logging] sebbASF closed pull request #1: default methods to Log interface to prevent unnecessary payload creation

2019-06-16 Thread GitBox
sebbASF closed pull request #1: default methods to Log interface to prevent 
unnecessary payload creation
URL: https://github.com/apache/commons-logging/pull/1
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services