Web Service que se ejecuta varias veces al mismo tiempo

[Recca]

Buenos días,

Estoy teniendo un problema que no estoy segura de cómo resolver.

He montado un servicio Web con distintos métodos, uno de los cuáles consume otro servicio Web para obtener información de una herramienta. Básicamente mi método obtiene un ID a partir de un nombre y realiza una serie de updates.

El problema es que se ejecutan varias llamadas a mi método y algunas veces al mismo tiempo. Por ejemplo:

- Se ejecutan 2 llamadas a la vez, la primera con el parámetro nombre1 y la segunda con el parámetro nombre2.
- La primera de ellas obtiene el resultado ID1 y lo almacena en una variable, pero antes de que termine de ejecutarse, la segunda de ellas obtiene el resultado ID2, lo que hace que (esto ya no entiendo por qué) el valor de la variable donde almaceno el ID de la primera llamada se modifique.
- Al final el resultado es que en mitad de la ejecución de la primera llamada, la variable cambia del valor ID1 al ID2 y posteriormente se realiza la update sobre el registro equivocado.

Actualmente la variable donde almaceno el ID la tengo declarada a nivel de clase, como un atributo de la misma, ¿serviría si en lugar de esto la creo como un atributo del método?

Muchas gracias por adelantado.

Un saludo.

[Malenko]

Tiene toda la pinta de que la variable "ID" del webservice esté definida como static. Es así?

[Recca]

No, eso es lo raro :(

A no ser que Java la defina así por defecto cuando no se especifica nada

[Malenko]

Usas java? Si quieres lo muevo al foro de Java que lo verán más programadores de este lenguaje y te podrán ayudar mejor.

[Recca]

Sí, por favor. Disculpad mi ignorancia, soy nueva :S

Muchas gracias! :)

[Xerelo]

Es difícil saber con tan poca información, ni siquiera has dicho qué tipo de servicio es ni qué usas para crearlo.

Tampoco hace falta que pongas todo el código, pero sí las partes importantes. Se me ocurre que el problema no está en en el servicio que recibe, si no en el cliente que usas para hacer las peticiones a los otros sericios.

[Recca]

Es cierto, intentaré aportar algo más de información:

- El servidor donde tengo publicado mi servicio es un Glassfish.
- Mi servicio está desarrollado en JAX-WS, aunque el mismo, a su vez, es un cliente que consume el servicio de otra herramienta usando Axis2 (dadas las necesidades del proyecto, esta era la única posibilidad).

Un ejemplo similar de mi servicio sería algo así:

Código:

class Modificador
{
	// Objeto para almacenar la estructura de una persona (objeto personalizado de la herramienta destino)
	Persona pers;

	...
	...
	...


	public cambiar_edad (string nombre, int edad)
	{
		string query = "SELECT * FROM personas WHERE nombre = " + nombre;

		// El método executeQuery devuelve un objeto de tipo "Persona"
		pers = servicio.executeQuery (query);

		...
		...

		// En el momento en que se están haciendo estas operaciones es cuando llega otra llamada al mismo método
		// y machaca el valor de la variable pers.

		...
		...

		pers.edad = edad;
		servicio.update(pers);	//Por tanto, el update se realiza sobre el registro que no debe
	}

	...
	...
}

No sé si se entiende bien... Yo creo que el problema está en el Glassfish que no gestiona bien las conexiones, pero creo que podría solucionarlo directamente en el código mediante la variable pers, quizá debería declararla a nivel de método en lugar de hacerlo a nivel de clase, o quizá ponerle un synchronized al método "cambiar_edad". El problema es que no estoy segura y no puedo hacer cambios a la ligera porque no hay entorno de desarrollo :(

Muchas gracias.

Un saludo.

[chuidiang]

Hola:

Glassfish entiendo que gestiona bien las conexiones. Si has puesto un web service, lo normal es que sea una clase que esté en un servlet espceífico de jax-ws y no se haga mas que una sola instancia de esa clase en todo glassfish. Cualquier atributo de la clase sea o no static (en tu ejemplo Persona pers) es compartido para todas las llamadas, por lo que si tienes varias peticiones simultaneas (una petición sin que haya acabado la anterior), pueden "machacar" ese atributo.

Intenta hacerlo usando solo variables locales de los métodos (en el ejemplo que pones es inmediato) y si no es posible, guarda esos valores bien en la sesión web, o bien déjalo como atributo de clase pero usa "synchronized" desde el primer método donde recibes la llamada, aunque no es buena idea porque un cliente tendrá que esperar a que termine el anterior.

Se bueno.

[Recca]

Muchas gracias!

Probaré entonces declarando la variable como local en los métodos que la utilice.

De todas formas, me resulta raro que Glassfish gestione así las llamadas. Por sentido común (tal como yo lo entiendo) cada llamada debería ser independiente de las demás, con sus propias variables y su espacio reservado para las mismas.

En fin, se ve que todavía me queda mucho por aprender :P

Muchas gracias de nuevo!

[chuidiang]

No es cosa de Glassfish, es cosa de cualquier servidor web. Cuando pones una aplicación (un .war o un .ear), sólo se instancian una vez las clases de la aplicación, no se instancia la aplicación completa cada vez que entra un cliente nuevo. Sería muy ineficiente instanciar toda la aplicación cada vez que un cliente nuevo haga una llamada. El servidor web lo único que hace es crear hilos nuevos para cada cliente, por ello tu aplicación web debe ser "thread safe".

Se bueno.